Port-Pins global in C definieren?

[El Noob (Gast)]

Hallo alle zusammen!

Ich habe folgendes Problem, auf einem vorgefertigten Board sind beim 
PORTE die Anschlüsse verschoben und somit leuchten die LEDs wie 
folgendermaßen:

                 PORTE = (0x01);        // LED 8   PE0
     PORTE = (0x02);  // LED 1   PE1
     PORTE = (0x04);  // LED 2   PE2
     PORTE = (0x08);  // LED 3   PE3
     PORTE = (0x10);  // LED 4   PE4
     PORTE = (0x20);  // LED 5   PE5
     PORTE = (0x40);  // LED 6   PE6
     PORTE = (0x80);  // LED 7   PE7

Ich hätte es natürlich gerne so:

                 PORTE = (0x01);        // LED 1   PE0
     PORTE = (0x02);  // LED 2   PE1
     PORTE = (0x04);  // LED 3   PE2
     PORTE = (0x08);  // LED 4   PE3
     PORTE = (0x10);  // LED 5   PE4
     PORTE = (0x20);  // LED 6   PE5
     PORTE = (0x40);  // LED 7   PE6
     PORTE = (0x80);  // LED 8   PE7

Umlöten kann man da leider garnix, ist auch nicht von mir das Board. Wie 
kann ich die Ports nun vor der main global in C definieren um diesem 
f**k-up zu entgehen? Oder gibt es andere Lösungsansätze?

[Guest (Gast)]

Du machst einfach

#define LED_DDR DDRE
#define LED_PORT PORTE
#define LED1 (1<<7)
#define LED2 (1<<0)
...

Und dann:

LED_PORT |= LED1

[El Noob (Gast)]

Hmm? Danke erst mal für die Antwort aber das verstehe ich jetzt 
überhaupt nicht, wozu LED definieren? Also so sieht es jetzt zu 
Testzwecken aus:

#include <avr/io.h>

int main(void)
{
  DDRE  = 0xFF;
  PORTE = 0xFF;

    while(1)
    {
                 PORTE = (0x01);        /* LED 8 */
     PORTE = (0x02);  /* LED 1 */
     PORTE = (0x04);  /* LED 2 */
     PORTE = (0x08);  /* LED 3 */
     PORTE = (0x10);  /* LED 4 */
     PORTE = (0x20);  /* LED 5 */
     PORTE = (0x40);  /* LED 6 */
     PORTE = (0x80);  /* LED 7 */
    }
}

Im Kommentar steht nur wie es jetzt gerade ist, die erste LED soll 
natürlich mit PE0 und die zweite mit PE1 ... leuchten.

[Peter D. (peda)]

#include "sbit.h"

#define LED8 PORT_E0
#define LED1 PORT_E1
// ...
  LED8 = 1; // LED 8 an
  LED1 = 1; // LED 1 an

[micha (Gast)]

El Noob schrieb:
> Im Kommentar steht nur wie es jetzt gerade ist, die erste LED soll
> natürlich mit PE0 und die zweite mit PE1 ... leuchten.

Du kannst nicht die physikalische Port-LED Zuordnung (LED1 == PE1) 
per Programm änderen (jedenfalls nicht bei AVRs). Was du allerdings 
machen kannst (und was oben schon gesagt wurde), sorg dafür, dass die 
physikalische Zuordnung richtig auf die logische/gewünschte Zuordnung 
abgebildet wird.

Anstatt "PORTE = (0x01);" schreibst Du einfach "LED_ON(LED1);" oder 
"LED_OFF(LED1);"(kann man sowiso besser lesen).

Dann musst Du noch die entsprechenden Funcktionen/Makros "LED_ON" und 
"LED_OFF" definieren z.B.

#define LED_PORT PORTE
#define LED1 PE2
#define LED2 PE4
....

#define LED_ON(led)  LED_PORT |= (1<<(led))
#define LED_OFF(led) LED_PORT &= ~(1<<(led))

[Karl H. (kbuchegg)]

.... und wenn du das dann auch noch vernünftig machen willst, dann 
benutzt du von vorne herein gleich Namen, die die Funktion der LED 
wiederspiegeln (egal ob du das jetzt mit PeDas Lösung machst oder die 
reine Makrolösung bevorzugst.)

Denn dann steht im Code beispielsweise

   ...
   LED_ON( ERROR_LED );
   LED_OFF( READY_LED );
   ...

und das dokumentiert im Code selber schon viel besser, was der Zweck des 
Code-Stückes ist. Nämlich in diesem Fall dem Benutzer das Auftreten 
eines Fehlers durch Aufleuchten der mit 'Error' beschrifteten LED 
anzuzeigen.

Das diese LED natürlich an irgendeinem Portpin hängen muss, ist klar. 
Aber diese Information gibt es nur an einer einzigen Stelle im Programm 
zusammengefasst. Nämlich hier

#define ERROR_LED   PE2
#define READY_LED   PE4

Dem restlichen Programmtext ist das piepschnurzegal, ob eine spezielle 
LED nun an PE2 oder doch an PE0 hängt. Diese Begriffe kommen im Programm 
selber nicht mehr vor, sondern nur noch ihre durch #define eingeführten 
Ersatznamen. Das ist zum einen in der Tradition des 'Code sollte nach 
Möglichkeit selbstkommentierend sein' und zum anderen ist es eine große 
Hilfe, wenn die LED (am selben Port) auf einen anderen Pin 'umziehen' 
muss. Dann ändert man einfach nur dieses eine #define und hat diese 
Anpassung erledigt.

> ... aber das verstehe ich jetzt überhaupt nicht,
> wozu LED definieren? ....
> ...
> ... Im Kommentar steht nur wie es jetzt gerade ist

genau das ist der Zweck: Den Kommentar komplett überflüssig zu machen. 
Denn dann kann man ihn weglassen ohne dass das Programm weniger schlecht 
zu lesen wäre. Das Weglassen können ist wiederrum etwas gutes. Denn ein 
Kommentar, der nicht dasteht, kann auch nicht falsch sein. Wenn du die 
Wahl hast, zwischen

   PORTE |= ( 1 << PE2 );     // Error LED einschalten

und

   LED_ON( ERROR_LED );

dann ist zweites zu bevorzugen. Denn hier steht bereits im Code das, was 
sonst mühsam (und fehlerträchtig) kommentiert werden müsste. Im Code 
steht das, was tatsächlich passiert. Denn ob der Kommentar korrekt ist 
oder nicht hängt ganz allein lediglich von deiner Disziplin ab, jegliche 
Codeänderungen auch auf den Kommentar auszudehen.

[Oliver (Gast)]

Der wesentlich Vorteil von

   LED_ON( ERROR_LED );

ist, daß man damit immer weiß, was gemeint ist, egal, ob die LED nun 
gegen VCC oder gegen Masse schaltet.

Oliver