MSP430 Timer + Port Interrupt

[bumpl (Gast)]

Hi,

ich habe ein Problem mit den Timern beim MPS430F5438 und den Interrupts.
Folgendes: Ich starte einen Timer, der per Interrupt eine Zählvariable 
setzt.
Zusätzlich soll aber noch auf eine Lo/Hi-Fanke an Pin 1.4 reagiert 
werden.

Code ist hier:

void main(void)
{
  WDTCTL = WDTPW + WDTHOLD;                 // Stop WDT
  P1DIR |= 0x01;                            // P1.0 output
  TBCTL = TBSSEL_2 + MC_2 + TBCLR + TBIE;   // SMCLK, contmode, clear TBR
                                            // enable interrupt
  
  P1REN |= 0x10;                            // Enable P1.4 internal resistance
  P1OUT |= 0x10;                            // Set P1.4 as pull-Up resistance
  P1IE |= 0x10;                             // P1.4 interrupt enabled
  P1IES |= 0x00;                            // P1.4 Lo/Hi edge
  P1IFG &= ~0x10;                           // P1.4 IFG cleared

  //__bis_SR_register(LPM4_bits + GIE);       // Enter LPM4, enable interrupts
  __no_operation();                         // For debugger
}

// Port 1 interrupt service routine
#pragma vector=PORT1_VECTOR
__interrupt void Port_1(void)
{
 //mach was
}


#pragma vector=TIMERB1_VECTOR
__interrupt void TIMERB1_ISR(void)
{
  count++;
}

Das komische ist, dass beides jeweils einzeln funktioniert. Also, wenn 
ich den Timer nicht starte, oder einfach den Interrupt dafür nicht 
aktiviere (also TBIE weglasse), funktioniert der Interrupt auf Pin 1.4 
super.
Schalte ich allerdings den Timer-Interrupt ein, funktioniert dieser auch 
wunderbar, allerdings springt er mir nicht mehr in den Port-Interrupt.

Gitb's da irgendeinen Konflikt mit den Prioritäten? Oder sonst was?

Gruß,

bumpl

[Peter D. (pdiener)]

Solange die Interrupts nicht global eingeschaltet sind, wird nicht viel 
passieren.

Sie sind nicht eingeschaltet, weil
  //__bis_SR_register(LPM4_bits + GIE);       // Enter LPM4, enable 
interrupts

auskommentiert ist.

Es würde auch __bis_SR_register(GIE); ausreichen

Außerdem verlässt das Programm Main am Ende. Das darf nie passieren, 
sonst wird Code ausgeführt, der danach kommt, was zu irgendeinem 
beliebigen Verhalten führen kann.

Also sollte am Ende von Main ein for(;;); stehen oder irgend etwas 
anderes, das dafür sorgt, dass main nie verlassen wird.

Grüße,

Peter

[Stefan (Gast)]

Muss man beim F5xxx nicht auch PxIFGx per Software zurücksetzen (wie 
beim F1xxx/2xxx) bzw. das PxIV Register auslesen um PxIFGx zu löschen?
Sonst springt er ja immer wieder in die Port-ISR!

[Raimund (Gast)]

Oh...
hab das Aktivieren der Interrupts Testweise mal auskommentiert, aber 
anders hats auch nicht funkioniert...

Main wird glaube ich nicht verlassen, weil ja beide Interrupts einzeln 
funktionieren. Außerdem habe ich das aus 2 Beispielen von TI 
zusammengebastelt, bei denen am Ende auch keine Englosschleißfe läuft.

>Muss man beim F5xxx nicht auch PxIFGx per Software zurücksetzen (wie
>beim F1xxx/2xxx) bzw. das PxIV Register auslesen um PxIFGx zu löschen?
>Sonst springt er ja immer wieder in die Port-ISR!

Das habe ich gemacht (dummerweise hier weggelassen). Wie gesagt: Die 
Interrupts einzeln genommen zeigen gewünschtes Verhalten. Zusammen 
allerdings springt er nie in die __interrupt void Port_1(void) Routine.

Gruß,

bumpl

[Jörg S. (joerg-s)]

