Globale Variable in ISR immer 0

[Korbinian (Gast)]

Hallo,

ich beschäftige mich seit kurzem mit dem TI MSP430. Dazu habe ich mir 
ein Launchpad gekauft und versuche mich gerade mit einem kleinem 
Programm, das über Interrupts gesteuert LED's ansteuert.

Leider wird die globale Variable wahl zwar in der Routine erhöht, 
allerdings ist sie beim nächsten Einsprung in die ISR wieder den Wert 0.
Ich sehe da keine Ursache, wer weiß eine Lösung.

gruß
Korbinian

#include "msp430.h"
unsigned char wahl; /* global variable */

#pragma vector = TIMERA1_VECTOR
__interrupt void TIMERA_OVF(void)
{
  TACTL = 0x02E0; /* Timer counts to 0xFFFF, Divider 8, INT disabled , INTFLAG disabled */
  if(wahl < 0x80) 
  {
    P1OUT ^= BIT0;
  }
  else if (wahl >= 0x80)
  {
    P1OUT ^= BIT6;
  }
  else if (wahl == 0xFF)
  {
    wahl = 0;
  }
  wahl++;
  __delay_cycles(10000);
  TACTL = 0x02E2; // Timer counts to 0xFFFF, Divider 8, INT enabled , INTFLAG disabled
}

int main( void )
{
  P1DIR = 0xFF; /* PORT1 direction: output */
  P1OUT = 0x00; /* Init PORT1 */
  __enable_interrupt(); /* enable global interrupt */
  TACTL = 0x02E2; /* Timer counts to 0xFFFF, Divider 8, INT enabled , INTFLAG disabled */
  for(;;)
  {
    /* do nothing */
  }
}

[Rufus Τ. F. (rufus)]

Das Stichwort lautet volatile.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Rufus Τ. Firefly schrieb:
> Das Stichwort lautet volatile.
Auch bei dieser Frage? Wie kommt der Compiler da drauf?

Wenn die Variable in der ISR geändert wird, dann sollte sie beim 
nächsten Interruptaufruf doch den geänderten Wert haben. Mag sein, dass 
ein volatile hier hilft, aber wo ist dieses Verhalten von globalen 
Variablen dokumentiert?


Aber das gibt mir in einer Interruptroutine doch sehr zu denken:
  __delay_cycles(10000);

[Korbinian (Gast)]

Hab ich schon probiert, es hilft nichts.

volatile unsigned char wahl;

[codehamster (Gast)]

Wo wird das Interrupt Flag des Timers gelöscht?

oder hab ich das übersehen?

[Korbinian (Gast)]

TACTL = 0x02E0; /* Timer counts to 0xFFFF, Divider 8, INT disabled , INTFLAG disabled */

Das INT Flag ist Bit0 laut User Manual des MSP430.

[Sascha (Gast)]

Hast du den Watchdog des MSP430 deaktiviert? Soweit ich das noch weiß, 
muss man den explizit deaktivieren, sonst zwingt er den uC dauernd in 
den Reset!

[Sascha (Gast)]

Sorry für den Doppelpost.

Sascha schrieb:
> Hast du den Watchdog des MSP430 deaktiviert? Soweit ich das noch weiß,
> muss man den explizit deaktivieren, sonst zwingt er den uC dauernd in
> den Reset!

Den Watchdog kannst du so deaktivieren:

// Stop watchdog timer to prevent time out reset
WDTCTL = WDTPW + WDTHOLD;

[Korbinian (Gast)]

AHHHHHH... jetzt wird mir auch klar warum in den Beispiel Projekten als 
erstes im Hauptprogramm steht:

[c]WDTCTL = WDTPW + WDTHOLD;             // Stop watchdog timer[c]

Vielen Dank für den Hinweis, jetzt funktioniert es und ich hab schon 
fast an meinen Kenntnissen gezweifelt ;).

[Korbinian (Gast)]

AHHHHHH... jetzt wird mir auch klar warum in den Beispiel Projekten als 
erstes im Hauptprogramm steht:

WDTCTL = WDTPW + WDTHOLD;             // Stop watchdog timer

Vielen Dank für den Hinweis, jetzt funktioniert es und ich hab schon 
fast an meinen Kenntnissen gezweifelt ;).

[Sascha (Gast)]

Aber die Zeile hier solltest du auch aus der ISR killen! Wartezeiten in 
der ISR = böse ;-)

 __delay_cycles(10000);

[Korbinian (Gast)]

Die ist schon lange wieder rausgeflogen ;) Sollte auch nur eine 
Notlösung sein. Nun funktioniert auch alles wie es soll.