Atmega644 ISR(WDT_vect)

[D a v i d K. (oekel)]

Hi,

ich würde gerne mit dem im Titel genannten Chip den WDT über den ISR 
verwenden.
(Zum testen ausschließlich den Vektor, später im Betrieb dann wie im 
Datenblatt beschrieben erst Vector zum Wegspeichern, dann Reset)

Nun verwende ich:

#include <util/delay.h>
#include <avr/wdt.h>

ISR(WDT_vect) {
        Beeper_ON();


/* 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 */
  WDTCSR |= (1<<WDIE);
}

void watchdogSetup() {

    // Reset the watchdog reset flag
    wdt_reset();
    wdt_enable(WDTO_4S);
    // Enable interrupts instead of reset
    WDTCSR |= (1 << WDIE);
}

void loop() {
    //Beeper_ON();
    LED_SW2_ON();
    for (int i = 0; i < 8; ++i) {
        _delay_ms(100);
    }
    //Beeper_OFF();
    LED_SW2_OFF();
    for (int i = 0; i < 8; ++i) {
        _delay_ms(100);
    }
}

int main() {
    watchdogSetup();
    ioinit();
    while (1) {
        loop();
    }
}

Als Minibeispiel.

Ich erkenne, dass die LED in gleichbleibenden Takt wie gewollt blinkt 
und dann pötzlich eine Unregelmäßigkeit aufweißt, was ich als den RESET 
deute.

Witziger weise, kommt kein Ton (wortwörtlich) aus dem ISR zur gleichen 
Zeit :(.
(Dabei sind LED_SW2_ON() und Beeper_ON() problemlos gegeneinander 
auszutauschen, womit ich 100% einen Fehler in der Ansteuerung 
ausschließen kann)

Vermutlich stimmt die Reihenfolge von
"WDTCSR |= (1 << WDIE);"

zu:    wdt_reset(); und  wdt_enable(WDTO_4S);
nicht. Habe aber bereits alle möglichen Konstellationen durchprobiert.

Daher wollte ich fragen, ob ihr mir auf die Sprünge helfe könnt den ISR 
sachgemäß zu aktivieren?

Danke und noch eine sonnigen Sonntag :)

[g457 (Gast)]

> Daher wollte ich fragen, ob ihr mir auf die Sprünge helfe könnt den ISR
> sachgemäß zu aktivieren?

Einfach nochmal die Doku zu WDIE lesen.

HTH

[D a v i d K. (oekel)]

g457 schrieb:
> Einfach nochmal die Doku zu WDIE lesen.

Ok, damit meinst du wohl den Part mit

WDTCSR |= (1 << WDCE) | (1 << WDE);

vor jedem Schreibzyklus.

Zusätzlich habe ich nun noch folgendende Section (die man wohl immer 
haben sollte) hinzugefügt.

//TODO: verschieben in *.h?
uint8_t mcusr_mirror __attribute__ ((section (".noinit")));

void get_mcusr(void) \
  __attribute__((naked)) \
  __attribute__((section(".init3")));//init3 korrekt auch für den Atmega 644?

void get_mcusr(void) {
    mcusr_mirror = MCUSR;
    MCUSR = 0;
    wdt_disable();//Darf diese nur verwendet werden, weil intern eh ASM steht?
}

bool wd_barked() {
    //return (mcusr_mirror & (1 << PC3));
    return (mcusr_mirror & (_BV(WDRF)));
}

Trotzdem verstehe ich die Logig noch nicht ganz.
Wenn ich folgendes auf den Chip bringe:

if (!wd_barked()) {
        WDTCSR |= (1 << WDCE) | (1 << WDE);
        MCUSR &= ~(1 << WDRF);
        unsigned char temp = ((1 << WDIE) | (1 << WDP3) | (1 << WDP0));//8sec
        //unsigned char temp = (1 << WDIE) | (1 << WDP3);//4sec
        WDTCSR |= (1 << WDCE) | (1 << WDE);
        WDTCSR = temp;
    }
    ioinit();
    sei();

Ertöhnt (direkt nach dem bestromen) das Piepen aus dem ISR genau 1x kurz 
und er startet neu.

Ich hätte, wie man hoffentlich aber erkennt, erst 8sec normalen Betrieb 
bzw, möchte das "if" hier nicht für den wdt verwenden, sondern 
anderweitig.
Wenn ich den wdt-init Teil hier aber ohne if lasse, habe ich ein 
endloses ISR-Restart drin.

Habt ihr nicht noch einen Rat?
Die Allgemeinen Infos und Datenblätter bringen mich hier nicht weiter 
und erst recht nicht diese Arduino Schnipsel, welche man überall findet.

LG

[HildeK (Gast)]

Ich bin nicht gut genug, um deine SW zu debuggen, aber ich habe dir ein 
ganz ähnlich funktionierendes Programm für den TinyX5 angehängt.
WD-Timer, WD-Interrupt, WD-Reset - alles enthalten.
Dient mir als Beispiel und sollte mit nur wenig Änderung auch beim 644 
funktionieren.

[D a v i d K. (oekel)]

HildeK schrieb:
> Ich bin nicht gut genug, um deine SW zu debuggen, aber ich habe dir ein
> ganz ähnlich funktionierendes Programm für den TinyX5 angehängt.

Beim Vergleichen erkenne ich keinen Makanten Unterschied.
Egal was ich mache, der WDTCSR scheint nicht lange genug zu warten.

Kurz nach dem Setup steht noch WDTCSR = 72 drin (debugged)
Und im ISR ist nur noch WDTCSR = 8.

Wo sind allso die verdammten Bits hin, oder muss ich die ALLE (WDIE ist 
klar) nach einem IR neu sezten?

