Keypad - Interrupt

[Rico H. (Firma: FHNW) (2she)]

Hallo Liebe Leute

ich stehe vor einem kleineren Problem.

Und zwar hab ich ein Displaymodul inklusive Folientastatur 
(Matrix-Ansteuerung) von Display-3000. Ich hab mich dafür entschieden, 
die Kolonnen der Tastaturmatrix auf Interruptleitungen zu ziehen (es 
waren ein,zwei zusätzliche Leitungen auf dem Board notwendig).
Die Interrupts funktionieren einwandfrei. Das Problem ist es jedoch in 
der Interruptroutine die Zeile, d.h. die richtige Taste auszulesen.

Die Interruptroutine sieht zur Zeit folgendermassen aus:

ISR(INT5_vect)
{
  key_press();

}

Im Unterprogramm key_press() wird eine case abfrage der gedrückten Taste 
gemacht, was normalerweise auch funktioniert hat. Ich nehme an, dass das 
Problem darin liegt, dass dieses Unterprogramm nur gerade einmal 
durchlaufen wird und die Taste nicht richtig erkannt wird...

Habt Ihr mir irgendwelche Vorschläge?
Besten Dank schon im Vorraus

[4toTakoe (Gast)]

Rico H. schrieb:
> key_press()

zeigen.

[Karol B. (johnpatcher)]

Ich kenne die Charakteristik von Folientastaturen nicht, aber im 
Allgemeinen ist es immer ein schlechter Ansatz Taster und ähnliches 
direkt per Interrupt auszuwerten, da diese dazu tendieren zu "prellen", 
siehe [1].

Bemühe einfach mal die Suchmachine deiner Wahl (z.B. [2]), um dir 
anzusehen wie es andere gelöst haben. Du bist nicht der erste der eine 
Matrixtastatur per Mikrocontroller auswerten möchte ;).

[1]: https://www.mikrocontroller.net/articles/Entprellung
[2]: 
https://www.google.de/webhp?q=matrix+tastatur+site:mikrocontroller.net

[Peter D. (peda)]

Es stimmt schon, daß man für Tasten einlesen/entprellen einen Interrupt 
benötigt. Es ist allerdings der Timerinterrupt.

[Rico H. (Firma: FHNW) (2she)]

