Watchdog Interrupt

[Martin (Gast)]

Hi,

ich würde den Watchdog gerne nutzen um meinen µC (Mega88PA) aus dem 
Standby aufzuwecken. Das funktioniert leider nur teilweise. Der Watchdog 
soll nur einen Interrupt auslösen, keinen Reset. Auslösen soll er nach 
einer Sekunde.
Passieren tut jetzt folgendes:
Der Controller geht in den Standby und wacht schon wieder nach 16ms 
(Default-Settings) auf. Laut Datenblatt muss der Zugriff auf das 
Register innerhalb von 4 Taktzyklen erfolgen, nachdem es freigeschaltet 
wurde mit dem WDCE-Bit. Ich hab schon etwas gesucht, aber eine richtige 
Lösung hatte niemand. Ich hab leider nur sehr wenig Ahnung von 
Assembler, ich geh aber davon aus (wenn ich das .lss-File anschaue) das 
der Compiler zu viele "Schritte" draus macht.

Folgenden Code habe ich aktuell:

void GlobalSleepMode(void)
{
  cli();
  WDTCSR |= (1<<WDCE);    //WD Interrupt Enable, WD Change Enable
  WDTCSR |= 0x70;        //128k, ca. 0,97s
  sei();
  
  _delay_ms(5);            //Auf evtl. UART Übertragungen warten
  set_sleep_mode(SLEEP_MODE_STANDBY);
  sleep_mode();
}

void GlobalWakeUp(void)
{
  SleepStatus = 0;
  
  cli();
   WDTCSR |= (1<<WDCE); //0x04;  //WD Change Enable
   WDTCSR = 0x00;        //Turn WD off
  sei();
}

int main (void)
{
  DDRB |= (1<<DDB0);

  uart_init(UART_BAUD_SELECT(38400,F_CPU));
  sei();

  while (1)
  {
    if (SleepStatus == 1)
    {
      PORTB |= (1<<PORTB0);
      
      GlobalSleepMode();
      GlobalWakeUp();
      
      PORTB &= ~(1<<PORTB0);
    }
    else
    {
//[...] (ADC Auswertung)
      if (rx_status() == STATUS_COMPLETE)
      {
        //[...] (Übertragung Daten UART)
      
        SleepStatus = 1;
      }
    }
  }
}
ISR(WDT_vect)
{
  
}

Die entsprechenden Zeilen aus dem .lss:

