LED einschalten - Code Variante

[Ingo (Gast)]

Hallo zusammen,

habe mal eine Anfängerfrage:

In einem ersten Testprogramm möchte ich eine LED an PORTB1 einschalten.

Kann ich über eine define-Anweisung erreichen das ich nur schreiben 
brauche
Z.B.

#define LED1 PB1
...
DDRB = 0x0F;
...

LED1 = 1;     ???


Ich möchte einen anderen weg ausprobieren als PORTB |= (1<<PB1) zum 
Beipsiel.
Ist das möglich bzw. macht das in euren Augen einen sinn?

[Jörg W. (dl8dtl) (Moderator)]

Ingo schrieb:

> Ich möchte einen anderen weg ausprobieren als PORTB |= (1<<PB1) zum
> Beipsiel.

Effektiv wird es immer darauf hinauslaufen müssen (und der Compiler
wird daraus den passenden SBI-Befehl basteln, wenn man ihn
optimieren lässt).

Wenn du nun Wert auf Portierbarkeit legst, kannst du sowas
schreiben:

#define GLUE(a, b)     a##b
#define SET_(what, p, m) GLUE(what, p) |= (1 << (m))
#define CLR_(what, p, m) GLUE(what, p) &= ~(1 << (m))
#define SET(what, x) SET_(what, x)
#define CLR(what, x) CLR_(what, x)

#define LED1 B, 1

#include <avr/io.h>
#define F_CPU 1000000UL
#include <util/delay.h>

void port_init(void)
{
  SET(DDR, LED1);  /* enable output for LED1 */
}

void flash_led1(void)
{
  SET(PORT, LED1);
  _delay_ms(100);
  CLR(PORT, LED1);
}

[Jörg W. (dl8dtl) (Moderator)]

p.s.: Hier der daraus generierte Assemblercode für einen ATmega88:

.global port_init
        .type   port_init, @function
port_init:
/* prologue: frame size=0 */
/* prologue end (size=0) */
        sbi 36-0x20,1
/* epilogue: frame size=0 */
        ret
/* epilogue end (size=1) */
/* function port_init size 2 (1) */
        .size   port_init, .-port_init
.global flash_led1
        .type   flash_led1, @function
flash_led1:
/* prologue: frame size=0 */
/* prologue end (size=0) */
        sbi 37-0x20,1
        ldi r24,lo8(25000)
        ldi r25,hi8(25000)
/* #APP */
        1: sbiw r24,1
        brne 1b
/* #NOAPP */
        cbi 37-0x20,1
/* epilogue: frame size=0 */
        ret
/* epilogue end (size=1) */
/* function flash_led1 size 9 (8) */
        .size   flash_led1, .-flash_led1

[technikus (Gast)]

Hallo,


warum nicht so ?

//Makros zur Bitverarbeitung
#define SET(x,y)  (x)|=(1<<(y))  //Bit set
#define CLR(x,y)  (x)&=~(1<<(y))  //Bit clear
#define TOGGLE(x,y)  (x)^=(1<<(y))  //Bit toggle

#define READ(x,y)  (x)&(1<<(y))  //Bit read

//Bit setzen
SET(DDRB,PB1);

sondern mit glue (noch nie gehört) ???


Daniel

[Gast (Gast)]

GLUE ist nur der Makroname, könntest auch KLEBEFIX nennen.

[Jörg W. (dl8dtl) (Moderator)]

technikus schrieb:

> warum nicht so ?

Weil du dann an mehr als einer Stelle ändern musst, wenn deine LED
demnächst nicht mehr an Port B sondern E klemmt.