msp430 lmp4 timer

[lukas s. (lukas_s46)]

Hallo,
ich habe ein sehr seltsames Problem mit dem msp430f5137.
Ich versuche etwas ähnliches wie hier: 
Beitrag "MSP430  Interrupts zählen (Herzfrequenz)" zu realisieren
mein code:

unsigned int interruptCounter = 0;
volatile unsigned int cycles = 2;

void Device_MSP430F5137_InitClock(int period){
 TA0CTL = 0x0004;   // Timer1_A3 clear.
 TA0CCTL0 = CCIE; // Timer1_A3 Capture/compare 0 interrupt enable.
 TA0CCR0 = period;  // Set TACCR0 value (8Hz//3d09) (1Hz 0x07A1)
 TA0CTL = TASSEL_1 + MC_1 + ID_3;   // Selected: ACLK, divider:8, Up mode.
}

void Device_MSP430F5137_Go_To_Sleep(){
 Device_MSP430F5137_InitClock(0x3d09);
 LED_CLR(1);
 _BIS_SR(LPM3_bits + GIE);
}

void set_Cycles(volatile unsigned int seconds){
 cycles = seconds;
}

void delay(volatile unsigned int i) { // volatile prevents unused loop to be pruned by optimizations
 while (i--);
}

void main(void) {
 _BIC_SR(GIE);
 WDTCTL = WDTPW + WDTHOLD;
 Device_MSP430F5137_InitClock(0x2EE0);
/**
some led init stuff
**/
 _BIS_SR(GIE);
 _BIS_SR(LPM3_bits + GIE);
 for(;;){
set_Cycles(20);
  Device_MSP430F5137_Go_To_Sleep();
 }

}

#pragma vector=TIMER0_A0_VECTOR
__interrupt void TimerA0_interrupt(void){
 interruptCounter++;
 if(interruptCounter==cycles){
  interruptCounter=0;
  _BIC_SR(LPM3_EXIT);
  LED_SET(1);
 }
}

So jetzt zum Problem ;)
Wenn ich das ganze über mein launchpad debugge funktioniert es wie 
gewollt.
Wenn ich den mController aber mit externer stromversorgung(usb ohne 
debug) starte übernimmt er die set_Cycles(20); nicht.

hat jemand eine Idee woran das liegen könnte?
Liebe Grüße
Lukas

[Rufus Τ. F. (rufus)]

lukas s. schrieb:
> übernimmt er die set_Cycles(20); nicht

Wie stellst Du das fest?

[lukas s. (lukas_s46)]

naja, wenn ich ihn mit dem debugger starte sieht der programmablauf etwa 
so aus:
~10 sekunden lang passiert nichts,
das led blinkt kurz,
~1,5 minuten lang passiert nichts,
das led blinkt kurz,
~1,5 minuten lang passiert nichts,
das led blinkt kurz,
...

wenn ich ihn per usb strom starte sieht das ganze dann so aus:
~10 sekunden lang passiert nichts,
das led blinkt kurz,
~10 sekunden lang passiert nichts,
das led blinkt kurz,
~10 sekunden lang passiert nichts,
das led blinkt kurz,
....

Liebe Grüße
Lukas

[Rufus Τ. F. (rufus)]

Wie sieht Deine Hardware aus?


Ansonsten:

  _BIS_SR(GIE);
  _BIS_SR(LPM3_bits + GIE);  // <--
  for(;;)
  {
    set_Cycles(20);
    Device_MSP430F5137_Go_To_Sleep();

Damit schickst Du Deinen MSP430 schon schlafen, bevor Du _Go_To_Sleep 
aufrufst.

[lukas s. (lukas_s46)]

ich benutze den 
MSP430-CCRFLCD(http://www.mcmelectronics.com/product/OLIMEX-MSP430-CCRFLCD-/83-14825) 
von Olimex.
 Wenn ich die Zeile

_BIS_SR(LPM3_bits + GIE);

auskommentiere habe ich wieder unstetiges Verhalten.
Per Launchpad funktioniert es - ohne geht er nur noch sehr kurz in lpm3 
und wacht sofort wieder auf...

Danke für dich Hilfe!,
Lukas

[Rufus Τ. F. (rufus)]

Womit erzeugst Du den Takt für das System?

Irgendwie fehlt mir in Deiner main die Initialisierung der 
Takterzeugung.

[Horst (Gast)]

Per default ist der DCO aktiv mit knapp nem MHz.

[Horst (Gast)]

Ach...seh grad...ACLK. Haste da was dran?

[LPM (Gast)]

lukas s. schrieb:
> void main(void) {
>  _BIC_SR(GIE);
>  WDTCTL = WDTPW + WDTHOLD;
_BIC... ist nicht nötig. Nach Reset ist das Bit gelöscht.

lukas s. schrieb:
> _BIS_SR(GIE);
>  _BIS_SR(LPM3_bits + GIE);
_BIS_SR(LPM3... wurde schon gesagt. Ist zuviel, einfach löschen.

und das Ergebnis passt doch:
lukas s. schrieb:
> wenn ich ihn per usb strom starte sieht das ganze dann so aus:
> ~10 sekunden lang passiert nichts,
> das led blinkt kurz,
> ~10 sekunden lang passiert nichts,
> das led blinkt kurz,
> ~10 sekunden lang passiert nichts,
> das led blinkt kurz,
> ....

Das mit dem Launchpad kommt mir eher spanisch vor.

[lukas s. (lukas_s46)]

Danke für die Antworten,

>  _BIS_SR(LPM3_bits + GIE);
_BIS_SR(LPM3... wurde schon gesagt. Ist zuviel, einfach löschen.

Ich habe nicht das ganze Script hier hoch geladen, weil es dann zu 
unübersichtlich geworden wäre, aber es macht schon Sinn, weil ich möchte 
das das System nach dem Start erst mal wieder ein schläft.

Das hier war aber tatsächlich 2 mal zu viel ;)
>  _BIC_SR(GIE);

>  Das mit dem Launchpad kommt mir eher spanisch vor.
Ich konnte den Fehler jetzt lokalisieren und es ist wohl so, dass

volatile unsigned int cycles = 2;

beim eintreten in lpm3 resettet/auf null gesetzt wird oder was auch 
immer - das konnte ich noch nicht genau feststellen. (in der debug 
session passiert das wohl nicht)

Benutze ich stattdessen lpm0 funktioniert es wie gewünscht.

Wo speichert man solche werte gewöhnlich? Ist es sinnvoll den Wert in 
der Interruptroutine den Flashspeicher zu schreiben/lesen?

Liebe Grüße
Lukas

[Oscar K. (sieges)]

Horst schrieb:
> Ach...seh grad...ACLK. Haste da was dran?
Das ist die entscheidende Frage und vor allem wie dein Init dazu 
aussieht.
cheers

[LPM (Gast)]

lukas s. schrieb:
> Ich habe nicht das ganze Script hier hoch geladen

Dann kann man dir nicht helfen. :-(((
Der Fehler steckt meistens in den Zeilen, dien nicht gepostet werden.

lukas s. schrieb:
> Ich konnte den Fehler jetzt lokalisieren und es ist wohl so,
> dassvolatile unsigned int cycles = 2;
> beim eintreten in lpm3 resettet/auf null gesetzt wird oder was auch
> immer - das konnte ich noch nicht genau feststellen. (in der debug
> session passiert das wohl nicht)

oder eine übersinnliche Erscheinung. ;-D

lukas s. schrieb:
> Benutze ich stattdessen lpm0 funktioniert es wie gewünscht.

Und wo unterscheiden sich die beiden LPMs? Achte einmal auf die 
Takterzeuger.