ISR mit MSP430: Globale Variable lässt sich nicht beeinflussen, seltsames Verhalten

[Alibijogger (Gast)]

Frohe Weihnachten,

ich wollte heute mal eine Interrupt Service Routine ausprobieren, doch 
das klappt nicht wie ich es mir vorstelle. Hier mal der Code:

#include "io430.h"
#include "intrinsics.h"
#include "stdint.h"

volatile int i = 0;

void main(void)
{
  //Pins als Output deklarieren
  P1DIR = 255;
  
  //Watchdog anhalten
  WDTCTL = WDTPW + WDTHOLD; 
  
  //Basic Clock System initialisieren
  BCSCTL3 |= XCAP1 + XCAP0; 
  
  //TimerA initialisieren:
  // ACLK, Teiler 1, Aufwaertszaehlen, Interrupt an, starten
  TACTL = TASSEL_1 + MC_1 + TAIE + TACLR;
  
  //Zaehlgrenze einstellen:
  TACCR0 = 32768;
  
  //Interrupts einschalten:
  __enable_interrupt();
  
  //Interrupts am TimerA Kanal0 einschalten:
  TACCTL0 = CCIE;
  
  for(;;)
  {
    
  }
}

#pragma vector = TIMERA0_VECTOR
__interrupt void sekuendliches_aufwachen (void)
{
  P1OUT++;
  i++;
}

Zweck:
Binärer Zähler, der durch Aufrufe der ISR hochzaehlt (f = 1Hz).

Zwei Dinge wundern mich doch sehr:
-während ich im Debugger und an meinen LEDs sehen kann, wie P1OUT schön 
inkrementiert wird, zählt i nur einmal auf 1, der Befehl i++ scheint 
(warum auch immer) danach nicht mehr ausgeführt zu werden. Eine lokale 
Definition von i in der ISR selbst ändert daran nichts.

-wenn ich an der Stelle:

//Pins als Output deklarieren
P1DIR = 255;

noch ein P1OUT = 0 anhänge, leuchtet die LED an P1OUT.0 viel schwächer, 
und der Zählerstand kommt auch nicht über die 1 hinaus.

Ich verstehe echt nicht, was hier vor sich geht, vielleicht kann es mir 
ein anderer sagen. Achja, falls es was zur Sache tut: ich entwickle mit 
IAR Kickstart.

[Alibijogger (Gast)]

Nach ausführlichem Debugging scheint es, als würde die CPU immer wieder 
an den Anfang des Programmcodes springen und so ungewollt die Variable i 
nullen. Das selbe Problem mit P1OUT. Bloß wie die CPU dazu kommt, wieder 
an den Anfang zu springen, das weiß wohl nur Gott allein...

[Christian R. (supachris)]

Du hast zwei Interrupt-Quellen aktiviert. Einmal den Überlauf-Interrupt 
durch das TAIE Bit im TACTL Register und einmal den Compare-Interrupt 
durch das CCIE im TACCR Register. Hast aber nur einen Interrupt-Handler. 
Der Timer A hat 2 Interrupts, einmal der TACCR0, dafür hast du deine ISR 
und einmal einen für TACCR1, TACCR2 und TAIV (Überlauf) zusammen. Dafür 
hast du keine ISR, daher schmiert dein µC immer ab und startet neu, wenn 
der Zähler überläuft. Mach das TAIE Bit raus und dann wird es 
funktionieren.

[Alibijogger (Gast)]

Jetzt geht es, ich danke dir! In dem Buch was ich gerade lese empfahl 
der Autor, am besten auch für die nicht benötigten Vektoren/ Flags 
Routinen zu schreiben. Jetzt verstehe ich auch was er damit meinte.

Gibt's vielleicht einen Plan wo alle Vektoren mit den zugehörigen Flags 
aufgeführt sind? Ich habs im User Guide probiert, aber da steht 
größtenteils "device specific". Und woher weiß ich wie ich die Vektoren 
benennen muss?
vector = TIMERA0_VECTOR habe ich auch aus dem Buch aber wie heißen die 
anderen? Kann das auch von der IDE abhängen? Im User Guide steht z.B. 
"TACCR0 interrupt vector for TACCR0 CCIFG". Aber was nutzt mir das wenn 
ich den stattdessen mit TIMERA=_VECTOR ansprechen muss?!

[Christian R. (supachris)]

Hm, die stehen zumindest in der Header-Datei für den µC. Welchen 
Compiler nimmst du denn? Von Syntax her CCE oder IAR. Eventuell kann man 
da irgendwo einstellen, dass nicht benutzte Interrupt Vektoren einfach 
mit einem RETI beschrieben werden. GCC macht das von sich aus, da 
passiert erst mal nix.
Viel besser ist aber, nur die Interrupts zu aktivieren, die man wirklich 
braucht.

[Alibijogger (Gast)]

Guter Tipp mit dem Header. Hier bin ich fündig geworden:

/************************************************************
* Interrupt Vectors (offset from 0xFFE0)
************************************************************/

#define PORT1_VECTOR        (2 * 2u)  /* 0xFFE4 Port 1 */
#define PORT2_VECTOR        (3 * 2u)  /* 0xFFE6 Port 2 */
#define TIMERA1_VECTOR      (8 * 2u)  /* 0xFFF0 Timer A CC1-2, TA */
#define TIMERA0_VECTOR      (9 * 2u)  /* 0xFFF2 Timer A CC0 */
#define WDT_VECTOR          (10 * 2u) /* 0xFFF4 Watchdog Timer */
#define COMPARATORA_VECTOR  (11 * 2u) /* 0xFFF6 Comparator A */
#define NMI_VECTOR          (14 * 2u) /* 0xFFFC Non-maskable */
#define RESET_VECTOR        (15 * 2u) /* 0xFFFE Reset [Highest Priority] */

Weihnachten ist gerettet. Danke!