Warum funktioniert mein Watchdog Timer-Interrupt nicht?

[Peter K. (peter_k575)]

Hallo zusammen,

ich versuche gerade, meinen uC (Attiny 84A) in einen deep Sleep Modus 
(Power Down) zu versetzen und ihn nur alle 8s aufzuwecken. Laut 
Datenblatt ist das ja möglich und das mit dem deep Sleep scheint auch zu 
funktionieren, jedenfalls fließen jetzt nur noch 0,3mA statt 3mA. Mehr 
als erwartet, aber schon mal ein Fortschritt.

Jedoch scheint meine ISR nicht zu funktionieren, jedenfalls wird die 
hier nur testweise implementierte Funktion nie ausgeführt. Weiß da 
jemand hier weiter?

Viele Grüße!

#include <stdint.h>
#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <avr/sleep.h>
#include <avr/wdt.h>
#include <stdbool.h>

volatile bool flag = 0;

ISR(WDT_vect)
{
  sleep_disable();
  flag = 1;
  sleep_mode();
  reti();
}

void WDtimerInit()
{
  cli();
  wdt_reset();
  WDTCSR |= (1 << WDIE) | (1 << WDCE) | (1 << WDP3) | (1 << WDP0);
  sei();
}

int main(void)
{
  DDRB = 0xff; // 0000 0111 = 0x07
  DDRA = 0xc7; // 1100 0111 = 0xc7
  
  WDtimerInit();
  
  set_sleep_mode(SLEEP_MODE_PWR_DOWN);
  sleep_mode();

  while(1)
  {  
    if ( flag == 1) 
    {  
      sleep_disable();
            PORTA = 0x01;
                  _delay_ms(1000);
                  PORTA = 0x00;

      WDtimerInit();
      flag = 0;
      sleep_mode();
    }
    
  }
}

[Peter D. (peda)]

Du mußt im WDT-Handler schon die Sequenz lt. Datenblatt ausführen, sonst 
macht der nächste WDT-Überlauf ein Reset.

Und sleep im Interrupt ist verboten, da Du dann nicht mehr aufwachen 
kannst.
Und reti auch, da Absturz (Epilog wird nicht ausgeführt).
sleep_disable ist eine sinnlose Funktion (bewirkt nichts).

[Peter K. (peter_k575)]

Danke für die schnelle Antwort!

Also die ISR sollte einfach nur so aussehen:

ISR(WDT_vect)
{ flag = 1; }

Ich dachte aber, dann würde der if-Vergleich in der main nie ausgeführt 
werden? Brauch ich denn nicht doch noch ein sleep_disable in der ISR?

Peter D. schrieb:
> Du mußt im WDT-Handler schon die Sequenz lt. Datenblatt ausführen, sonst
> macht der nächste WDT-Überlauf ein Reset.

Hab ich dich so richtig verstanden, dass ich meine WDtimerInit Funktion 
noch um _WDR() und MCUSR = 0x00 erweitern muss?

[Peter D. (peda)]

Siehe:
https://www.instructables.com/AVR-Watchdog-as-timed-interrupt/

ISR(WDT_vect)
{
/* ReEnable the watchdog interrupt, as this gets reset when entering this ISR and automatically enables the WDE signal that resets the MCU the next time the  timer overflows */
  WDTCR |= (1<<WDIE);
}

Ein Flag brauchst Du nicht, die CPU wartet nach jedem Sleep, bis der 
Interrupt ausgeführt wurde.

[Georg M. (g_m)]

Hier ist ein funktionierendes Beispiel:

// ATtiny84A
// LED: PA2 (Pin 11): ~1s ON, ~8s OFF 

#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/sleep.h>

ISR(WATCHDOG_vect)
{

}

int main(void)
{
  DDRA |= (1<<DDA2);                             // set PA2 as output
  set_sleep_mode(SLEEP_MODE_PWR_DOWN);           // set sleep mode
  sei();                                         // enable interrupts

  while(1)
  {
    PORTA |= (1<<PORTA2);                        // LED on
    WDTCSR = (1<<WDCE) | (1<<WDE);               // enable WDT time-out change
    WDTCSR = (1<<WDIE) | (1<<WDP2) | (1<<WDP1);  // enable WDT interrupts, set time-out 1s
    sleep_mode();                                // sleep

    PORTA &= ~(1<<PORTA2);                       // LED off
    WDTCSR = (1<<WDCE) | (1<<WDE);               // enable WDT time-out change
    WDTCSR = (1<<WDIE) | (1<<WDP3) | (1<<WDP0);  // enable WDT interrupts, set time-out 8s
    sleep_mode();                                // sleep
  }
}

Stromverbrauch ohne LED ca. 5µA @ 3.8V.