MSP430G2553 Launchpad, Timer0 problem

[Sören K. (foxalem)]

Moin,
ich versuch nun seit 2 Stunden den Timer zum laufen zu bekommen.
Im grunde möchte ich nur den Timer mit 20ms durchlaufen lassen und bei 
einem Interrupt die Bits von P2.0 bis P2.2 "durchshiften". D.h. das 
immer nur einer von diesen Portpins aktiv ist.

Mein Problem, er springt anscheinend nicht in die ISR des Timers. Ich 
weiß nicht ob er nicht anläuft oder ob er einfach nich reinspringt etc. 
pp.
Evtl. kann mir auch jemand erklären wie ich mir das anzeigen lassen 
könnte beim Debugging (wie in Keil zum beispiel, allerdings ist dies ja 
ein Simulator.)

Anbei mein Code und schonmal danke für die Hilfe.

/*
 * main.c
 * Ein Programm zum Ansteuern und durchlaufen lassen von einem RGB-SMD-LED-Streifen.
 * Die ausgänge gehen über einen 1k-Widerstand auf die Basis eines BC538, welcher die
 * Masse auf die einzelnen LEDs durschaltet.
 */

#include "msp430g2553.h"

void main(void) {
  WDTCTL = WDTPW + WDTHOLD;  // Stop WDT

  P2DIR = 0xFF;      //Port 2 als Ausgang nutzen
  P2OUT = 0x00;      //Port 2 ausstellen
  
  //Timer0 einstellen
  TACCR0 = 0xC350;    //=50000 * 250000 * e-1 = 20ms
  TACTL = TAIE + TASSEL1 + ID1;  //Up-Mode, zählt bis TACCRO0, Interrupt enabled

  P2OUT |= BIT0;
  TACTL |= MC0;
  __bis_SR_register(GIE); // interrupts enabled

  while (1)
  {

  }
}


// Timer0 interrupt Service Routine
#pragma vector=TIMER0_A0_VECTOR
__interrupt void Timer0_ISR (void)
{
  if (P2OUT == BIT3)
  {
    P2OUT = BIT0;
  } else {
    P2OUT = P2OUT << 0x01;
  }
}

Mfg

[yes MSP430 can (Gast)]

Welche IDE verwendest du? Ich nutze IAR und den integrierten Simulator 
auch für Interrupts.

Probier einmal die Beispiele hier aus dem Forum:
http://www.mikrocontroller.net/articles/MSP430_Codebeispiele#Initialisierung_des_Timers_A

oder eine fertige Lösung:
http://processors.wiki.ti.com/index.php/MSP430_LaunchPad_LED_Timer#Code

[Sören K. (foxalem)]

Danke für die Codebeispiele ;)
Mir ist beim ansehen aufgefallen, dass ich ein Interrupt nicht aktiviert 
habe (CCIE im TACCTL0 / Compare-Capture Register 0).
Allerdings ist es wohl jener welcher, den ich dafür benötige.

Ich benutze CCS5 von TI, muss mich da wohl einfach nur noch richtig 
reinfuchsen, damit ich evtl besser Debuggen kann.

Mfg

Edit:
Jetzt springt er zwar in die ISR rein, allerdings is der Programmcounter 
nach dem RETI bei 0xFFFF und bleibt dort hängen.
Is irgendwie nich mein Tag heute

Edit2:
"This is typically the case when you have an itnerrupt enabled but no 
ISR for it. The CPU will then fetch the default 0xffff from the 
interrupt vector table as the ISR's address and jumps there. Of course 
there is no source code available or disassembly possible. It's a crash 
situation. And yes, the MSP won't forgive this mistake :)" <---
Ich hatte in TACTL noch das TAIE-Interrupt gesetzt, von dem ich ja 
dachte es löst den Interrupt für den Vector TIMER0_A0_VECTOR aus.

[Torsten C. (torsten_c)]

TAIE geht doch auf den TIMER0_A1_VECTOR und nicht auf den 
TIMER0_A0_VECTOR.

Also CCS5 von TI macht's einem doch echt leicht, sowas in Zukunft auch 
selbst nachzuschauen, siehe Anlage.

[Torsten C. (torsten_c)]

Hallo Sören, geht's nun mit dem TIMER0_A1_VECTOR bei Dir?

