MSP430 Watchdog aktivieren nach definierter Zeit

[mikrocomputerstudent (Gast)]

Hallo,
ich komme gerade bei einer Hausaufgabe für die Uni nicht weiter.
Es wäre nett wenn mir jemand dabei helfen könnte.

Aufgabe:
Man soll eine grüne LED mit f = 4 Hz blinken lassen. (Die rote LED habe 
ich nur zum Debuggen hinzugeschaltet)
Falls man den Button drückt, dann soll der Controller in einen 
künstlichen Deadlock geraten und nach 5,5 Sekunden durch den Watchdog 
Timer zurückgesetzt werden. Um den Watchdog Timer zu initialisieren, 
soll man ACLK als Watchdog Timer Source benutzen und das Watch Dog Timer 
Control Register entsprechend setzen.

Achso, ACLK hat eine Frequenz von 32768 Hz.

Hier wäre mein Code dazu:

#include <msp430g2553.h>

int buttonPressed;

int main(void)
{
    WDTCTL = WDTPW | WDTHOLD;  // stop watchdog timer

    // initialization of the basic clock module
    BCSCTL3 |= LFXT1S_2;

    // initialization of wdtctl register
    WDTCTL = (WDTPW | WDTSSEL | WDTCNTCL );

    // P1.4 -> output to red LED
    // P1.0 -> output to green LED
    P1DIR |= BIT0;
    P1DIR |= BIT4;

    // Initialization for interrupts
    // P1.3 -> input of button
    P1SEL &= (~BIT3); // set P1.3 SEL as GPIO
    P1DIR &= (~BIT3); // set P1.3 SEL as Input
    P1IES |= (BIT3); // falling edge 1 -> 0
    P1IFG &= (~BIT3); // clear interrupt flag for P1.3
    P1IE |= (BIT3); // enable interrupt for P1.3
    P1REN |= BIT3;  // enable pull-up/-down resistor

    __enable_interrupt();

    buttonPressed = 0;


    while (1) {
        P1OUT ^= BIT0;
        if (buttonPressed == 1) {
            P1OUT ^= BIT4;
            P1OUT = 0x00;
            buttonPressed = 0;
            while (1) {
                // artificially created dead-lock
            }
        } else {
            // f = 4 Hz => T = 1 s / 4 Hz = 250 ms
            __delay_cycles(250000);
        }
        WDTCTL = (WDTPW | WDTSSEL | WDTCNTCL );  // feed the dog
    }
}


// Port 1 interrupt vector
#pragma vector = PORT1_VECTOR
__interrupt void Port_1(void)
{
    buttonPressed = 1;
    P1IFG &= ~BIT3; // clear interrupt flag
}

Probleme/Fragen hierzu:
- Wenn ich den Button drücke, dann hört die rote und die grüne LED für 
ein paar Sekunden auf zu blinken. Woraus ich schlussfolgere dass sich 
der Controller im Deadlock befindet. Danach blinkt nur noch die grüne 
LED und wenn ich den Button nochmal drücke, scheint der Controller nicht 
darauf zu reagieren, also es ändert sich nichts mehr an dem Zustand. 
Woran liegt das, was genau ist passiert?
- Wie kann ich gewährleisten dass der Watchdog den Deadlock nach exakt 
5,5 Sekunden zurücksetzt?
- Fallen jemandem irgendwelche Fehler bei meinem Code auf?

ps: Ich erwarte natürlich nicht dass hier jemand meine Aufgabe für mich 
löst. Ein paar hilfreiche Antworten erhoffe ich mir allerdings schon :)

[mikrocomputerstudent (Gast)]

Ich habe noch eine Frage:
Mit dem Bitwert von WDTTMSEL kann man die watchdog timer+ mode 
einstellen, bei einer gesetzten 0 kommt er laut Datenblatt in die 
watchdog mode und bei einer 1 in die interval timer mode.
Ich bin jetzt davon ausgegangen dass man dies einstellt in dem man für 
eine 1 WDTCTL = ... | WDTTMSEL hinzuverodert und das für eine 0 einfach 
weglässt.
Denn WDTCTL = ... | ~WDTTMSEL für eine logische 0 scheint nicht zu 
funktionieren.
Ist dies korrekt?

[Clemens L. (c_l)]

mikrocomputerstudent schrieb:
> Wie kann ich gewährleisten dass der Watchdog den
> Deadlock nach exakt 5,5 Sekunden zurücksetzt?

Schau dir mal die Doku zu WDTCTL an.

mikrocomputerstudent schrieb:
> Fallen jemandem irgendwelche Fehler bei meinem Code auf?

P1OUT.3 wird am Anfang nicht initialisiert, und dann in der Schleife 
überschrieben.

[mikrocomputerstudent (Gast)]

Clemens L. schrieb:
> mikrocomputerstudent schrieb:
>> Fallen jemandem irgendwelche Fehler bei meinem Code auf?
>
> P1OUT.3 wird am Anfang nicht initialisiert, und dann in der Schleife
> überschrieben.

O man, du hast natürlich recht, danke.

Habe den entsprechenden Abschnitt jetzt wie folgt korrigiert:

...
    P1OUT |= BIT0;  // ---- das ist neu

    __enable_interrupt();

    pb5Pressed = 0;


    while (1) {
        P1OUT ^= BIT0;
        if (pb5Pressed == 1) {
            P1OUT ^= BIT4;
            P1OUT &= (~BIT0);  // ---- das ist neu
            pb5Pressed = 0;
            while (1) {
                // artificially created dead-lock
...

 und es funktioniert.
Ich kann den Watchdog Timer jetzt auch mehrmals mit dem Button auslösen, 
so wie es sein sollte :)

Clemens L. schrieb:
> Schau dir mal die Doku zu WDTCTL an.

Dann sehe ich mir nochmal die Doku an. Echt mühselig das ganze :)

[mikrocomputerstudent (Gast)]

Also das mit den 5,5 Sekunden habe ich jetzt glaub hingekriegt, bin mir 
aber nicht ganz sicher warum es so funktioniert.
Habe es mit einer Stoppuhr nachgemessen :)

Ich habe etwas im MSP430_Guide rumgesucht und bin darauf gekommen, dass 
man ja auch die Basic Clock Modul Frequenz verkleinern kann.
Das habe ich dann auch getan, mit dem Hinzufügen folgender Code-Zeile:

    
    // divide ACLK with 2
    BCSCTL1 |= DIVA0;

Laut dem Datenblatt wird die Basic Clock, die in diesem Fall ja gleich 
ACLK ist und somit eine Frequenz von 32768 Hz hat, durch 2 geteilt.
Also müsste sie ja 16384 Hz betragen.
Im Datenblatt steht dass der Watchdog aktiv wird sobald einmal "eine 
Periode" hochgezählt wurde. Warum ist der Intervall dann nicht 2 
Sekunden?
Oder 4 Sekunden, falls nur bei der falling oder rising edge gezählt 
wird?

[Clemens L. (c_l)]

mikrocomputerstudent schrieb:
> Im Datenblatt steht dass der Watchdog aktiv wird sobald einmal "eine
> Periode" hochgezählt wurde. Warum ist der Intervall dann nicht 2
> Sekunden?

Die Periode wird mit den WDTIS-Bits eingestellt.

Aber solange der Quarz-Oszillator nicht korrekt initialisiert wurde, 
läuft ACLK nicht mit 32768 Hz. Und schau mal nach, was du in BCSCTL3 
eingestellt hast.