ATMega328p & GCC: Watchdog-Problem

[Karsten (Gast)]

In intWDT() wird der Watchdog initialisiert. Er soll alle 8 Sekunden 
(Flags:WDP0, WDP3 einen Interrupt auslösen.

Ausgelöst wird der Interrupt jedoch alle 16 ms.

Wo liegt der Fehler?

#include <avr/sleep.h>
#include <avr/wdt.h>
#include <util/delay.h>
#include <avr/io.h>
#include <util/atomic.h>

ISR  (WDT_vect, ISR_NAKED)
{
  PORTD |= (1 << PORTD5);
  _delay_ms(10);
  PORTD &= ~(1 << PORTD5);

  reti();
}

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

int main(void)
{
  DDRD = (1 << PORTD5);

  intWDT();

  while (1)
  {
  }
  
  return (0);
}

[MWS (Gast)]

Karsten schrieb:
> Ausgelöst wird der Interrupt jedoch alle 16 ms.

Wahrscheinlich wird der WDChange nicht vorbereitet.

Probier' mal:
>   WDTCSR = (1 << WDCE) | (1 << WDE);
>   WDTCSR = (1 << WDIE) | (1 << WDP3) | (1 << WDP0);

[Karsten (Gast)]

Danke, das war es :)

[Karl M. (Gast)]

Nun,

dies macht man auch nicht in einer ISR!

ISR  (WDT_vect, ISR_NAKED)
{
  PORTD |= (1 << PORTD5);
  _delay_ms(10);
  PORTD &= ~(1 << PORTD5);

  reti();
}

Setze ein (Event-)Flag und arbeite es in der Mainloop ab.

[MWS (Gast)]

Karl M. schrieb:
> dies macht man auch nicht in einer ISR!

Zum Testen ist das wurscht, ich würde mir eher Gedanken machen, ob das 
reti() dem Compiler bei der Restaurierung der Register in die Suppe 
spuckt und für Stacküberlauf sorgt, weil die Register nicht mehr vom 
Stack gePOPpt werden.

Das sollte raus, vom Compiler wird in einer ISR das RETI nach dem 
{}-Block automatisch erzeugt.

[MWS (Gast)]

Ok, bei naked ist's egal.

[Stefan (Gast)]

Bei ISR_NAKED werden keinerlei Register gesichert. Wenn irgendwann in 
Deinem main() mal richtiger Cde steht, wirst Du so merkwürdige Fehler 
bekommen.

bdw.:
Warum ein paar ns durch ISR_NAKED sparen und danach _delay_ms(10) 
aufrufen???

Gruß, Stefan