Ich selber hatte bis eben noch mit dem Assembler-Beispiel in slau144i, 
Kapitel 12.2.6.3 zu kämpfen. Ich arbeite mit Funktions-Vektoren, da ich 
die Interrupt-Pointer für die Zustandsautomaten verbiege:

Beitrag "Interrupt-Pointer verbiegen für Zustandsautomat"

#include <msp430.h>
void dummy() {volatile int test = TA0IV;} // für asm("…") - ToDo: kann man TA0IV geschickter definieren?
…
#pragma vector=TIMER0_A1_VECTOR
__interrupt void TIMER0_TAIV_ISR(void) {
  // aus slau144i, Kapitel 12.2.6.3
  asm("        ADD &TA0IV,PC     ; Add offset to Jump table");
  asm("        RETI              ; Vector 0: No interrupt");
  asm("        JMP CCIFG_1_HND   ; Vector 2: TACCR1");
  asm("        JMP CCIFG_2_HND   ; Vector 4: TACCR2");
  asm("        RETI              ; Vector 6: Reserved");
  asm("        RETI              ; Vector 8: Reserved");
  asm("        RETI              ; Vector 10: TAIFG Flag");
  asm("        RETI              ; Back to main program");
  asm("CCIFG_2_HND:              ; Vector 4: TACCR2");
  asm("        CALL &I2C_State+0 ; Task starts here");
  asm("        RETI              ; Back to main program");
  asm("CCIFG_1_HND:              ; Vector 2: TACCR1");
  asm("        CALL &PWM_State+0 ; Task starts here");
  asm("        RETI              ; Back to main program");
}

Für TAIE musst Du natürlich das Vector-10-RETI durch ein entsprechendes 
CALL ersetzen.

Ich hoffe, ich konnte helfen.

Wenn einer 'ne Idee hat, wie ich/wir wir das "void dummy()" vermeiden 
können, nur zu. Es dient nur dazu das Symbol "TA0IV" für die erste 
Assembler-Zeile bekannt zu machen.

[nur mal so (Gast)]

Torsten C. schrieb:
> Ich arbeite mit Funktions-Vektoren, da ich
> die Interrupt-Pointer für die Zustandsautomaten verbiege:

Hmmm, ich sehe keine Interrupt-Pointer und keine verbogenen 
Funktions-Vektoren. In dem Beispiel von TI geht es um etwas anderes:
Für den einen Timer-Interrupt gibt es mehrere Quellen. Um die Quelle 
heraus zu finden, muss man das TAIV auswerten. Üblicherweise macht man 
das mit einer switch case in der ISR.

Beim TAIV gibt es eine Besonderheit und zwar wird nur der Wert des 
höchstwertigen Interrupts eingetragen. Die Werte der einzelnen 
Interrupt-Quellen im TAIV unterscheiden sich immer um zwei (0, 2, 4, 
...). Dies macht TI sich zum Nutzen und addiert einfach den Wert vom 
TAIV auf den PC. Der Code direkt dahinter ist eine Tabelle, in der die 
gewünschten Ausführungen für die jeweilige Interrupt-Quelle stehen. Je 
Eintrag stehen nur 2 Byte zur Verfügung, also steht da ein Jump oder 
gleich RETI.

Also besser du nimmst die switch case und nutzt die die Option für nur 
"gerade" Werte.

[nur mal so (Gast)]

Torsten C. schrieb:
> Wenn einer 'ne Idee hat, wie ich/wir wir das "void dummy()" vermeiden
> können, nur zu.

Ach ja, TAIV ist nur eine Adresse. Du kannst den Wert auch direkt 
angeben.

[Torsten C. (torsten_c)]

nur mal so schrieb:
> … TAIV ist nur eine Adresse. Du kannst den Wert auch direkt angeben.

Danke für die gute Erklärung. Genau das macht das o.g. 
Assembler-Beispiel aus dem Datenblatt slau144i in Kapitel 12.2.6.3.

nur mal so schrieb:
> Also besser du nimmst die switch case und nutzt die die Option für nur
> "gerade" Werte.

Switch Case dauert bei der Ausführung deutlich länger als

ADD &TA0IV,PC

Interrupts sollen "so kurz wie möglich" das laufende Programm 
unterbrechen, da sonst andere Interrupts verloren gehen könnten, es sei 
denn, man erlaubt Interrupts in Interrupts. Das ist aber egal, wenn man 
keine zeitkritischen Funktionen hat.

