watchdog atmega328

[grundschüler (Gast)]

hallo

habe Probleme mit dem wd:

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

wdt_enable(WDTO_8S); 
//============== Hauptschleife =================
while(1){
//================================================
....

//++++++++++++ volleSecunde ++++++++++++++++++++++
if(timer_smh>100){//1sec
wdt_reset();
led1_tog;

sec=sec+timer_smh/100;
timer_smh=timer_smh%100;
if(sec>59){sec=sec-60;  min++;
if(min==60){min=0; hou++;
if(hou==24){hou=0;}}}
}//-1sec

...

Der WD wird alle 8sec ausgelöst, funktioniert also grundsätzlich.

Jetzt soll der WD aber nur dann auslösen, wenn sich das Hauptprogramm 
wegen eines Fehlers aufgehängt hat. Deswegen soll der WD-counter bei 
jedem Sekundendurchlauf im Hauptprogramm zurückgesetzt werden.

Mit >wdt_reset();< funktioniert das nicht. Was mache ich falsch?

[Ulrich F. (Gast)]

Nach:
>timer_smh=timer_smh%100;

Kann dieses niemals wahr werden:
> timer_smh>100

[grundschüler (Gast)]

deswegen kommt die Abfrage auf  timer_smh>100 vor und nicht nach.

Der timer

//===========================================================
void timer_proc_10ms(void){
timer_smh++;
timer_10ms++;
#if USE_MMC
if(Timer1)Timer1--;//100Hz f�r sd-karte
#endif
}
//===========================================================

funktioniert.



Ich habe jetzt
wdg_reset();
durch
wdt_enable(WDTO_8S);
ersetzt.

Damit geht das ganze. Der aufruf der vollen Funktion

#define wdt_enable(value)   \
__asm__ __volatile__ (  \
    "in __tmp_reg__,__SREG__" "\n\t"    \
    "cli" "\n\t"    \
    "wdr" "\n\t"    \
    "sts %0,%1" "\n\t"  \
    "out __SREG__,__tmp_reg__" "\n\t"   \
    "sts %0,%2" "\n\t" \
    : /* no outputs */  \
    : "M" (_SFR_MEM_ADDR(_WD_CONTROL_REG)), \
    "r" (_BV(_WD_CHANGE_BIT) | _BV(WDE)), \
    "r" ((uint8_t) ((value & 0x08 ? _WD_PS3_MASK : 0x00) | \
        _BV(WDE) | (value & 0x07)) ) \
    : "r0"  \
)

kann ja nicht die Lösung des Problems sein?

[Ulrich F. (Gast)]

In Scheibchen sie sich offenbart, die Realität....

> void timer_proc_10ms(void)
Wo kommt denn die plötzlich her?

Von wo wird sie aufgerufen?

[grundschüler (Gast)]

das ist langbewährte software, wenn du da noch einen Fehler findest, 
bist du wirklich gut - jedenfalls danke für das Interesse.

init_timer_smh() {
/*
cs02+01+00:
0 no clock
1 no presc
2 /8
3 /64
4 /256
5 /1024
*/

TCCR0A |= (1<<WGM01);
    // 1; Timer0 Vorteiler: 64 -> bei 16 MHz: 
    // 16.000.000 / 64 = 250000 Mal pro Sek gibts einen Takt an den Timer
    // 8.000.000 / 1024 = 7812 Mal pro Sek gibts einen Takt an den Timer
TCCR0B |= ((1<<CS02)|(1<<CS00));//1024
    
    // Timer0 soll bei 250 einen Output Compare Interrupt ausl�sen ->
    // 250.000 / 250 = 1000 mal pro Sek gibts einen Interrupt
    // 7812 /78 = 100 mal pro Sek gibts einen Interrupt
OCR0A = 77;//abgleich mit uhr

    //Compare Interrupt aktivieren
TIMSK0 |= (1<<OCIE0A);

    //Globale Interrupts aktivieren
sei();
}


ISR(TIMER0_COMPA_vect)
{
timer_proc_10ms();
}


int main(void)
{
MCUSR = 0;//wdg statusregister
wdt_disable();
zucMain();   
    
return(0);
}

[Ulrich F. (Gast)]

grundschüler schrieb:
> wenn du da noch einen Fehler findest,
> bist du wirklich gut - jedenfalls danke für das Interesse.

Meister!!
Ich habe wenig Interesse deine Fehler zu finden, wenn du dich so 
sperrst....
Auch wenig Spass an der Salami Taktik.

Die Aussage timer_proc_10ms(); wird in einer ISR aufgerufen hätte fast 
gereicht.....
Schön wäre es noch zu sehen, wie du timer_smh definiert hast.....
(aber das ist wohl erst die nächste Salamischeibe)

§1:
timer_smh muss volatile  deklariert werden.

§2:
Das Bearbeiten von timer_smh im Hauptprogramm muss ATOMIC erfolgen.

> das ist langbewährte software,
Und auch diese muss man richtig verwenden...

Tipp:
Der Fehler findet sich immer an der Stelle, wo man zuletzt sucht.

Tipp2:
Es macht keinen Sinn, den Boten mit Verachtung zu überschütten.

[grundschüler (Gast)]

Ulrich F. schrieb:
> Tipp2:
> Es macht keinen Sinn, den Boten mit Verachtung zu überschütten.

Das war keineswegs gewollt, ich bin für alle Hinweise dankbar, auch wenn 
sie nicht direkt zielführend sind. Ich gehe davon aus, dass du besser 
programmieren kannst als ich.

timer_smh ist volatile  deklariert, da es außerhalb der ir im 
Hauptprogramm bearbeitet wird

Der Timer funktioniert und ist nicht das Problem. Die Zeit wird am LCD 
fortlaufend angezeigt.Die LED blinkt.



Die Frage bleibt, warum die Atmel-wdt_reset-Funktion den wdg-Timer nicht 
ordnungsgemäß zurücksetzt und wie man das richtig handelt. WDG müsste 
doch eigentlich Standard in vielen Programmen sein. Ich habe aber trotz 
einigem Suchen keinen lauffähigen Beispielcode gefunden.

[Ulrich F. (Gast)]

Vielleicht ist es ja unter gegangen.....


Darum nochmal:
Ulrich F. schrieb:
> §2: Das Bearbeiten von timer_smh im Hauptprogramm muss ATOMIC erfolgen.

Denn das fehlt in deinem gepostetem Code.

grundschüler schrieb:
> ich gehe davon aus, dass du besser
> programmieren kannst als ich.
Da kannst du von ausgehen, wobei ich mir aber nicht sicher bin, ob das 
stimmt....
KA!

grundschüler schrieb:
> timer_smh ist volatile  deklariert,
Ja, da ist sie ja die Salamischeibe....
Zumindest die halbe...
Der Datentype ist mir dadurch auch noch nicht bekannt....
Wie gesagt, wenn die zu bearbeitende Struktur mehr als 1 Byte 
beinhaltet, oder mehrfach gelesen/geschrieben wird (und das wird sie bei 
dir), ist die Angelegenheit ATOMIC zu machen.


Solange das nicht ATOMIC ist, hast du da einen Zufallsgenerator.
Und das wollen wir doch nicht.....

[Ulrich F. (Gast)]

Nachtrag:

Bei mir funktioniert es in der Arduino IDE!
(nach dem ich Timer0 frei geschaufelt habe)

[grundschüler (Gast)]

Ulrich F. schrieb:
> Bei mir funktioniert es in der Arduino IDE!
> (nach dem ich Timer0 frei geschaufelt habe)

danke für deine hilfe. ich hab es jetzt nocheinmal ohne timer aufgebaut. 
Es funktioniert wie es soll. hätte ich nicht gedacht, dass der wdg von 
anderen timern beeinflusst wird.

[holger (Gast)]

>hätte ich nicht gedacht, dass der wdg von
>anderen timern beeinflusst wird.

Das wird er nicht. Du hast einfach nur Mist gebaut irgendwo
in deinem Programm.

[Ulrich F. (Gast)]

holger schrieb:
> Das wird er nicht. Du hast einfach nur Mist gebaut irgendwo
> in deinem Programm.

Das sehe ich ähnlich.

Timer0 musste ich von dem Arduino - Ballast befreien, um deinen 10mSec 
Timer betreiben zu können.
Bin halt nur so ein Arduino C++ Jünger.....

[grundschüler (Gast)]

holger schrieb:
> Das wird er nicht. Du hast einfach nur Mist gebaut irgendwo
> in deinem Programm.

guter Hinweis. Fehler gesucht, Fehler gefunden, hatte absolut nichts mit 
dem Timer zu tun.

Ulrich F. schrieb:
> Das sehe ich ähnlich

Nochmals danke für die Unterstützung. Der Hinweis auf den 
funktionierenden wdg beim arduino war hilfreich.