nested #if makro

[nest (Gast)]

wie macht man sowas hier richtig?

#define LED_SET_ON(PRT,PIN,INV)\
  #if(INV == 0)\
    PORT##PRT |= (1<<PIN)\
  #else\
    PORT##PRT &=~(1<<PIN)

#define LED_SET_OFF(PRT,PIN,INV)\
  #if(INV == 1)\
    PORT##PRT |= (1<<PIN)\
  #else\
    PORT##PRT &=~(1<<PIN)

Im code bzw. in weiteren makros will ich es so nutzen:

LED_SET_ON(D,5,1);  // PD5 low  -> LED an
LED_SET_ON(D,6,0);  // PD6 high -> LED an
(...)
LED_SET_OFF(D,5,1); // PD5 high -> LED aus
LED_SET_OFF(D,6,0); // PD6 low  -> LED aus

Die LED-Markos sind noch falsch definiert, sodass der Compiler einen 
Fehler wirft:

myfile.c:203:31: error: '#' is not followed by a macro parameter

daher werden die Markos auch im code als Funktionen mit Parametern 
interpretiert:

LED_SET_OFF(D,5,1)

myfile.c:223:16: error: 'D' undeclared (first use in this function)

[nest (Gast)]

Stromlauf, falls man es sich nicht schon aus dem Code-Beispiel herleiten 
kann:

uC
 ........
 .      .    LED1
 .  PD5 o-----|<|--R---Vcc
 .      .
 .      .
 .      .    LED2
 .  PD6 o-----|>|--R---GND
 .      .
 ........

[nest (Gast)]

ok ... hab es doch selbst gefunden:

#define LED_SET_ON( PRT,PIN,INV) ((INV)==0)?(PORT##PRT|=(1<<PIN)):(PORT##PRT&=~(1<<PIN))
#define LED_SET_OFF(PRT,PIN,INV) ((INV)!=0)?(PORT##PRT|=(1<<PIN)):(PORT##PRT&=~(1<<PIN))

[Keiner N. (nichtgast)]

Warum brauchst du zwei Makros, die eigentlich das gleiche tun?

[Rolf M. (rmagnus)]

Wenn man meint, das zu brauchen, könnte man auch einfach schreiben:

#define LED_SET_OFF(PRT,PIN,INV) LED_SET_ON(PRT, PIN, !(INV))

[Peter D. (peda)]

Beitrag "Re: Port als Variable"

[(prx) A. K. (prx)]

nest schrieb:
> wie macht man sowas hier richtig?

#if Statements sind so nicht anwendbar. Sie wirken zum Zeitpunkt der 
Makrodefinition, nicht bei der Verwendung des Makros.

[Eric B. (beric)]

#define DEFINE_LED(NAME,PRT,PIN,INV)   \
  inline void Led_##NAME_Off() {       \
    if(INV) PORT##PRT |= (1<<##PIN);   \
    else    PORT##PRT &= (1<<##PIN); } \
  inline void Led_##NAME_On()  {       \
    if(INV) PORT##PRT &= (1<<##PIN);   \
    else    PORT##PRT |= (1<<##PIN); }

DEFINE_LED(Warning,D,5,true)
DEFINE_LED(Power,  D,6,false)

...
Led_Warning_Off();
Led_Power_On();

Und dann hoffen, dass der Compiler alles schön inline-t ;-)

Hat wenigstens den Vorteil dass
* ich mich nur an einer Stelle drum kümmern muss ob eine LED mit 
Negativ-Logik angesprochen werden soll
* ich mich nicht Port und Pin merken muss im Code, sondern eine 
sinnvolle Name benutzen kann,
* und nicht definierte LEDs zur Compile-time schon gefunden werden.