Watchdog und sleep mode

[Christoph -. (con-f-use)]

Hallo,

ich habe ein kleines Projekt auf einem AtTiny45. Ich möchte ein paar 
Daten auslesen, den Tiny in den power down schicken und nach vier 
Sekunden über den watchdog wieder aufwecken damit das Spielchen von 
vorne losgehen kann. Ich bin mir allerdings nicht sicher, ob ich alles 
richtig gemacht habe.

Ich wäre dankbar, wenn sich jemand meinen Code anschauen könnten und 
evtl. Vorschläge zur Verbesserung macht. Vielen Dank im voraus.

#include <avr/power.h>
#include <avr/wdt.h>
#include <avr/sleep.h>
#include <avr/interrupt.h>

#define MASK_SENSOR  (1<<PB4)
#define MASK_LED     (1<<PB3)
#define TOTAL_INPUTS  3

void wdt_init() {
    cli();
    wdt_reset();
    WDTCR = (1<<WDE) | (1<<WDCE);   // Enable watchdog
    WDTCR = (1<<WDIE) | WDTO_4S;    // Watchdog interrupt instead of reset with 4s timeout
    sei();
}

void sleep_now() {
    power_all_disable();
    set_sleep_mode(SLEEP_MODE_PWR_DOWN);
    sleep_enable();
    sleep_mode();
    sleep_disable();
    wdt_init();
}

int main(void) {
    int i;

    // SETUP
    DDRB |= MASK_BUZZER | MASK_LED;                     // Set buzzer and led pin to output
    for(i=0; i<TOTAL_INPUTS; ++i) calibrate_sensors(i); // Set sensor pins and calibrate them (all digital inputs with enabled pull-ups)

    wdt_init();

    // MAIN LOOP
    while( 1 ) {
        get_sensor_data(); // full of wdt_resets()
        sleep_now();
    }

    return 0;
}

[Christoph -. (con-f-use)]

Okay ein Update: Mit diesem Code scheint der Chip einen reset zu machen, 
anstatt einfach nur aus dem Schlaf zu erwachen. Weiß aber nicht woran 
das liegt, weil ja WDIE eingeschaltet ist. Wäre für jede Hilfe dankbar.

[Hanno (Gast)]

Zum Aufwachen aus dem Schlaf kann man je nach Modus jeden Interrupt 
benutzen, z.B. auch einen normalen Timer-Overflow-Int. - Scheint mir der 
saubere Weg zu sein, anstatt den Watchdog für sowas zu missbrauchen.

Grüße
 Hanno

[Thomas E. (thomase)]

Hanno schrieb:
> Zum Aufwachen aus dem Schlaf kann man je nach Modus jeden Interrupt
> benutzen, z.B. auch einen normalen Timer-Overflow-Int.
Nein. Kann man nicht.

Hanno schrieb:
> anstatt den Watchdog für sowas zu missbrauchen.
Blödsinn.

Christoph -. schrieb:
> Mit diesem Code scheint der Chip einen reset zu machen,
Wo ist deine "ISR(WDT_vect)"? Die brauchst du natürlich, sonst springt 
er beim Interrupt auf den Reset-Vektor.

mfg.

[Watchdog (Gast)]

Nee, ohne ISR macht der normalerweise einfach gar keinen ISR, weckt aber 
trotzdem. Mag je nach Compiler anders sein.
Und der WDT ist toll für sowas, da geb' ich Thomas recht.
Ist bei den Fuses der Watchdog aktiviert? Wenn ja: Abschalten.

[Thomas E. (thomase)]

Watchdog schrieb:
> Nee, ohne ISR macht der normalerweise einfach gar keinen ISR, weckt aber
> trotzdem. Mag je nach Compiler anders sein.
So einfach ist das nicht.
Wird ein Interrupt ausgelöst, egal welcher, wird der Program Counter des 
Controllers auf den Interrupt-Vektor gesetzt. Dann wird der Befehl 
ausgeführt, der da steht. Da kann der Compiler gar nichts gegen machen. 
Das ist reine Hardware.
Üblicherweise steht dort ein Jump auf die ISR.
Ist keine ISR angelegt, wird ein Bad Interrupt ausgeführt, der wiederum 
einen Jump auf 0 enthält. Sozusagen ein Warmstart. AVR-GCC.
Vielleicht machen das andere Compiler anders. Keine Ahnung. 
Assemblerporgrammierer würden in diesem Falle einfach ein reti auf den 
Interrupvektor schreiben.

mfg.

[Watchdog (Gast)]

Oops... Hast recht, gerade nochmal ausprobiert. Ich dachte ich hätte das 
schonmal gemacht, aber da hat mich meine Erinnerung getrogen. Sorry!


Christoph: Ergänze irgendwo

ISR(WDT_vect) { }  //WatchDogTimer interrupt

[Christoph -. (con-f-use)]

Vielen Dank euch allen! Werde es morgen mal probieren. Habe mir schon 
gedacht, dass ich etwas offensichtliches vergessen hab.

@Hanno: Aus den anderen sleep modi reicht je nach Chip meistens jeder 
interrupt. Bei power_dwn ist allerdings der watchdog gefragt, besonders, 
weil der seinen eigenen Timer mitbringt.

[Christoph -. (con-f-use)]

Der Vollständigkeit halber: Das Problem war tatsächlich die fehlender 
ISR(wdt_vec). Fehlt das, wird ein reset vorgenommen.

Danke an alle Beteiligten.