>Zusammen allerdings springt er nie in die __interrupt void Port_1(void)
>Routine.
Und im PxIFG Register steht dann nix?

[Stefan (Gast)]

>Also sollte am Ende von Main ein for(;;); stehen oder irgend etwas
>anderes, das dafür sorgt, dass main nie verlassen wird.
Das wäre sicher ein "sauberer" Programmierstil ;-)
Das ganze sieht aber nach IAR aus... der macht das automatisch!

>P1DIR |= 0x01;       // P1.0 output
Wäre ein Input nicht sinnvoller?

[Stefan (Gast)]

>>P1DIR |= 0x01;       // P1.0 output
>Wäre ein Input nicht sinnvoller?
Vergiß es... hab mich verguckt.

[bumpl (Gast)]

Hi,

im Register P1IFG ist P1IFG4 (für den Port 1.4) immer 0.

In dieser Konfiguration funktioniert der Interrupt auf Port 1.4 
hervorragend:

void main(void)
{
  WDTCTL = WDTPW + WDTHOLD;                 // Stop watchdog timer
  P1DIR |= 0x01;                            // Set P1.0 to output direction
  P1REN |= 0x10;                            // Enable P1.4 internal resistance
  P1OUT |= 0x10;                            // Set P1.4 as pull-Up resistance
  P1IE |= 0x10;                             // P1.4 interrupt enabled
  P1IES |= 0x10;                            // P1.4 Hi/Lo edge
  P1IFG &= ~0x10;                           // P1.4 IFG cleared

//  TBCTL = TBSSEL_2 + MC_2 + TBCLR + TBIE;   // SMCLK, contmode, clear TBR

  __bis_SR_register(LPM4_bits + GIE);       // Enter LPM4 w/interrupt
  __no_operation();                         // For debugger
}

// Port 1 interrupt service routine
#pragma vector=PORT1_VECTOR
__interrupt void Port_1(void)
{
  P1OUT ^= 0x01;                            // P1.0 = toggle
  P1IFG &= ~0x010;                          // P1.4 IFG cleared
}

Kommentier ich TBCTL wieder ein (sagt man das so? :)), dann geht der 
Interrupt wieder nicht. Starte ich den Timer mit

TBCTL = TBSSEL_2 + MC_2 + TBCLR;

geht auch der Interrupt auf Port 1.4 wie vorher ohne Probleme.

Es liegt also wohl an dem Aktivieren des Interrupts für den Timer (bei 
Timer1/A übrigens das gleiche Problem).

Gruß,

bumpl

[Stefan (Gast)]

Kontaktier mal den TI support.
Wäre ja nicht das erste Mal, dass die Errata nachgebessert werden muss.
Hab die Jungs auch schon ein paar mal in Erklärungsnot gebracht ;-)

[Raimund (Gast)]

Hi,

tut mir leid, dass ich da nicht früher drauf gekommen bin und das Thema 
hier aufgebracht habe.
Die Lösung ist ziemlich naheliegend. Der Timer läuft mit einem Quarz auf 
32768 Hz. Die Priorität des Timers ist die höchste (15). Der Port 
Interrupt wird einfach nie ausgeführt, weil immer der Timer-Interrupt 
früher dran ist. Ich habe den Timer-Interrupt auf ca. 6,67 Hz gesetzt 
(mehr brauch ich) und jetzt geht auch der Port-Interrupt.

Gibt es denn eine Möglichkeit (vielleicht auch ASM), die Prioritäten der 
Interrupts zu ändern?

Gruß,

bumpl

[Rufus Τ. F. (rufus)]

Nein, die Interruptprioritäten sind fest vorgegeben.

[S. Z. (szimmi)]

-Raimund schrieb:
> Hi,
>
> tut mir leid, dass ich da nicht früher drauf gekommen bin und das Thema
> hier aufgebracht habe.
> Die Lösung ist ziemlich naheliegend. Der Timer läuft mit einem Quarz auf
> 32768 Hz. Die Priorität des Timers ist die höchste (15). Der Port
> Interrupt wird einfach nie ausgeführt, weil immer der Timer-Interrupt
> früher dran ist. Ich habe den Timer-Interrupt auf ca. 6,67 Hz gesetzt
> (mehr brauch ich) und jetzt geht auch der Port-Interrupt.
>
> Gibt es denn eine Möglichkeit (vielleicht auch ASM), die Prioritäten der
> Interrupts zu ändern?
>
> Gruß,
>
> bumpl

