Kam Reset durch Watchdog

[Watchdog Reset (Gast)]

Heyho.

Ich hab zwar hier schon viel gesucht, aber leider nichts passendes 
gefunden. Ich hab einen watchdog in auf meinem Tiny85 am laufen. Wenn es 
aber von dem einen reset gab, will ich das herausfinden:

void wdt_init(void)
{
  
  uint8_t buffer;
  buffer = MCUSR & (1 << WDRF);
  
  if (buffer == 1) {
    IsWatchdogReset = true;
  } else {
    IsWatchdogReset = false;
  };
  
  MCUSR = 0;
  wdt_disable();

  return;
}

Nur ist bei mir IsWatchdogReset immer false. Was mache ich falsch?

Danke für die Hilfe :-)

[Cyblord -. (cyblord)]

> if (buffer == 1)

ersetze durch

if (buffer)

Deine Programmierung ist echt umständlich.

IsWatchdogReset = (MCUSR & (1 << WDRF));

und fertig.

[Route_66 (Gast)]

Hallo!
WDRF ist Bit 3 in MCUSR.
Nach dem binären AND hast Du in buffer 0 oder 8 und nie 1.

[Panorama (Gast)]

Eventuell ein simpler Fehler: WDRF ist Bit 3 im Register.
Wenn das Bit gesetzt war steht daher dezimal 8 im Buffer, du prüfst 
allerdings auf 1. Ein simples "if(buffer)" wäre hier richtig, das prüft 
ob ein beliebiges Bit gesetzt ist.

Probier mal so:

uint8_t buffer = MCUCSR;

if(buffer & (1 << WDRF))
  IsWatchdogReset = 1;

[Panorama (Gast)]