Switch Case ist in C natürlich einfacher zu programmieren.

"&I2C_State" und "&PWM_State" sind übrigens die o.g. "verbogenen 
Interrupt-Pointer", aus dem verlinkten Beitrag, die vorher wie folgt 
deklariert sind und je nach Zustand der "State-Machine" andere Werte 
haben:

void(*PWM_State)(void) = &PWM_State_idle;
void(*I2C_State)(void) = &I2C_State_idle;

@"nur mal so": kann man TA0IV auch geschickter definieren, als durch die 
Dummy-Funktion? Der Assembler kennt sonst das Symbol nicht.

VG Torsten

[nur mal so (Gast)]

Torsten C. schrieb:
> Interrupts sollen "so kurz wie möglich" das laufende Programm
> unterbrechen

Sehe ich auch so, aber

Torsten C. schrieb:
> asm("        CALL &I2C_State+0 ; Task starts here");

ist dann eher suboptimal, denn vom CALL will das Programm mit RET wieder 
zurück. Und solange ist die ISR noch aktiv, solange bis sie auf ein RETI 
läuft.



Torsten C. schrieb:
> nur mal so schrieb:
>> … TAIV ist nur eine Adresse. Du kannst den Wert auch direkt angeben.
Torsten C. schrieb:
> Der Assembler kennt sonst das Symbol nicht.

Hinter TAIV oder TA0IV steckt die Adresse eines Registers. Schau ins 
Datenblatt.

Torsten C. schrieb:
> "&I2C_State" und "&PWM_State" sind übrigens die o.g. "verbogenen
> Interrupt-Pointer"

Dat sind käne verbojene Intrups!!!
In der ISR vom Timer nutzt du eine Sprungtabelle, um weiteren Code 
auszuführen oder eine Funktion (auch über Funktionszeiger) aufzurufen. 
Richtig wäre direkt die Interrupt-Vektortabelle zu verändern und dort 
einen Sprung auf die aktuell gewünschte ISR einzutragen. Das geht 
natürlich im Flash nicht so gut, schnell und häufig.

Ein guter Compiler erzeugt eine effektive Sprungtabelle automatisch. 
Beim MSP430 solltest du einmal nach "even_in_range" suchen. Die 
Bedeutung wird meist im Zusammenhang mit dem TAIV erklärt.

[Torsten C. (torsten_c)]

nur mal so schrieb:
> Torsten C. schrieb:
>> Der Assembler kennt sonst das Symbol nicht.
>
> Hinter TAIV oder TA0IV steckt die Adresse eines Registers. Schau ins
> Datenblatt.

Der Assembler kennt das Datenblatt nicht. ;-) Die Zeile sorgt dafür, 
dass dem Assembler das Label bekannt wird. Hat niemand eine schlauere 
Lösung? Egal, so geht's auch.

nur mal so schrieb:
> Dat sind käne verbojene Intrups!!!  … Das geht
> natürlich im Flash nicht so gut, schnell und häufig.

OK, es sind keine "verbogenen Interruppts", es ist eine Art, 
Interrupt-Vektoren zu verbiegen, ohne in's Flash zu schreiben. Die 
meisten Alternativen wurden im 
Beitrag "Interrupt-Pointer verbiegen für Zustandsautomat" ^^ ja Diskutiert.

> Beim MSP430 solltest du einmal nach "even_in_range" suchen.

Ich hab's mal ausprobiert, so richtig optimerit sieht das noch nicht 
aus. "ADD &TA0IV,PC" wäre optimal. Vielleicht mache ich noch was falsch:

Timer0_TAIV_ISR:
        PUSH.W    r15
        MOV.W     &TA0IV+0,r15
        SUB.W     #2,r15
        JEQ       $C$L3
        SUB.W     #2,r15
        JEQ       $C$L3
        SUB.W     #6,r15
        JEQ       $C$L3
$C$L2:    
        JMP       $C$L2

Hier der C-Code dazu:

