msp430f5529 Timer-ISR wird nicht aufgerufen

[Commer (Gast)]

Hallo
folgender Code funktioniert nicht (vgl. 
Beitrag "Rote LED an Port 1.0 will nicht")

 
#include "msp430f5529.h"            // Einbinden der Definitionen
#define  LED_X     0x01     // LED (rot) an P1.0



int main( void )
{
   WDTCTL = WDTPW + WDTHOLD;   // WatchDogTimer abschalten
   
 // -- INIT_Port1
  P1SEL &= ~(LED_X);            // Beide sind BIN-IO       
  P1DIR |= LED_X;               // LED    = BIN-OUT
  P1OUT &=~ (LED_X);            // LED_X = AUS (weil H-aktiv)
  
  
  
 //-- TimerB0

  TB0CTL = TBCLR;
  TB0CTL = TBSSEL_2 + ID_0 + MC_1;
  TB0CCR0 = 1048;  // entspricht 1000 Hz, Freq [18]
  TB0CCR2 = 2096;  // entspricht 2*10^-3 s, also CCR0/2 (Pulsbreite halb so groß wie Freq)
  TB0CCTL2 = OUTMOD_7;
 // TB0CCTL0 = CLLD_1;          // Änderung der Frequenz wird erst übernommen, wenn 
                            // CCR0 erreicht ist
 // TB0CCTL2 = CLLD_1;
_BIS_SR(GIE);
  while(1) {_NOP();} 
}


#pragma vector = TIMER0_B0_VECTOR
__interrupt void TB0_ISR ()
{
 P1OUT ^= (LED_X);
}


#pragma vector = TIMER0_B1_VECTOR
 __interrupt void TIMERB0_1_ISR ()
{
  switch (TB0IV)
  {
  case TB0IV_TBCCR2:       // TB0CCR2, da LED nur so lange wie PBreite [i] andauert leuchten soll
    {
      P1OUT ^= (LED_X);
      break;
    } 
  }
}

Die LED geht problemlos außerhalb der ISR an.

Grüße

[der nixwois (Gast)]

Bleib doch beimn alten Tread und mach kein neues auf.
Und nimm doch endlich die Bretter von deinem Kopf weg.

[Herman Kokoschka (Gast)]

Ich kann die Verstrickung der beiden ISR im Moment nicht nachvollziehen,
liegt aber auch daran, dass ich Zweck und Hintergrundinfos nicht habe.

Commer schrieb:
> TB0CTL = TBSSEL_2 + ID_0 + MC_1;
Z.B. kann ich dies nicht auflösen.

Grundsätzliche Frage:
Beide ISR schalten ja die Led "um".
Kann es sein, dass die Feuerbedingungen der beiden ISR so ungünstig 
sind,
dass die LED zwar an geht, aber SO SCHNELL wieder aus, dass Du es nicht 
sehen kannst?

Probiere doch die beiden ISR mal EINZELN aus, auch bei schneller 
Feuerrate kannst Du dann Flimmern sehen oder messen und daran ermitteln 
ob die ISR überhaupt auslösen.

[Der müde Joe (Gast)]

Dir ist schon klar, dass

... = ... + ...

etwas anderes ist als

... = ... +| ...

Im ersten Fall uberschreibst du den Wert, im zweiten Fall veroderst du. 
Ich denke, du willst verodern.

[MaWin (Gast)]

der nixwois schrieb:
> Bleib doch beimn alten Tread und mach kein neues auf.

Das wollte doch der Oberforumshirte Stefan so. (???)

[Commer (Gast)]

Herman Kokoschka schrieb:
> Probiere doch die beiden ISR mal EINZELN aus, auch bei schneller
> Feuerrate kannst Du dann Flimmern sehen oder messen und daran ermitteln
> ob die ISR überhaupt auslösen.

guter Gedanke, auch das habe ich schon probiert. Bei beiden (jeweils 
einzeln getestet) leuchtet sie nicht

[Commer (Gast)]

MaWin schrieb:
> Das wollte doch der Oberforumshirte Stefan so. (???)

hahahha genau so ist es. soll doch nicht so beratungsresistent sein

[Commer (Gast)]

Der müde Joe schrieb:
> Dir ist schon klar, dass
>
> ... = ... + ...
>
> etwas anderes ist als
>
> ... = ... +| ...
>
> Im ersten Fall uberschreibst du den Wert, im zweiten Fall veroderst du.
> Ich denke, du willst verodern.

absoluter bullshit. bei der initialisierung ist + oder | gleichbedeutend

[Commer (Gast)]

habe die ISR von CCR0 zum laufen benkommen (CCIE). die von CCR2 
funktioniert immernoch nicht

[Commer (Gast)]

also habe die isr für ccr2 noch immer nicht zum laufen gebracht. stimmt 
etwas mit dem syntax nicht?

[Clemens L. (c_l)]

Commer schrieb:
> TB0CTL = TBSSEL_2 + ID_0 + MC_1;

Magische Zahlen erschweren das Verständnis. Bitte benutze:

TB0CTL = TBSSEL__SMCLK + UD__1 + MC__UP;

> TB0CCR0 = 1048;
> TB0CCR2 = 2096;
> TB0CCTL2 = OUTMOD_7;

Der Interrupt für CCR2 wird genau dann ausgeführt, wenn CCIE in CCTL2 
gesetzt ist (was nicht der Fall ist) und wenn der Timer 2096 erreicht 
(was nie passiert, weil der Timer im Up-Modus nur bis CCR0 zählt).

[der nixwois (Gast)]

Ich hab mit der Addition ein Verständnisproblem :
 TB0CTL = TBSSEL_2 + ID_0 + MC_1;

Müsste den die Zeile nicht  so aussehen?
 TB0CTL = ( TBSSEL_2 | ID_0 | MC_1 );

Es sollen doch nur die entsprechenden Bits gesetzt werden in TB0CTL.

Bei der Addtion könnte es in die Hose gehen.

[Commer (Gast)]

Clemens L. schrieb:
> wenn CCIE in CCTL2
> gesetzt ist (was nicht der Fall ist)

das habe ich getan, habe ich oben mitgeteilt.

Clemens L. schrieb:
> wenn der Timer 2096 erreicht
> (was nie passiert, weil der Timer im Up-Modus nur bis CCR0 zählt)

d.h. ich muss die Werte der beiden CCRn-Register tauschen?

[Commer (Gast)]

Commer schrieb:
> d.h. ich muss die Werte der beiden CCRn-Register tauschen?

P.S es funktioniert tatsächlich haha

[MaWin (Gast)]

der nixwois schrieb:
> Bei der Addtion könnte es in die Hose gehen.

Ja, bei Überdeckung oder Überlappung von Werten/Wertebereiche der Bits. 
Die Schreibweise ist daher nicht zu empfehlen und besser | zu verwenden.

Ganz Sch... ist (1 << BIT4711)!