Knopf geht nicht!?

[Gernot F. (gernotfrisch)]

Wenn ich nix mach, steht da "Enter" =4. Wenn ich GND auf die pins
lege, geht nur der 3er ("right"). Die anderen scheinen keine Reaktion
zu zeigen. Ist schon spät. Liegt vielleicht daran ;)

void HandleKeys(void)
{
  if(KEY_VALID) return;

  for (char i=0;i<5;i++)
  {
    if(KEY_PIN & (1<<i))
    {
      KEY_VALID = 1;
      KEY = i;
    }
  }
}
int main(void)
{  
    lcd_init();
  KEY_DDR = 0xff; KEY_PORT = 0xff;
  switch(GetKey())
  {
  case 0: lcd_string_P ("UP"); break;
  case 1: lcd_string_P ("LEFT"); break;
  case 2: lcd_string_P ("DOWN"); break;
  case 3: lcd_string_P ("RIGHT"); break;
  case 4: lcd_string_P ("ENTER"); break;
  }
}

[Gernot F. (gernotfrisch)]

PS. Ich ruf in main() schon immer wieder HandleKEys auf. Klar, oder?

[Alex Trusk (Gast)]

was bewirkt folgende zeile?
KEY_DDR = 0xff; KEY_PORT = 0xff;

[MasterFX (Gast)]

Ohne es zu wissen würde ich mal sagen dass it KEY_DDR = 0xff; alle
PortPins auf Ausgang geschaltet werden und mit KEY_PORT = 0xff; eben
die  Ports auf High gesetzt werden.

[Alex Trusk (Gast)]

korrekt. und wenn man dann gnd an die pins legt, machts brrrzzzzt...

zum glueck sind avrs hart im nehmen. davon mal ab, ist der code eher
suboptimal. hier ein bisschen lesestoff, wie man taster abfragt:

http://www.mikrocontroller.net/articles/Entprellung

[Peter D. (peda)]

"Ich ruf in main() schon immer wieder HandleKEys auf. Klar, oder?"


Nö, überhaupt nicht klar !

Ich sehe da nur lcd_init(), lcd_string_P(), GetKey(), welche nirgends
definiert sind und auch sämtliche Variablen und Konstanten fehlen.


Was ist denn daran bloß so schwierig, Copy+Paste zu benutzen, d.h.
exakt das hinzuschreiben, was man auch ausprobiert hat ???

Und Code als Anhang sollte doch auch klar sein.

Für uncompilierbare und falsche Codefragmente braucht man sich nicht
wundern, wenns darauf keine Hilfe gibt.



Peter

[Gernot F. (gernotfrisch)]

Ich Depp! Es muss so sein:

int main(void)
{
lcd_init();
KEY_DDR = 0x00 KEY_PORT = 0xff; // input + pullup
switch(GetKey())
{
case 0: lcd_string_P ("UP"); break;
case 1: lcd_string_P ("LEFT"); break;
case 2: lcd_string_P ("DOWN"); break;
case 3: lcd_string_P ("RIGHT"); break;
case 4: lcd_string_P ("ENTER"); break;
}
}

und bei GetKey muss ich (! (KEY_PIN & i)) abfragen, damit er gedrückt
ist, nicht? GND == 0!?

Was so ein bischen Schlaf alles ausmacht... ;)