#pragma vector=TIMER0_A1_VECTOR
__interrupt void Timer0_TAIV_ISR (void) {
  switch (__even_in_range(TA0IV,10)) {
    case 0x02: // TACCR1 CCIFG
      // ToDo: Event 1 ergänzen
      break;
    case 0x04: // TACCR2 CCIFG
      // ToDo: Event 2 ergänzen
      break;
    case 0x0A: // TAIFG
      // ToDo: Event 3 ergänzen
      break;
    default:
      while (1); // trap CPU on strange TAIV
  }
}

Eigentlich sollte es so gehen, wie hier beschrieben:

http://e2e.ti.com/support/microcontrollers/msp430/f/166/t/238317.aspx

[Torsten C. (torsten_c)]

PS: Komisch, so macht der Compiler die Vereinfachung:

Aus diesem C-Code:

  … TA0IVRead = TA0IV;
    switch (__even_in_range(TA0IVRead,10)) …

Wird dieser Assembler:

    MOV.W     &TA0IV+0,&TA0IVRead$4+0
    ADD.W     &TA0IVRead$4+0,PC

Irgendwie kann "even_in_range" das SFR TA0IV nicht vertragen. :-(

[Torsten C. (torsten_c)]

Torsten C. schrieb:
> Hallo Sören, geht's nun mit dem TIMER0_A1_VECTOR bei Dir?
Keine Antwort? Ich habe den Haken bei "E-Mail-Benachrichtigung 
einschalten" entfernt. Wenn's noch 'ne Frage gibt:
http://www.mikrocontroller.net/user/show/torsten_c

[nur mal so (Gast)]

Torsten C. schrieb:
> Irgendwie kann "even_in_range" das SFR TA0IV nicht vertragen.

Hmmmh, ich weiss nicht wie du das anstelltst, aber mein IAR macht ohne 
große Optimierung aus

void foo(void);
void (*funcarr[3])(void) = {foo, foo, foo};

#pragma vector = TIMER0_A1_VECTOR
__interrupt void TIMER0_A1_VECTOR_ISR (void)
{
  switch (__even_in_range(TAIV, 10)){
    case 0x02: // TACCR1 CCIFG
      funcarr[0]();
      break;
    case 0x04: // TACCR2 CCIFG
      funcarr[1]();
      break;
    case 0x0A: // TAIFG
      funcarr[2]();
      break;
    default:
      while (1); // trap CPU on strange TAIV
  }
}

das

   \   000000   0D12         PUSH.W  R13
   \   000002   0C12         PUSH.W  R12
   \   000004   0F12         PUSH.W  R15
   \   000006   0E12         PUSH.W  R14
   \   000008   10522E01     ADD.W   &0x12e, PC
   \                     `?<Jumptable for TIMER0_A1_VECTOR_ISR>_0`:
   \   00000C   123C         JMP     ??TIMER0_A1_VECTOR_ISR_1
   \   00000E   043C         JMP     ??TIMER0_A1_VECTOR_ISR_2
   \   000010   0A3C         JMP     ??TIMER0_A1_VECTOR_ISR_3
   \   000012   0F3C         JMP     ??TIMER0_A1_VECTOR_ISR_1
   \   000014   0E3C         JMP     ??TIMER0_A1_VECTOR_ISR_1
   \   000016   0A3C         JMP     ??TIMER0_A1_VECTOR_ISR_4
   \                     ??TIMER0_A1_VECTOR_ISR_2:
   \   000018   9212....     CALL    &funcarr
   \                     ??TIMER0_A1_VECTOR_ISR_0:
   \   00001C   3E41         POP.W   R14
   \   00001E   3F41         POP.W   R15
   \   000020   3C41         POP.W   R12
   \   000022   3D41         POP.W   R13
   \   000024   0013         RETI
   \                     ??TIMER0_A1_VECTOR_ISR_3:
   \   000026   9212....     CALL    &funcarr + 2
   \   00002A   F83F         JMP     ??TIMER0_A1_VECTOR_ISR_0
   \                     ??TIMER0_A1_VECTOR_ISR_4:
   \   00002C   9212....     CALL    &funcarr + 4
   \   000030   F53F         JMP     ??TIMER0_A1_VECTOR_ISR_0
   \                     ??TIMER0_A1_VECTOR_ISR_1:
   \   000032   FF3F         JMP     ??TIMER0_A1_VECTOR_ISR_1

Da braucht man keinen ASM mehr.

Und nebenbei kannst du sehen, wie anstatt TAIV die Register-Adresse aus 
dem Datenblatt eingesetzt wurde.