Taster reagieren nach einiger Zeit nicht mehr

[Bastian F. (bastian_f)]

Hallo!

Ich habe ein einfaches Programm geschrieben (mit Hilfe der debounce 
Funktion die in den Tutorials beschrieben wird), um ein paar LEDs nach 
einem bestimmten Muster zum leuchten zu bringen.
Spaßeshalber habe ich versucht dieses mit vier verschiedenen Tastern 
auszulösen, was soweit auch funktioniert.
ABER, nach ein paar Aufrufen (Drücken der Taster) passiert nichts mehr.
Erst nach einer Weile (ca. 30sec) reagiert der Controller wieder auf die 
Tasten.
Kann mir das eventuell jemand erklären bzw was ich im Code falsch 
gemacht habe?

Danke!

Code (im Anhang ist die led_defines.h):

#define F_CPU 1000000
#include <avr/io.h>
#include <util/delay.h> 
#include "led_defines.h"

int main() {

int a;
int b;
int delay=150; 


DDRC |= (1 << PC5) | (1 << PC4) | (1 << PC3) | (1 << PC2) | (1 << PC1);
DDRB |= (1 << PB2) | (1 << PB6) | (1 << PB1);

while (1) {


if (Taster4 | Taster2 | Taster3 | Taster1) {
for (a=1, b=8; a<=4; a++, b--) {
  
  _delay_ms(delay);
  light(a);
  light(b);
  }

for (a=1, b=8; a<=4; a++, b--) {
  
  _delay_ms(delay);
  nolight(a);
  nolight(b);
  } 

}
}
}

[Karl H. (kbuchegg)]

Ich seh nicht, dass du die Pullupwiderstände an den Eingabe (Tasten) 
Ports einschaltest.


_delay_ms immer nur mit einer Zahl aufrufen (keine Variable), sonst 
stimmt die Zeit nicht.

[Bastian F. (bastian_f)]

Stimmt, das habe ich vergessen...
Funktioniert jetzt wie es soll - Vielen Dank!

Zu _delay_ms: Sorry, für die blöde Frage, aber würde es denn einen 
Unterschied machen, wenn ich delay nicht als Variable, sondern als 
Konstante definiere?

[mr. mo (Gast)]

Sobald das Programm bei delay_ms ankommt, muss er natürlich erstmal 
nachgucken was in der Variable "delay" steht, dadurch vergeht natürlich 
auch wieder etwas zeit und wird unpräzise.

[Karl H. (kbuchegg)]

Bastian F. schrieb:

> Zu _delay_ms: Sorry, für die blöde Frage, aber würde es denn einen
> Unterschied machen, wenn ich delay nicht als Variable, sondern als
> Konstante definiere?

_delay_ms ist darauf angewiesen, dass der Optimizer den kompletten delay 
sich vornehmen und optimieren kann. Da müssen dann 
Gleitkommaberechnungen vom Compiler durchgeführt und wieder wegoptimiert 
werden. Das geht aber nur, wenn der Compiler den Zahlenwert sieht.
Daher: keine Variablen.

[Bastian F. (bastian_f)]

Das macht natürlich Sinn.
Ob das bei meinem kleinen Lichtspiel jedoch einen Unterschied macht, 
glaube ich zwar nicht, aber ich werde es mir jedenfalls für die Zukunft 
merken.