Irgendetwas schleift hier trotzdem noch. In Deinem Ursprungsprogramm 
wird der Timer vom SMCLK, und dieser vom DCO (ca. 1MHz) getaktet. somit 
bekommst Du aller 65536 * 1us = 65,536 ms einen Timer-Interrupt. Wenn 
der P1-Interrupt nun deswegen nicht drankommt, weil der Timer ständig 
läuft, ist mir das schleierhaft, da die Timer-ISR dann schlappe 65ms 
etwas zu tun hätte...
Das Problem hat nichts mit Prioritäten zu tun. Kannst Du nochmal Dein 
vollstaendiges Programm posten, welches nun problemlos läuft?

[Jörg S. (joerg-s)]

>Irgendetwas schleift hier trotzdem noch. In Deinem Ursprungsprogramm
>wird der Timer vom SMCLK, und dieser vom DCO (ca. 1MHz) getaktet. somit
>bekommst Du aller 65536 * 1us = 65,536 ms einen Timer-Interrupt. Wenn
>der P1-Interrupt nun deswegen nicht drankommt, weil der Timer ständig
>läuft, ist mir das schleierhaft, da die Timer-ISR dann schlappe 65ms
>etwas zu tun hätte...
Richtig, ausserdem müsste das IFG Bit vom Pin ja trotzdem gesetzt 
werden.

[Raimund W. (bumpl)]

Hi,

macht Sinn... Hier ist nun der Quellcode, der klappt:

#include "msp430x54x.h"

const F_A = 4915;

unsigned int count = 0;

void main(void)
{
  WDTCTL = WDTPW + WDTHOLD;                 // Stop WDT
  P1DIR |= 0x01;                            // P1.0 output
  
  TA0CCTL0 = CCIE;                          // CCR0 interrupt enabled
  TA0CCR0 = F_A;
  TA0CTL = TASSEL_1 + MC_1 + TACLR;         // ACLK, upmode
  
  P1REN |= 0x10;                            // Enable P1.4 internal resistance
  P1OUT |= 0x10;                            // Set P1.4 as pull-Up resistance
  P1IE |= 0x10;                             // P1.4 interrupt enabled
  P1IES |= 0x00;                            // P1.4 Lo/Hi edge
  P1IFG &= ~0x10;                           // P1.4 IFG cleared

  __bis_SR_register(LPM3_bits + GIE);       // Enter LPM3, enable interrupts
  __no_operation();                         // For debugger
}

// Port 1 interrupt service routine
#pragma vector=PORT1_VECTOR
__interrupt void Port_1(void)
{
 //mach was

  P1IFG &= ~0x010;                          // P1.4 IFG cleared
}


#pragma vector=TIMER0_A0_VECTOR
__interrupt void TIMER_A_ISR(void)
{
  count++;
  P1OUT ^= 0x01;
  TA1CCR0 += F_A;
  
}

[S. Z. (szimmi)]

Hiho,
also nochmal zu Deinem Urspungsprogramm.
Du sagtest, es hat entweder der eine oder der andere Interrupt 
zugeschlagen.
Den PORT1 Interrupt sehe ich ja noch ein, der andere (Timer) Interrupt 
kann gar nicht gekommen sein, da im LPM4 der SMCLK nicht läuft, somit 
auch der Timer steht.
Alles sehr seltsam. Dein letztes Programm sieht allerdings ganz gut aus, 
ACLK und LPM3, das passt.

[bumpl (Gast)]

Hi,

hab's grad eben mit dem ersten Code noch mal versucht. Wenn der 
Timer-Interrupt aktiviert ist (... + TBIE), funktionert NUR der 
Timer-Interrupt. Entferne ich den Timer-Interrupt, dann funktioniert der 
Port-Interrupt (aber logischerweise nicht mehr der Timer-Interrupt).

Gruß,

bumpl