Ooh schon wieder zu langsam :(

Eine kleine Anmerkung habe ich noch. Leg in so einen allgemeinen Puffer 
immer das vollständige Register. Wenn du irgendwann mal auf Power-On 
Reset prüfen willst wunderst du dich sonst wieso das Bit nicht kommt, 
hattest doch sogar gepuffert. Bei einer größeren Funktion geht dann die 
Sucherei und Umbau los.

[Cyblord -. (cyblord)]

Panorama schrieb:
> Ooh schon wieder zu langsam :(
>
> Eine kleine Anmerkung habe ich noch. Leg in so einen allgemeinen Puffer
> immer das vollständige Register. Wenn du irgendwann mal auf Power-On
> Reset prüfen willst wunderst du dich sonst wieso das Bit nicht kommt,
> hattest doch sogar gepuffert. Bei einer größeren Funktion geht dann die
> Sucherei und Umbau los.

Der bessere Tipp wäre auf solch "allgemeine Puffer" komplett zu 
verzichten. Variabeln und Funktionen sprechend benennen. Namen wie 
"buffer","x","a" usw. einfach weg lassen.

[Watchdog Reset (Gast)]

Ach mist, da hatte ich einen denkfehler. Ich hatte das shift genau 
falsch herum gelesen - dann käme da eine 1 raus.

wenn ich jetzt aber nach einen watchdog reset prüfe:

int main () {
  // Gab es einen Watchdog-Reset?                            
  if (IsWatchdogReset) {
    
    // Blinken
    for (int i = 0; i < 10; i++) {
      
      PB1_on();
      _delay_ms(100);
      PB1_off();
      _delay_ms(500);
    };      
  };
};

Dann blinkt die led nie, auch wenn der watchdog einen reset gemacht 
hat??!!

Weis einer wieso? Vielen Dank :-)

[Frank M. (ukw) (Moderator)]

Watchdog Reset schrieb:
>

> int main () {
>   // Gab es einen Watchdog-Reset?
>   if (IsWatchdogReset) {
>

Wo setzt Du denn IsWatchdogReset? Ich sehe in main() lediglich die 
Abfrage der Variable. Deine Funktion wdt_init() wird auch nirgendwo 
aufgerufen.

Zeig Deinen kompletten Code. So ist das sinnfrei.

[Watchdog Reset (Gast)]

Frank M. schrieb:
> Deine Funktion wdt_init() wird auch nirgendwo
> aufgerufen.

void wdt_init(void) __attribute__((naked)) __attribute__((section(".init1")));

void wdt_init(void)
{
  
  IsWatchdogReset = (MCUSR & (1 << WDRF));
  
  MCUSR = 0;
  wdt_disable();

  return;
}

int main () {
  // Gab es einen Watchdog-Reset?                            
  if (IsWatchdogReset) {
    
    // Blinken
    for (int i = 0; i < 10; i++) {
      
      PB1_on();
      _delay_ms(100);
      PB1_off();
      _delay_ms(500);
    };      
  };

  // Watchdog einschalten
  wdt_enable(WDTO_8S);
  
  while(1)
  {
     // nix
  };
};

[Frank M. (ukw) (Moderator)]

Watchdog Reset schrieb:
> void wdt_init(void) __attribute__((naked))
> __attribute__((section(".init1")));

Aha. Wo ist die Definition von IsWatchdogReset? Wann werden globale 
Variablen mit 0 initialisiert? Vor oder nach der init1-Section?

[Michael S. (stroggi)]

Und an welcher Stelle wird wdt_init überhaupt aufgerufen?

[Frank M. (ukw) (Moderator)]

Michael Strosche schrieb:
> Und an welcher Stelle wird wdt_init überhaupt aufgerufen?

Das passiert durch die Platzerung der Funktion in die Section namens 
"init1". Aber ich nehme mal nicht an, dass der Compiler dadurch 
automatisch weiß, dass wdt_init() vor main() aufgerufen werden könnte 
und zudem auch noch eine globale Variable setzt - Stichwort: volatile.

Aber gerade die Definition der globalen Variablen hat der TO 
unterschlagen.... damit die Kristallkugeln weiterhin brav poliert 
werden...

[Cyblord -. (cyblord)]

Diese merkwürdige Verlagerung der Init Funktion in die init1 Sektion ist 
mir nicht so ganz klar. Nötig ist sie für das was wir hier sehen 
jedenfalls nicht. Das ganze Spiel geht auch direkt aus der main heraus.

Sollte man solche Handstände machen, wenn man grade noch mit einem 
"bitwise and" gekämpft hat? Ich glaube nicht Tim.

[Frank M. (ukw) (Moderator)]

Cyblord ---- schrieb:
> Diese merkwürdige Verlagerung der Init Funktion in die init1 Sektion ist
> mir nicht so ganz klar.

Es gab hier in der Vergangenheit schon diverse Threads, wo wdt_disable() 
als erster Aufruf in main() zu spät kam und man dadurch den µC in eine 
Art Dauer-Watchdog-Reset versetzt hat.

Die Lösung, die dann immer angeboten wurde, war, den Aufruf von 
wdt_reset() in die init1-Section zu verlagern.

Ich habe aber gerade mal die Funktionen des TOs kompiliert und gesehen, 
dass standardmäßig erstmal wdt_init() komplett wegoptimiert wird. Es 
landet schon gar nicht mehr im lss-File.

[Jay (Gast)]

Hallo,

ich glaube das Verlagern der Funktion hat etwas damit zu tun, dass der 
Watchdog bei einigen AVRs nach einem Reset weiterhin aktiv ist. Dadurch 
kann es passieren, dass der Watchdog in der main() evt. nicht 
rechtzeitig deaktiviert werden kann. Aber ich bin mir bei init1 auch 
nicht sicher.

CU,
Jay

[Jay (Gast)]

... kurz mal Google befragt: 
http://www.nongnu.org/avr-libc/user-manual/group__avr__watchdog.html

Dort gibt es auch Beispielcode. In der main() dann stattdessen 
mcusr_mirror benutzen:
IsWatchdogReset = (mcusr_mirror & (1 << WDRF));
So sollte es funktionieren...

[Watchdog Reset (Gast)]

Frank M. schrieb:
> Watchdog Reset schrieb:
>> void wdt_init(void) __attribute__((naked))
>> __attribute__((section(".init1")));
>
> Aha. Wo ist die Definition von IsWatchdogReset? Wann werden globale
> Variablen mit 0 initialisiert? Vor oder nach der init1-Section?

Ach entschuldigung, das vergaß ich tatsächlich:

#define F_CPU 8000000L

#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <avr/wdt.h>
#define F_CPU 8000000L

void wdt_init(void) __attribute__((naked)) __attribute__((section(".init1")));

void wdt_init(void)
{
  
  IsWatchdogReset = (MCUSR & (1 << WDRF));
  
  MCUSR = 0;
  wdt_disable();

  return;
}

int main () {
  // Gab es einen Watchdog-Reset?                            
  if (IsWatchdogReset) {
    
    // Blinken
    for (int i = 0; i < 10; i++) {
      
      PB1_on();
      _delay_ms(100);
      PB1_off();
      _delay_ms(500);
    };      
  };

  // Watchdog einschalten
  wdt_enable(WDTO_8S);
  
  while(1)
  {
     // nix
  };
};

Jay schrieb:
> ich glaube das Verlagern der Funktion hat etwas damit zu tun, dass der
> Watchdog bei einigen AVRs nach einem Reset weiterhin aktiv ist

Vollkommen richtig. Beim Tiny85 resettet sich sogar der vorteiler sodass 
er in den kürzesten interrupt von 16ms fällt. Dann kommt er nichteinmal 
bis zum wdt_disable(); selbst wenn es an derster stelle steht.

[Watchdog Reset (Gast)]

Ach mist jetzt hat er was beim kopieren verloren:

volatile bool IsWatchdogReset;

void wdt_init(void) __attribute__((naked)) __attribute__((section(".init1")));
...
...

[Michael (Gast)]

Watchdog Reset schrieb:
> Ach mist jetzt hat er was beim kopieren verloren:

Ohne das auszuprobieren: Funktioniert der Code so wie beschrieben, wenn 
du es schaffst, beim Einfachen Copy & Paste eines gesamten 
Codeschnippsels eine einzelne Zeile dazwischen zu verlieren? Jede 
Zwischenablage, die ich kenne, kann das nicht...

[Watchdog Reset (Gast)]

Michael schrieb:
> Jede Zwischenablage, die ich kenne, kann das nicht...

Ich verstehe irgendwie nicht wie das bei der lösungs des problems hilft.

[Michael (Gast)]

Das hilft insofern, dass ich so nicht ganz glauben kann, dass der Code 
ein vollständiges Beispiel darstellt bzw. scheinbar ja nicht so 
ausprobiert wurde (sonst wäre die Variablendeklaration ja im ersten 
Copy&Paste-Vorgang mitgekommen). Deshalb auch die Frage, ob der Code so 
wirklich funktioniert? (oder halt nicht, aber mit beschriebenem Symptom)

[Fritz G. (fritzg)]

Das funktioniert bei mir (printBootReason in der main aufrufen):

uint8_t mcusr_mirror __attribute__ ((section (".noinit")));    // Für Reset-Grund

void get_mcusr(void) __attribute__((naked)) __attribute__((section(".init3")));
void get_mcusr(void)
{
    mcusr_mirror = MCUSR;
    MCUSR = 0;
    wdt_disable();
}
void wdt_init(void) __attribute__((naked)) __attribute__((section(".init3")));
void wdt_init(void)
{
    MCUSR = 0;
    wdt_disable();
    return;
}


/**
 *  @brief Gibt den Grund des Resets aus
 *
 *  @return  none
 */

void printBootReason(void) {
    printf_P(PSTR("Reset: "));
    
    if (mcusr_mirror&(1<<WDRF)) {
        printf_P(PSTR("Watchdog "));
    }
    if (mcusr_mirror&(1<<BORF)) {
        printf_P(PSTR("Brownout "));
    }
    if (mcusr_mirror&(1<<EXTRF)) {
        printf_P(PSTR("External "));
    }
    if (mcusr_mirror&(1<<PORF)) {
        printf_P(PSTR("Power-on "));
    }
    if (mcusr_mirror==0)
        printf_P(PSTR("unknown "));
    printf_P(PSTR("\n"));
}

[Watchdog Reset (Gast)]

Vielen Dank Fritz, Dein Ansatz funktioniert.

Dem verständnis nach muss man also die globale Variable dann in den 
Noinit-Block schieben, damit sie nicht beim Startvorgang des µC 
überschrieben wird, oder?

Und woher weiß der Controller, dass get_mcusr() vor wdt_init() 
aufgerufen werden muss? Eine Hirarchie kann ich nicht erkennen. Oder 
liegt das nur daran, da das mcusr vor dem init steht und das ganze 
sequenziell abgearbeitet wird?

[Fritz G. (fritzg)]

Watchdog Reset schrieb:
> Eine Hirarchie kann ich nicht erkennen. Oder
> liegt das nur daran, da das mcusr vor dem init steht und das ganze
> sequenziell abgearbeitet wird?

Nehme ich mal an. Ich glaube den Code habe ich hier im Forum gefunden.

[Watchdog Reset (Gast)]

Ok. Jedenfalls tut es genau das, was es soll.

Vielen Dank :)