4x3 Keypad Matrix

[Florian (Gast)]

Servus Leute,

Ich habe hier ein 4x3 Keypad mit
PA3:PA0 ^= Row    (OUT)
PA6:PA4 ^= Column (IN)
Externe Pullups an den Eingängen vorhanden.

Matrix:

    PA6  PA5  PA4
PA3  x    x    x
PA2  x    x    x
PA1  x    x    x
PA0  x    x    x

Ich setzte nach der Reihe die Reihen PA3 bis PA0 auf High und lese an 
PA6 bis PA4, ob der Taster gedruckt worden ist.

Aber irgendetwas scheine ich falsch zu machen.
Kann mir jemand sagen, wass ich da falsch mache, abgesehen vom return?

uint8_t key_check2(){
  uint8_t delay   = 100;
  uint8_t row    = 0;
  uint8_t col    = 0;
  //Ausgänge
  DDRA |= (1<<PA0) | (1<<PA1) | (1<<PA2) | (1<<PA3);
  
  //Eingänge
  DDRA &= ~(1<<PA4) & ~(1<<PA5) & ~(1<<PA6);
  
  for(uint8_t i=0;i<4;i++){
    
    if(i==0){
      PORTA |=  (1<<PA3);
      PORTA &= ~(1<<PA2) & ~(1<<PA1) & ~(1<<PA0);
      row = 1;
    }else if(i==1){
      PORTA |=  (1<<PA2);
      PORTA &= ~(1<<PA3) & ~(1<<PA1) & ~(1<<PA0);
      row = 2;
    }else if(i==3){
      PORTA |=  (1<<PA1);
      PORTA &= ~(1<<PA3) & ~(1<<PA2) & ~(1<<PA0);
      row = 3;
    }else if(i==3){
      PORTA |=  (1<<PA0);
      PORTA &= ~(1<<PA3) & ~(1<<PA2) & ~(1<<PA1);
      row = 4;
    }
    
    if(PINA & (1<<PA6)){
      _delay_ms(delay);
      col = 1;
    }else if(PINA & (1<<PA5)){
      _delay_ms(delay);
      col = 2;
    }else if(PINA & (1<<PA4)){
      _delay_ms(delay);
      col = 3;
    }
  }
  return (row*col);
}

Danke!

Florian

[Stefan F. (sfrings)]

Funktionieren die Pull-Up WIderstände? Wenn sie zu hochohmig sind, wird 
es nicht klappen, denn du fragst die Eingänge ziemlich schnell nach dem 
Setzen der Ausgänge ab. Ich schätze 10k Ohm oder weniger sollten gehen.

Was passiert denn zur Zeit? Du hast geschrieben, dass es nicht 
funktioniert, aber nicht, wass denn stattdessen passiert.

[Stefan F. (sfrings)]

Wenn Du mit Pull-Up Widerständen arbeitest, musst Du die Ausgänge auf 
LOW setzen und die Eingänge auf LOW prüfen. Dein Code funktioniert nur 
mit Pull-Down Widerständen.

Korrekturvorschlag:

uint8_t key_check2() {
  // Ein/Ausgänge konfigurieren
  DDRA = (1<<PA0) | (1<<PA1) | (1<<PA2) | (1<<PA3);

  // erste Reihe ansteuern
  PORTA = ~PA3;

  for (uint8_t row=1; row<5; row++) {

    // Rückgabewert entsprechend der gedrückten Spalte berechnen
    if (!(PINA & (1<<PA6))) {
      delay(100);
      result=1*row;
    }
    else if (!(PINA & (1<<PA5))) {
      delay(100);
      return 2*row;
    }
    else if (!(PINA & (1<<PA4))) {
      delay(100);
      return 3*row;
    }

    // nächste Reihe ansteuern
    PORTA = (PORTA>>1) | 128;
  } 

  // keine Taste gedrückt
  return 0;
}

Als Nebeneffekt aktiviert dieser Code die internen Pull-Up Widerstände. 
Ich hoffe, das stört Dich nicht. Wenn Du ein bisschen Zeit zwischen dem 
Setzen der Ausgänge und dem Lesen der Eingänge einfügst, kannst Du auf 
externe Pull-Up's verzichten:

  for (uint8_t row=1; row<5; row++) {

    // Ein bisschen Zeit verplempern
    (PINA); (PINA);

    // Rückgabewert entsprechend der gedrückten Spalte berechnen
    ...
  }

Diese beiden Lesezugriffe auf PINA werden vom Compiler nicht weg 
optimiert.

[Florian (Gast)]

Servus Stefan,

danke für den Hinweis mit den Pullups!
Diesen blöden Fehler hab ich noch nie gemacht...

jetzt funktioniert der Code auch ;-)

Besten Gruß,
Florian