Ich weiß zwar nicht, ob der WDT im Debugg weiter läuft, aber wenn ich 
die Breakpoints raus nehme, sollte ich doch an der blöden LED 2/4/8 Sek 
erkennen können.:(

[S. Landolt (Gast)]

Haben Sie sich so etwas vorgestellt?

[D a v i d K. (oekel)]

S. Landolt schrieb:
> Haben Sie sich so etwas vorgestellt?

Lieb gemeint, aber da entdecke ich einige Fehler und keine Neuerungen zu 
meinem Wissen/Fehler:

Sollte nicht im ISR sein...

ISR (WDT_vect) {
  WDTCSR = (1<<WDIE);
}

interrupt kann den kritischen Bereich unterbrechen:

  sei();
  WDTCSR = (1<<WDE) | (1<<WDCE);//aktivieren und sofort ...
  WDTCSR = (1<<WDE) | (1<<WDIE) | (1<<WDP3);        // interrupt&reset, 4 s

Ich glaube ich beschreibe lieber PORTB:

    PINB = 0b00000001;

[S. Landolt (Gast)]

Es ging mir darum, ein lauffähiges Minimalprogramm anzubieten. Dass es 
nicht weiterhilft, ist schade.

[Jörg W. (dl8dtl) (Moderator)]

Habe gerade mal ein eigenes Projekt angesehen, in dem ich den 
Watchdog-Interrupt als regelmäßigen Interrupt (ohne Reset) benutze.

Aus dem Kopf, ohne ins Datenblatt zu gucken:

WDTCSR beschreibt man am besten jeweils komplett, also kein "|=", 
sondern eine direkte Zuweisung.

Eine erfolgreiche Änderung des WDE-Bits benötigt unmittelbar zuvor ein 
aktives WDCE (watchdog change enable).

Vor allem Gefummel an WDTCSR zur Sicherheit immer erstmal den 
Watchdog-Counter auf Anschlag setzen.

static void
xxo_sleep(void)
{
  set_sleep_mode(SLEEP_MODE_PWR_DOWN);

  cli();
  wdt_reset();
  WDTCSR = _BV(WDCE) | _BV(WDE);
  WDTCSR = _BV(WDIF) | _BV(WDIE) | 6; /* 1 s interrupts */
  sei();
}

static void
xxo_unsleep(void)
{
  set_sleep_mode(SLEEP_MODE_IDLE);

  cli();
  wdt_reset();
  WDTCSR = _BV(WDCE) | _BV(WDE);
  WDTCSR = 0;           /* turn off */
  sei();
}

ISR(WDT_vect)
{
  if (++trigger_seconds == TRIGGER_LEVEL)
    {
      trigger_seconds = 0;
      timer_triggered = true;
      xxo_unsleep();
    }
  else
    {
      xxo_sleep();
    }
}

Bei mir wacht der Watchdog (in etwa) jede Sekunde auf, aber nur um 
nachzusehen, ob "was zu tun ist". Das ist immer dann der Fall, wenn die 
TRIGGER_LEVEL Anzahl an Sekunden erreicht ist. Dann wird der WDT erstmal 
abgeschaltet, damit die Firmware normal arbeiten kann. Wenn sie fertig 
ist, ruft sie wieder xxo_sleep() auf.

Ja, das sei() gibt, wenn die Funktionen aus der ISR heraus aufgerufen 
werden, die Interrupts vorzeitig frei. Stört hier nicht. Man könnte 
natürlich auch die übliche Logik drumrum bauen, SREG retten, cli(), und 
am Ende SREG wieder herstellen.

[S. Landolt (Gast)]

D a v i d K. schrieb:
> keine Neuerungen zu
> meinem Wissen/Fehler

Da möchte ich denn doch widersprechen.
Mein Programm macht, in seiner Schlichtheit, zumindest das, was es soll: 
eine LED an B0 blinkt gleichmäßig. Ihr Programm hingegen läuft offenbar 
nicht korrekt, es lässt sich in der eingangs vorgestellten Version nicht 
einmal übersetzen, da Teile nicht gezeigt wurden; so ließ sich auch der 
Verdacht nicht bestätigen, dass es am fehlenden sei lag.
  Eine Kritik jetzt anzubringen, ist etwas merkwürdig - vor zwei Tagen 
wäre man in gemeinsamer Arbeit&Diskussion sicher weitergekommen.

[D a v i d K. (oekel)]

S. Landolt schrieb:
> D a v i d K. schrieb:
>> keine Neuerungen zu
>> meinem Wissen/Fehler
>
> Da möchte ich denn doch widersprechen.
> Mein Programm macht, in seiner Schlichtheit, zumindest das, was es soll:

Bei mir leider nicht. Hab es 1:1 eingespielt.
Erst danach, sind mir die Dinge aufgefallen,  welche ich als Hinweise 
aus dem Datenblatt habe.

Meine Experimentierzeit beschränkt sich leider derzeit aufs Hotel unter 
der Woche, weswegen ich leider nicht so spontan antworte.

[D a v i d K. (oekel)]

Jörg W. schrieb:
>...
> Bei mir wacht der Watchdog (in etwa) jede Sekunde auf, aber nur um
> nachzusehen, ob "was zu tun ist". Das ist immer dann der Fall, wenn die
> TRIGGER_LEVEL Anzahl an Sekunden erreicht ist. Dann wird der WDT erstmal
> abgeschaltet, damit die Firmware normal arbeiten kann. Wenn sie fertig
> ist, ruft sie wieder xxo_sleep() auf.


Danke das werde ich so noch mal testen!

[S. Landolt (Gast)]

> Bei mir leider nicht.

Auf einem ATmega644 im Auslieferungszustand? Seltsam. Einer von uns 
beiden macht etwas falsch - allerdings ist C nicht meine Sprache.