Tastenmatrix spinnt herum

[Grevak D. (xorda)]

Hallo,

ich wollte eine Tastenmatrix ansteuern. Allerdings spinnt diese 
Tastenmatrix. Ich habe ein Programm geschrieben, welches "Hello World" 
ausgibt, wenn man auf einer diesen Tasten drückt.

Das Problem ist jedoch, dass es nicht so ganz klappt. Die Tastenmatrix 
scheint überempfindlich zu sein. Der Pin Change Interrupt wird schon 
ausgelöst nur wenn ich diese Tastenmatrix bewege oder sofort zu 
Programmstart oder manchmal garnicht.

Ich möchte interne Pullup Widerstände benutzen. Mit externen Pulldown 
habe ich es ausprobiert, damit klappt es.

Hier ist der Code. Ein Bild mit dem Aufbau der Schaltung ist angehängt

#define F_CPU 16000000UL

#include <stdbool.h>
#include <avr/pgmspace.h>
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include "i2clcd.h"
#include "i2cmaster.h"

ISR(PCINT2_vect) {
    lcd_printlc(1, 1, "Hello World");
}

int main() {
    DDRC = 0b00000000;
    PORTC = 0b00110000;
      
    i2c_init();
    lcd_init();
      
    lcd_command(LCD_DISPLAYON | LCD_CURSORON | LCD_BLINKINGOFF);
    lcd_light(true);
    lcd_command(LCD_CLEAR);
  
    _delay_ms(500);
  
    DDRD = 0b00000000;
    PORTD = 0b00001111;
  
    PCMSK2 = 1 << PCINT20 | 1 << PCINT21 | 1 << PCINT22 | 1 << PCINT23;
    PCICR = 1 << PCIE2;
    sei();
  
    while (1) {
    
    }
}

Was kann ich tun?

[Christian H. (netzwanze)]

Grevak D. schrieb:
> Was kann ich tun?

Verwende keinen Interrupt zur Tastenabfrage (außer dem Timer).

[Grevak D. (xorda)]

Das Problem ist, dass unser Lehrer möchte, dass wir Interrupts nutzen. 
Und wir kennen gerade nur den Pin Change Interrupt. Außerdem hat es ja 
mit externen Pulldown Widerständen funktioniert

[Lothar (Gast)]

Du kannst im Interrupt nochmal prüfen ob und welche Taste gedrückt 
wurde.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Grevak D. schrieb:
> Das Problem ist, dass unser Lehrer möchte, dass wir Interrupts nutzen.
Dann nimm einen Timerinterrupt.

[Grevak D. (xorda)]

Er soll aber nicht pollen, das ist ineffizient

[Peter D. (peda)]

Grevak D. schrieb:
> Er soll aber nicht pollen, das ist ineffizient

Seit wann ist eine CPU-Last <0,1% ineffizient?

Alle einwandfrei funktionierenden Tastenroutinen pollen per Timer.
Alle anderen bringen den Anwender durch Fehlfunktion (Prellen usw.) 
regelmäßig zur Weißglut.

[Frank M. (ukw) (Moderator)]

Lothar M. schrieb:
> Dann nimm einen Timerinterrupt.

Grevak D. schrieb:
> Und wir kennen gerade nur den Pin Change Interrupt.

Peter D. schrieb:
> Seit wann ist eine CPU-Last <0,1% ineffizient?

Blöd, wenn sein Lehrer offensichtlich etwas anderes behauptet.

[Frank M. (ukw) (Moderator)]

Grevak D. schrieb:
> Ich möchte interne Pullup Widerstände benutzen.
>
>     PCMSK2 = 1 << PCINT20 | 1 << PCINT21 | 1 << PCINT22 | 1 << PCINT23;

Da Du nicht angegeben hast, welchen AVR Du verwendest, tippe ich einfach 
mal auf ATMega88/168/328.

Hier sind:

PCINT20 = PD4
PCINT21 = PD5
PCINT22 = PD6
PCINT23 = PD7

> DDRD = 0b00000000;
> PORTD = 0b00001111;

In der zweiten Zeile schaltest Du die internen Pullups für PD0...PD3 
ein, aber nicht für PD4...PD7.

Kein Wunder, dass Deine Tastatur "überempfindlich" ist. Dein Inputs 
floaten einfach so rum und reagieren bereits bei schiefem Angucken.

Schreibe also:

PORTD = 0b11110000;

um die richtigen Pullups einzuschalten.

P.S.
Taster per PCINT auszulesen ist - wie oben schon gesagt - wirklich die 
schlechteste Idee überhaupt. Prellende Taster erzeugen nicht nur mehrere 
Interrupts (was auch die CPU belastet), sondern sind auch noch schwierig 
zu erkennen und zu filtern. Kannst Du ja mal bei Gelegenheit in einer 
der nächsten Lehrstunden zur Diskussion stellen. Siehe auch 
Entprellung.

[Grevak D. (xorda)]

Wir haben entprellen schon durch genommen. Dass ich die falschen Pullups 
aktiviert habe ist mir nicht aufgefallen xD

Ich werde es mal ausprobieren

Mit Pollen war allerdings eigentlich gemeint, dass wir nicht in einer 
Schleife die ganze Zeit über PINx abfragen

Wir hatten halt Timerinterrupts noch nicht im Unterricht