zwei Pins vom Mikrocontroller im define

[Mark (Gast)]

Hi,
ich habe einen dspic30f4011. ich möchte gerne, dass wenn ich eine 
Variable aufrufe, der entsprechende Pin auf High geht.

Für einen Pin habe ich es so gemacht

...
#define Power LATDbits.LATD3
...
...
Power = 1;
...
Power = 0;
...

Jetzt möchte ich beim Aufruf aber nicht nur einen Pin auf High/Low 
schalten, sondern zwei. Aber wie bekomme ich zwei Pins in einem define 
rein?

Oder gibt es da eine andere Methode zu?

[DirkF (Gast)]

So


...
#define Power LATDbits.LATD3; LATDbits.LATD4
...
...
Power = 1;
...
Power = 0;
...

[DirkF (Gast)]

Sorry, war falsch.
So sollte es gehen:


...
#define Power_on LATDbits.LATD3=1; LATDbits.LATD4=1;
#define Power_off LATDbits.LATD3=0; LATDbits.LATD4=0;
...
...
Power_on
...
Power_off
...

[Mark (Gast)]

ah,
so hätte ich nicht gedacht. aber Perfekt.

Danke

[N. G. (newgeneration)]

DirkF schrieb:
> #define Power_on LATDbits.LATD3=1; LATDbits.LATD4=1;
> #define Power_off LATDbits.LATD3=0; LATDbits.LATD4=0;

schlecht, weil:

//...
if(x) 
   Power_on;

//wird umgesetzt zu:

if(x)
    LATDbits.LATD3=1;
LATDbits.LATD4=1;

ich habs extra so eingerückt dass man den Fehler sieht.

Besser:

#define Power_on do{LATDbits.LATD3=1; LATDbits.LATD4=1;}while(0)
#define Power_off do{LATDbits.LATD3=0; LATDbits.LATD4=0;}while(0)

[H.Joachim S. (crazyhorse)]

#define Power_on {LATDbits.LATD3=1; LATDbits.LATD4=1;}

sollte reichen, oder?

[Karl H. (kbuchegg)]

H.Joachim S. schrieb:
> #define Power_on {LATDbits.LATD3=1; LATDbits.LATD4=1;}
>
> sollte reichen, oder?

Nein.

  if( irgendwas )
    Power_on;
  else
    ...

unmatched else

der ; ist in diesem Fall dann im Weg.
Man könnte ihn natürlich weg machen

  if( irgendwas )
    Power_on
  else
    ...

aber das sieht dann reichlich seltsam aus.

[Thomas E. (thomase)]

H.Joachim S. schrieb:
> sollte reichen, oder?
Nein.

Steht doch schon oben, wie es richtig ist:

N. G. schrieb:
> Besser:

#define Power_on do{LATDbits.LATD3=1; LATDbits.LATD4=1;}while(0)
#define Power_off do{LATDbits.LATD3=0; LATDbits.LATD4=0;}while(0)

mfg.

[Dirk F (Gast)]

Noch besser ist es, das in eine Funktion zu packen.
Spart Programmspeicher bei häufigen Aufrufen.


void Power (unsigend char switch)
   {
   LATDbits.LATD3=switch;
   LATDbits.LATD4=switch;
   }


Power (1);  // zum Einschlten
Power (0);  // zum Ausschlten