HILFE! Tasten sollen LED einschalten AT90CAN128

mit dem komentar war mehr gemeint : "das soll dazu führen"
aber das tut es ja nicht.

wenn ich folgendes eingebe:

while(1){

           if (PINE==0b00000001) // taster
       {
        PORTA=0b00000001; // leds
       }

        }

dann geht die LED0 an wenn ich TASTER0 drücke.
Daher vermute ich jetzt mal Pull-down.

das hier ist meine include:

#define LED   PORTA   // LEDs an Port A
#define LED0   PORTA0
#define LED1   PORTA1
#define LED2   PORTA2
#define LED3   PORTA3
#define LED4   PORTA4
#define LED5   PORTA5
#define LED6   PORTA6
#define LED7   PORTA7


#define TASTEN   PINE    // Tasten an Port E
#define TASTE0   PINE0
#define TASTE1   PINE1
#define TASTE2   PINE2
#define TASTE3   PINE3
#define TASTE4   PINE4
#define TASTE5   PINE5
#define TASTE6   PINE6
#define TASTE7   PINE7

Erstmal vielen dank für die antworten.
Ich hab jezt nochmal selber ein bischen rumbrobiert und folgendes 
geändert:

int main(){

 DDRA=0xff;  //LED Port =Ausgang
 DDRB=0xff;  //Grafikdisplay und Display Beleuchtung  =Ausgan
 DDRC=0xff;  //LCD Display (Daten & Befehle )  =Ausgang
 DDRE=0x00;  //Taster Port = Eingang
 DDRG=0x00;  //Cursor Port = Eingang


lcd_init(LCD_DISP_ON); //Befehl an Display (Diplay on. Cursor off)

PORTB = (1<<PINB7);  //Beleuchtung LCD einschalten
                     //entspricht :PORTB = 0b10000000:;
lcd_clrscr();
lcd_gotoxy(7,1);   //Angabe in Stelle (horizontal) , Zeile (vertikal)
lcd_puts("Rot");
lcd_clrscr();
lcd_gotoxy(7,1);   //Angabe in Stelle (horizontal) , Zeile (vertikal)
lcd_puts("Grün");
lcd_clrscr();
lcd_gotoxy(7,1);   //Angabe in Stelle (horizontal) , Zeile (vertikal)
lcd_puts("Blau");

unsigned char col = 0;
  while(1){

    if(col==0)      //Beginn LED steuerung
    {
     LED = 0b00000000;
      }
    if(col==1)
    {
     LED = 0b00000001;
      }

    if(CURSOR==OBEN) //Beginn Cursor
    {
     col++;
    }
    if(CURSOR==UNTEN)
    {
     col--;
    }

    }
}//main zu

Das ganze Für den LCD ist erstmal noch nicht so wichtig. Ich will später 
nur mal darauf hinaus, dass ich auf meinem LCD z.B. Blau anzeige und 
dann durch drücken der MITTE, auch die blaue LED angeht.
Ich probiere jetzt erstmal nur mit den internen LEDs auf meinem Board 
rum.

Ich hatte vor durch die "unsigned char col" (durch drücken von OBEN und 
UNTEN) einen Wert rauf zu setzen (++) oder runter (--).
Und dann z.B.: "if (col=1)
            {
             lcd_clrscr();
             lcd_gotoxy(7,1);
             lcd_puts("Blau");
            }
Aber den Display wolte ich erstmal weg lassen.
Das ist noch nicht so wichtig und (ich vermute) auch nicht mehr so ein 
Problem, wenn ich ertmal das mit den LEDs im griff hab.


Mir gehts jetzt erstmal nur darum mit OBEN und UNTEN zwischen
LED0 bis LED3 zu wechseln.

So wie es oben Steht kann ich LED0 mit "OBEN" ein schalten und mit 
"UNTEN"
wieder ausschalten.
Wenn ich jetzt, nach meinem Verständnis nach, versuche die zweite LED 
wie folgt einzubinden:

while(1){

    if(col==0)      //Beginn LED steuerung
    {
     LED = 0b00000000;
      }
    if(col==1)
    {
     LED = 0b00000001;  //LED0
      }
    if(col==2)
    {
     LED = 0b00000010;  //LED1
      }

Leuchtet nur LED1 :-(

Ich versteh das nicht.

nochmal die include mit geänderten Cursorn:

#define LED   PORTA   // LEDs an Port A
#define LED0   PORTA0
#define LED1   PORTA1
#define LED2   PORTA2
#define LED3   PORTA3
#define LED4   PORTA4
#define LED5   PORTA5
#define LED6   PORTA6
#define LED7   PORTA7

#define G_DISPLAY PORTB     //Anschluss Grafikdisplay
#define BELEUCHTUNG PORTB7  //Beleuchtung Display

#define TASTEN   PINE    // Tasten an Port E
#define TASTE0   PINE0
#define TASTE1   PINE1
#define TASTE2   PINE2
#define TASTE3   PINE3
#define TASTE4   PINE4
#define TASTE5   PINE5
#define TASTE6   PINE6
#define TASTE7   PINE7

#define CURSOR   PING   // Cursorblock an Port G
#define LINKS   0b00000001
#define OBEN   0b00000010
#define MITTE   0b00000100
#define RECHTS   0b00001000
#define UNTEN   0b00010000

Christian H. schrieb:
> Meinst Du etwa, die Tastenaktion wird nur einmal beim Drücken des
> Tasters ausgeführt?

So hatte ich mir das eigentlich gedacht.

Damit das ganze also funktioniert muss ich einfach einen delay einbauen
wie z.B 0,3s und dann sollte es ja eig funktionieren (oder?), wenn man 
die taste nicht länger drückt.