4toTakoe schrieb:
> Rico H. schrieb:
>> key_press()
>
> zeigen.

  void Init_Key_Port()
  {
    /* All Ports input of the matrix keyboard */
    DDRE = ( (0<<DDE7) | (0<<DDE6) | (0<<DDE5) | (0<<DDE4));
    DDRD = ( (0<<DDD7) | (0<<DDD4) );
    DDRG = ( (0<<DDG3) | (0<<DDG0) );
    /* Pullup resistance active for keyboard ports*/
    PORTE = ( (1<<PE7) | (1<<PE6) | (1<<PE5) | (1<<PE4));
    PORTD = ( (1<<PD7) | (1<<PD4) );
    PORTG = ( (1<<PG3) | (1<<PG0) );
  }

  char key_press(int pos, int site)
  {
    cli();
    char number=0;
    int keycode, keyflag = 4;
    Init_Key_Port();                  // make sure all keyboard ports are initialized

    /* check line D */
    PORTD = (0<<PD4);                   // PORT D4 at pullup desactive
    DDRD = (1<<DDD4);                   // PORT D4 in input
    keycode = 0x08 &0x0F;                // line code
    /* check column */
    if ((PINE &(1<<PINE7)) == 0) ;            // faking condition
    else if ((PINE &(1<<PINE5)) == 0) keycode += 0x10 &0xF0;
    else if ((PINE &(1<<PINE4)) == 0) keycode += 0x20 &0xF0;
    else if ((PINE &(1<<PINE6)) == 0) keycode += 0x40 &0xF0;
    else keyflag --;
    PORTD = (1<<PD4);                   // PORT D4 at pullup active
    DDRD = (0<<DDD4);                   // PORT D4 in output

    if (keyflag == 3)
    {
      /* check line C */
      PORTD = (0<<PD7);                 // PORT D7 at pullup desactive
      DDRD = (1<<DDD7);                 // PORT D7 in input
      keycode = 0x04 &0x0F;              // line code
      /* check column */
      if ((PINE &(1<<PINE7)) == 0) ;          // faking condition
      else if ((PINE &(1<<PINE5)) == 0) keycode += 0x10 &0xF0;
      else if ((PINE &(1<<PINE4)) == 0) keycode += 0x20 &0xF0;
      else if ((PINE &(1<<PINE6)) == 0) keycode += 0x40 &0xF0;
      else keyflag --;
      PORTD = (1<<PD7);                 // PORT D7 at pullup active
      DDRD = (0<<DDD7);                 // PORT D7 in output
    }

    if (keyflag == 2)
    {
      /* check line B */
      PORTG = (0<<PG0);                 // PORT G0 at pullup desactive
      DDRG = (1<<DDG0);                 // PORT G0 in input
      keycode = 0x02 &0x0F;              // line code
      /* check column */
      if ((PINE &(1<<PINE7)) == 0) ;          // faking condition
      else if ((PINE &(1<<PINE5)) == 0) keycode += 0x10 &0xF0;
      else if ((PINE &(1<<PINE4)) == 0) keycode += 0x20 &0xF0;
      else if ((PINE &(1<<PINE6)) == 0) keycode += 0x40 &0xF0;
      else keyflag --;
      PORTG = (1<<PG0);                 // PORT G0 at pullup active
      DDRG = (0<<DDG0);                 // PORT G0 in output
    }
    
    if (keyflag == 1)
    {
      /* check line A */
      PORTG = (0<<PG3);                 // PORT G3 at pullup desactive
      DDRG = (1<<DDG3);                 // PORT G3 in input
      keycode = 0x01 &0x0F;              // line code
      /* check column */
      if ((PINE &(1<<PINE7)) == 0) ;          // faking condition
      else if ((PINE &(1<<PINE5)) == 0) keycode += 0x10 &0xF0;
      else if ((PINE &(1<<PINE4)) == 0) keycode += 0x20 &0xF0;
      else if ((PINE &(1<<PINE6)) == 0) keycode += 0x40 &0xF0;
      else keyflag --;
      PORTG = (1<<PG3);                // PORT G3 at pullup active
      DDRG = (0<<DDG3);                 // PORT G3 in output
    }

    /* Switch table which return the value of the pressed key */
    /****************************************************************************/
    /* Change the following code lines to attribute some action for each button */
    /****************************************************************************/
    switch (keycode)
    {
      case 0x21 :  number = '0';
      break;
      case 0x18 : number = '1';
      break;
      case 0x28 : number = '2';
      up();
      break;
      case 0x48 : number = '3';
      break;
      case 0x14 : number = '4';
      break;
      case 0x24 : number = '5';
      break;
      case 0x44 : number = '6';
      break;
      case 0x12 :  number = '7';
      Test_Interrupt();
      break;
      case 0x22 : number = '8';
      down();
      break;
      case 0x42 : number = '9';
      break;
      case 0x00 :  number = 'N';
      break;
      case 0x81 :  number = 'S';
      
      break;
      case 0x41 : number = 'E';
      break;
    }
    
    /* Interrupt Ports input */
    DDRE = ( (0<<DDE7) | (0<<DDE6) | (0<<DDE5) | (0<<DDE4));
    /* Ports output */
    DDRD = ( (1<<DDD7) | (1<<DDD4) );
    DDRG = ( (1<<DDG3) | (1<<DDG0) );
    /* Pullup resistance Interrupt Ports active */
    PORTE = ( (1<<PE7) | (1<<PE6) | (1<<PE5) | (1<<PE4));
    /* Pullup resistance output Ports inactive */
    PORTD = ( (0<<PD7) | (0<<PD4) );
    PORTG = ( (0<<PG3) | (0<<PG0) );
    sei();
    return number;
  }

[Peter D. (peda)]

#include <avr\io.h>

#define NOP();  asm volatile("nop"::);


#define KEY_PIN         PINB
#define KEY_PORT        PORTB
#define KEY_DDR         DDRB

// Pin 0..3 = column 1..4
// Pin 4..6 = row 1..3


uint8_t keyscan( void )
{
  uint8_t col = 0, row = 0;

  KEY_PORT |= 0x7F;                     // all pullups on
  KEY_DDR = (KEY_DDR & 0x80) | 0x70;    // pin 6..4 = output
  KEY_PORT &= 0x8F;                     // pin 6..4 = output low
  NOP();                                // wait until inputs sampled
  if( ~KEY_PIN & 1<<0 )                 // if pin 0 = low
    col = 4;
  if( ~KEY_PIN & 1<<1 )
    col = 3;
  if( ~KEY_PIN & 1<<2 )
    col = 2;
  if( ~KEY_PIN & 1<<3 )
    col = 1;
  row = col;
  if( col ){                            // if column found, check row
    KEY_PORT |= 0x7F;                   // all pullups on
    KEY_DDR = (KEY_DDR & 0x80) | 0x0F;  // pin 3..0 = output
    KEY_PORT &= 0xF0;                   // pin 3..0 = output low
    NOP();                              // wait until inputs sampled
    if( ~KEY_PIN & 1<<4 )
      row = 5;
    if( ~KEY_PIN & 1<<5 )
      row = 9;
    if( ~KEY_PIN & 1<<6 )
      row = 13;
  }                                     // 0 = no key
  return row - col;                     // 1..12 = key
}