uc hängt nach watchdog isr

[seb (Gast)]

Guten Abend,

ums kurz zu machen: Ich habe ein batteriebetriebenes gerät welches sich 
automatisch ausschalten soll sobald längere Zeit kein Taster mehr 
gedrückt wurde. Dazu benutze ich den auf 1s programmierten watchdog. In 
der ISR des WD habe ich eine Funktion implementiert welche mir das gerät 
ausschaltet. Dies funktioniert einwandfrei.
Da das jedoch erst nach 20s passieren soll wird jedes mal wenn er 
aufwacht eine variable inkrementiert. Hat sie den Wert 20 (sekunden) 
erreicht soll sich das gerät ausschalten.
Problem: Nachdem das Ende der ISR erreicht wurde hängt sich der 
controller auf. Zumindest reagiert das Gerät auf keinerlei Tastendruck 
mehr (Das Display zeigt den jeweiligen letzten wert an)

Hat jemand evtl. sowas ähnliches programmiert und kennt sich aus ??
Danke schonmal !

// global.h

uint8_t wdto_ctr;
#define WDT_timeout 20;

// system.c

ISR(WDT_vect)
{  
  wdto_ctr++;
  if (wdto_ctr >=  WDT_timeout)  // Timeout nach 20s
  {
    wdt_disable(); // Damit der Ausschaltvorgang nicht unterbrochen wird
    PWR_off();   // Schaltet das Gerät aus
  }

}

ISR(PCINT0_vect)  // wird nach tastendruck ausgeführt  
{
wdt_reset();
wdto_ctr=0;
}

// main.c

main
{
wdt_enable(WDTO_1S);  // Enable WDT
WDTCSR |=(1<<WDIE);  // Enable Interrupt on WD Timeout
wdto_ctr=0;             // Init Timeout counter

while(1)
{
// [..] ändere xyz in abhängigkeit von der tasteneingabe und zeige wert an
}
}

[Gregor B. (gregor54321)]

In deiner PWR_off() steht sowas wie
    SleepMode(Power_Down);
Dann geht wecken nur noch mit über einen INT-Eingang.
Schau mal ins Datenblatt unter SleepModes. Gibts immer ne hübsche 
Tabelle am Ende des Kapitels.

[Knut B. (Firma: TravelRec.) (travelrec)]

Der PinChange Interrupt sollte funktionieren, wenn er ordnungsgemäß 
eingerichtet ist.

[Mehmet K. (mkmk)]

Im Watchdog Interrupt muss das WDIE bit wieder gesetzt werden, ansonsten 
wird danach dieser Interrupt nicht mehr angesprungen.
Also noch die Zeile

WDTCSR |= (1<<WDIE);

einfügen.

[Knut B. (Firma: TravelRec.) (travelrec)]

Trotzdem sollten die Tasten funktionieren.

[Michael (Gast)]

Was mir so spontan auffällt: Wo wird denn beim Watchdog-Interupt der 
Watchdog resetet/abgeschaltet wenns nicht in die If-Schleife rein geht?

[Edi (Gast)]

> uint8_t wdto_ctr;

Globale Variablen, die sowohl in der ISR als auch in der main() 
verwendet werden, sollten mit volatile deklariert werden:

volatile uint8_t wdto_ctr;

[Peter D. (peda)]

seb schrieb:
> Hat jemand evtl. sowas ähnliches programmiert und kennt sich aus ??

Ja.
Allerdings nicht über den Umweg Watchdoginterrupt, sondern direkt über 
den allgemeinen  Timerinterrupt, den wohl jede Applikation hat und der 
auch das Tastenentprellen macht.

Funktioniert super:

http://www.avrfreaks.net/index.php?module=Freaks%20Academy&func=viewItem&item_type=project&item_id=2265


Da das Main im Idle-Mode ist (10ms je Loop), zähle ich dort auch das 
Timeout.
Aber wenn man noch andere Interrupts benutzt, sollte man es direkt im 
Timerinterrupt zählen.


Peter

[seb (Gast)]

Hallo allerseits,

erstmal danke für die vielen Antworten.

An Mehmet Kendi: Vielen Dank für den Tipp mit dem enabeln des 
Interrupts, jetzt funktionierts einwandfrei.

An Edi: Warum sollten globale variablen mit volatile deklariert werden ?

[Edi (Gast)]

> An Edi: Warum sollten globale variablen mit volatile deklariert werden ?

Wenn eine Variable mit "volatile" deklariert worden ist, dann optimiert 
der Compiler die Variable nicht als Register, sondern verwendet eine 
"echte" RAM-Speicherstelle dafür.

Das bedeutet: Bei einer volatile-Variable rechnet der Compiler immer 
damit, dass der Inhalt der Variablen irgendwo anders (also z. B. in 
einer ISR) verändert werden kann.

[Edi (Gast)]

Kleiner Nachtrag: Das hab ich mir natürlich nicht aus den Fingern 
gesaugt, sondern von hier:
http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Datenaustausch_mit_Interrupt-Routinen