void GlobalSleepMode(void)
{
  cli();
 3e8:  f8 94         cli
  WDTCSR |= (1<<WDCE);    //WD Interrupt Enable, WD Change Enable
 3ea:  e0 e6         ldi  r30, 0x60  ; 96
 3ec:  f0 e0         ldi  r31, 0x00  ; 0
 3ee:  80 81         ld  r24, Z
 3f0:  80 61         ori  r24, 0x10  ; 16
 3f2:  80 83         st  Z, r24
  WDTCSR |= 0x70;        //128k, ca. 0,97s
 3f4:  80 81         ld  r24, Z
 3f6:  80 67         ori  r24, 0x70  ; 112
 3f8:  80 83         st  Z, r24
  sei();
 3fa:  78 94         sei
[...]
void GlobalWakeUp(void)
{
  SleepStatus = 0;
 424:  10 92 21 01   sts  0x0121, r1
  rfm12_sleepmode(0);
 428:  80 e0         ldi  r24, 0x00  ; 0
 42a:  d3 df         rcall  .-90       ; 0x3d2 <rfm12_sleepmode>
  
  cli();
 42c:  f8 94         cli
   WDTCSR |= (1<<WDCE); //0x04;  //WD Change Enable
 42e:  e0 e6         ldi  r30, 0x60  ; 96
 430:  f0 e0         ldi  r31, 0x00  ; 0
 432:  80 81         ld  r24, Z
 434:  80 61         ori  r24, 0x10  ; 16
 436:  80 83         st  Z, r24
   WDTCSR = 0x00;        //Turn WD off
 438:  10 82         st  Z, r1
  sei();
 43a:  78 94         sei
 43c:  08 95         ret

0000043e <main>:
}

Wie bekomm ich das also jetzt so hin, das das funktioniert?
Ich habe schon versucht die Hex-Werte direkt zu schreiben, dann Wacht 
der µC aber gar nicht mehr auf.
Die wdt.h habe ich schon genutzt, aber keine Möglichkeit gefunden 
einzustellen, das nur die ISR aufgerufen wird. Weiß jemand, wie ich das 
lösen kann?
Vielen dank schon mal!

[Martin (Gast)]

Hm, hab ich was vergessen oder ist etwas unklar, oder weiß wirklich 
niemand Rat?

[Oliver (Gast)]

Rat weiß das Datenblatt. Das solltest du nochmal anschauen, incl. des 
dort gezeigten Beispielcodes.

Dazu dann die Optimierung beim kompilieren einschalten.

Oliver

[Martin (Gast)]

Hi,

Optimierung ist auf -Os (Optimize for size).
Code aus dem DB habe ich probiert, gerade extra nochmal. Nur das WDE 
habe ich zu einem WDIE gemacht, da ich ja einen Interrupt und keinen 
Reset haben will (Siehe DB Seite 54, Table 11-1). Geht auch nicht. Der 
µC geht nur für 16ms schlafen.

  cli();
  wdt_reset();
  WDTCSR |= (1<<WDCE)|(1<<WDIE);        //WD Interrupt Enable, WD Change Enable
  WDTCSR = (1<<WDIE)|(1<<WDP2)|(1<<WDP1);    //128k, ca. 0,97s      
  sei();

[Uwe (Gast)]

> WDTCSR |= 0x70;
Probier doch mal
WDTCSR |= 0x24;

[Uwe (Gast)]

Wenn die prescaler alle auf 0 stehen sinds 16ms

[Martin (Gast)]

Hi Uwe,

mit 0x24 setz ich WDP2 und WDP3 auf 1 - diese Einstellung gibt es laut 
Datenblatt nicht.

Ja, ich weiß das es 16ms sind wenn alles auf 0 steht. Aber ich will ja 
eine Sekunde erreichen und das funktioniert nicht.

[Uwe (Gast)]

Stimmt ist dann aber trotzdem nicht :
> WDTCSR |= 0x70;
sondern
WDTCSR |= 0x06;

mit
> WDTCSR |= 0x70;
setzt du nur WDIF auf 1 und die Prtescaler bleiben auf 0 also 16ms bis 
zu Watchdog Interupt.

[Martin (Gast)]

Ich hab nochmal etwas gegoogelt. In Anlehnung an den Code: 
http://electronics.stackexchange.com/questions/74840/use-avr-watchdog-like-normal-isr

Hab ichs dann so geändert (einfach nur die schreibweise)

   WDTCSR |= (1<<WDCE)|(1<<WDE);        //WD Interrupt Enable, WD Change Enable
   WDTCSR = (1<<WDIE)|(1<<WDP2)|(1<<WDP1);    //128k, ca. 0,97s

Und so geht es. Man muss also das WDE Bit mit setzen, bevor man 
Änderungen am Register vornehmen kann. Das Datenblatt ist da sehr wage 
zu interpretieren. (Meiner Meinung nach)

Danke an alle helfenden!

[Oliver (Gast)]

Martin schrieb:
> Das Datenblatt ist da sehr wage
> zu interpretieren. (Meiner Meinung nach)

Na ja, selbst wenn man den Text als vage mißversteht, steht das ganz 
genauso in dem Codebeispiel. Nocht konkreter geht das eigentlich gar 
nicht.

Oliver

[Martin (Gast)]

Jetzt im nachhinein ja - aber es wird halt als Reset Programmiert im 
Beispiel, und das WDE Bit ist definitiv das Bit um die Reset-Funktion zu 
aktivieren (siehe Tabelle 11-1). Daher ergab das für mich keinen Sinn, 
das mit rein zu schreiben.