MSP430 ADC misst Mist

[Colin M. (colin80)]

Hallo zusammen,

ich hab gerad ein ganz merkwürdiges ADC-Interrupt Verhalten.
Ich habe einen digitalen Filter im Interrupt am laufen. Bis zu einem 
bestimmten Zeitraum nimmt der msp430f1611 verschiedene Werte auf, die er 
als Array mit 32 Werten speichert. Irgendwann fängt dann der ADC12MEM an 
zu spinnen und misst nur noch dieselben Werte. Gibt es beim ADC auch so 
etwas wie ein Überlauf? Was läuft hier bloß falsch?

void INIT_ADC12()
{
  ADC12CTL0 = ADC12ON | MSC | REFON | REF2_5V | SHT0_15;           // Turn on ADC12, extend sampling time
                                            // to avoid overflow of results
  ADC12CTL1 = SHP + CONSEQ_3;                 // Use sampling timer, repeated sequence
  ADC12MCTL0 = INCH_0 + SREF_1;                      // ref+=AVcc, channel = A0
  ADC12MCTL1 = INCH_1 + SREF_1;                      // ref+=AVcc, channel = A1
  ADC12MCTL2 = INCH_2 + SREF_1;                      // ref+=AVcc, channel = A2
  ADC12MCTL3 = INCH_3 + SREF_1 + EOS;                  // ref+=AVcc, channel = A3, end seq.
  ADC12IE = 0x08;                           // Enable ADC12IFG.3
  ADC12CTL0 |= ENC;                         // Enable conversions
  ADC12CTL0 |= ADC12SC;                     // Start conversion  
}

#pragma vector=TIMERB0_VECTOR
__interrupt void Timer_B (void)
{ 
  P5OUT ^= BIT1;                // Toggle P1.0 
  TBCTL ^= MC_1;                //Stoppe Timer B
  TACTL ^= MC_1;                //Stoppe Timer A
  time= count + TAR ;                  //Hier Zeit aus Timer A rausholen

  ADC12CTL0 ^= ADC12SC;
  ADC12CTL0 ^= ENC;                                                     // Enable conversions
  ADC12IE = 0x08;                                                       // Enable ADC12IFG.3*/

}

#pragma vector=ADC12_VECTOR
__interrupt void ADC12ISR (void)
{
  A0results = ADC12MEM1;           
  mittelwert_filter1(A0results);                  
}


int mittelwert_filter1(int newval)
{
   if (mm < messfenster && newval > 0)                            //Mit dieser if Schleife wird die Fenstrgröße festgelegt
                                                  //hier sollte eine 2er potenz für die Größe gewählt werden.
   {
      avgsum1 [mm]= newval;                       //Wert n-mal aufaddieren
      avg += avgsum1[mm];
      mm++;                                       //n um eins erhöhen
   }
   else if( mm == messfenster && newval > 0)
   {
     avg = avg  - avgsum1[0];
     avg += newval;
     for(aa=1; aa<messfenster; aa++)
     {
        avgsum1[aa-1] = avgsum1[aa];
        avgsum1[messfenster-1] = newval;
     }
   }
   mittelwert = avg/ mm;
}

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

- Was soll Timer B steuern?
- Ist der ADC immer noch Busy (ADC12CTL1) wenn dieselben Werte gemessen 
werden?
- Ist "Bis zu einem bestimmten Zeitraum" immer gleich lang oder 
variabel?

[colin80 (Gast)]

- Der Timer B misst die Inkremente eines Drehmomentsensors. Solange soll 
auch der ADC gemessen werden.

- Der ADC ist noch Busy. Und das verhalten bleibt auch erhalten, wenn 
ADC ausgeschaltet wird und beim nächsten Messverfahren wieder 
eingeschaltet wird. Heißt, nach dem zweiten Messverfahren bleibt 
ADC12MEM1 auf einen einzigen Wert

- Durch dieses Verschieben im digital-filter kann ich das schwer sagen. 
Mir ist gerad eingefallen wie ich das testen kann. Indem ich zähle, wie 
oft der ADC Interrupt auftritt. Wenn ich mich nicht täusche, dann tritt 
es immer an derselben stelle auf. Kann ich morgen erst testen. An 
variabel kann es nicht liegen, weil Mittelwert und avg volatile unsigned 
long Datentypen sind.

Testweise habe ich das Messfenster auf 500 statt 32 vergrößert, um zu 
schauen, wann immer gleiche Werte rauskommen. Hier kann ich nicht so 
genau sagen, wie oft schon verschoben worden ist, aber dann sieht das 
ungefähr so aus:

avgsum1[450]= 36
avgsum1[451]= 40
avgsum1[452]= 25
avgsum1[453]= 32
avgsum1[454]= 41
avgsum1[455]= 35
avgsum1[456]= 35
avgsum1[457]= 35
avgsum1[458]= 35
avgsum1[459]= 35
avgsum1[460]= 35 usw.

[colin (Gast)]

Hat niemand eine Idee? Stoppe ich die conversation überhaupt richtig?

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

Bist du sicher das es am ADC liegt und nicht an der Berechnung von 
avgsum1[]?

Das einzige was mir noch auffällt ist
  ADC12CTL0 ^= ADC12SC;
  ADC12CTL0 ^= ENC;
Normalerweise setzt man die beim einschalten immer zusammen. ("ADC12SC 
and ENC may be set together with one instruction")

[source reader (Gast)]

colin schrieb:
> Hat niemand eine Idee?

Du zeigst uns nur einen Teil der Quelle. Soll das ein Ratespiel werden?

Beschreibe einmal was du überhaupt machen willst, wie dein gedachter 
Programmfluss ist und hänge das vollständige Programm an.

Du bekommst dann Tipps zur Optimierung der Problemlösung und Tipps zur 
Programmierung.

Die Profis hier helfen gerne, machen aber nicht deinen Job. ;-P