Troubleshoot einfache Entprell-Routine

[Thomas (Gast)]

Hallo,

bin noch relativ neu im Bereich uC Programmieren und C-Programmieren. 
Ich habe nun mit AVR Studio und einem Atmega 328P versucht folgende 
einfache Funktion zu realisieren:

Wird der Taster kurz gedrückt, wird die LED eingeschaltet, wird der 
Taster lang gedrückt geht diese wieder aus.
Leider tut sich schlicht und ergreifend, gar nichts.

Vielleicht kann mir hier jemand meine Programm/Programmierfehler 
aufzeigen und Tipps für das Weitermachen geben.

Vielen Dank im Vorraus.

mfg Thomas

#include <asf.h>
#include <avr/io.h>
#include <stdbool.h>

#define F_CPU 16000000

bool lastState=0;
bool newState=0;
bool ledOn=0;

bool kurz=0;
bool lang=0;

uint8_t toggle=0;

uint8_t shortPress=5; //50ms debounce Zeit
uint16_t longPress=50; //500ms Wartezeit
uint16_t i=0;

ISR (TIMER0_COMPA_vect){    //ISR Routine - Wird ca alle 10ms aufgerufen
  newState=PIND3;        //Taster Zustand abrufen
  if(newState==lastState){  //Wenn sich der Zustand seit dem letzten Aufruf nicht geändert hat Zählvariable erhöhen
    i++;          
  }
  if(i==shortPress){      //Wenn Taster 5 Durchläufe, also 50ms gedrückt wird, wird die Toggle Variable gesetzt
  toggle=1;  
  }
  if(i==longPress){      //Wenn Taster 50 Durchläufe, also 0,5s gedrückt wird, wird die Toggle Variable zurückgesetzt
  toggle=0;
  }
  lastState=newState;
} 
  
int main (void)
{
  DDRB |= (1<<PB5);    //LED-Pin als Output
  DDRD &= ~(1<<PD3);    //Taster-Pin als Input
  
  TCCR0A |= (1<<WGM01);        //Timer in CTC Mode
  TCCR0B |= (1<<CS02) | (1<<CS00);  // F_CPU/1024 
  OCR0A = 156;            //Compare Wert einstellen
  TIMSK0 |= (1<<OCIE0A);        //Compare Match Interrupt aktivieren
  sei();                //Interrupts aktivieren

  for(;;){
    if(toggle==1){
      PORTB |= (1<<PINB3);    //Toggle Variable auf den LED-Pin übertragen
    }
    else{PORTB &= ~(1<<PINB3);
      }
  }
  return 0;
}

[Jim M. (turboj)]

Die "toggle" Variable ändert sich im Interrupt, daher deklariert man die 
besser so:

volatile uint8_t toggle=0;

Ansonsten optimiert der Compiler den Zugriff darauf gnadenlos weg.

[Flohzirkus (Gast)]

Auch die anderen in der ISR verwendeten Variablen sollten volatile 
deklariert sein.

Außerdem: Deine Zählervariable zählt nicht nur die Zeit, in der der 
Taster gedrückt ist, sondern auch die Zeit, in der er nicht gedrückt 
ist.
Sie wird nie zurückgesetzt - zählt also munter weiter bis zum Überlauf.

So sollte die LED dann eigentlich einmal zu Beginn und dann alle elf 
Minuten wieder für 450 ms leuchten.
Durch einen Tastendruck kannst du das um ein paar Millisekunden 
verzögern.

Schönen Sonntag noch :-)

[Thomas (Gast)]

Hallo, vielen Dank schonmal,

habe gestern dann auch noch gemerkt, dass erstens die Zählvariable nie 
gedrückt wird und zweitens die Variable auch erhöht wird wenn der Taster 
nicht gedrückt ist und sich der Zustand lediglich nicht ändert.
Habe auf dieser Seite aber nun eine ganz gute Anleitung gefunden.

mfg Thomas