MSP430 adc12 auslesen problem

[TM (Gast)]

Hallo,

Ich bin grad dabei den MSP430f169 zu programmieren leider krieg ich 
meine ADC Funktionen nicht vernünftig hin und ich sehe grad echt nicht 
den Fehler.
Ich kann von einem Channel die Daten lesen wenn ich allerdings von zwei 
channels lesen will bekomme ich anscheinend trotzdem Daten von dem 
selben Channel wie vorher.

Hier der wesentliche Codeausschnitt:

void main(void)
{
    int adc_value=0;
   char buffer[5];
    STOP_WATCHDOG;
   initClock();
   initUSART0();
   initADC();
   while(1)
   {
         adc_value = readADC(8);
        itoa(buffer,adc_value,10);
         sendString(buffer);
       sendString(";");
       adc_value = readADC(0);
       itoa(buffer,adc_value,10);
       sendString(buffer);
       sendString(";\n");
   }

}
void initADC(void)
{
 P6SEL |= 0xFF; // select the hole port as ADC
 ADC12CTL0 |= ADC12ON+SHT00+SHT01+SHT02; // Set sample and hold time
 ADC12CTL1 |= ADC12SSEL1+SHP;// Select MCLK, enable sample and hold timer
}
unsigned int readADC(unsigned int channel)
{
 ADC12MCTL0 = channel; // select channel for mem0
 ADC12CTL0 |= ADC12SC + ENC; // Start sampling 
 ADC12CTL0 &= ~ADC12SC;     // End sampling start conversion
 while ((ADC12CTL1 & ADC12BUSY) == 1);// while ADC busy
 return ADC12MEM0;
}

An AVcc sind 3.3V angeschlossen an AVss GND.
MCLK läuft mit 8 MHz.

Am ADC0 ist ein Poti, welches einzeln ausgelesen kann erst wenn ich 
ADC10 für interne VCC messung hinzufüge bekomme ich auch für ADC0 immer 
die selben Werte wie für ADC10.

Weiß jemand was hier falsch läuft ?

Gruß TM

[Sepp (Gast)]

Hallo,
ich würde vermuten der Fehler liegt hier:

ADC12CTL0 |= ADC12SC + ENC; // Start sampling 
ADC12CTL0 &= ~ADC12SC;     // End sampling start conversion

Du startest das ADC Sampling um es im nächsten Schritt wieder zu 
beenden? Ich würde "ADC12CTL0 &= ~ADC12SC;" im ADC ISR verwenden nachdem 
die Konvertierung abgeschlossen ist.
Auf jeden Fall mal nach

while ((ADC12CTL1 & ADC12BUSY) == 1);// while ADC busy

 setzen.
Viele Grüße,
Sepp

[Nico (nico123)]

Du kannst das Register ADC12MCTL0 nur verändern wenn ENC = 0 ist!

[TM (Gast)]

Nico ... schrieb:
> Du kannst das Register ADC12MCTL0 nur verändern wenn ENC = 0 ist!

Danke das wars!

Sepp schrieb:
> Hallo,
> ich würde vermuten der Fehler liegt hier:ADC12CTL0 |= ADC12SC + ENC; // Start 
sampling
> ADC12CTL0 &= ~ADC12SC;     // End sampling start conversion
> Du startest das ADC Sampling um es im nächsten Schritt wieder zu
> beenden? Ich würde "ADC12CTL0 &= ~ADC12SC;" im ADC ISR verwenden nachdem
> die Konvertierung abgeschlossen ist.
> Auf jeden Fall mal nachwhile ((ADC12CTL1 & ADC12BUSY) == 1);// while ADC busy 
setzen.
> Viele Grüße,
> Sepp

Das funktioniert so. Aber du hast schon recht über interrupt ist die 
bessere Lösung werde es darauf mal anpassen.

Gruß TM

[Sepp (Gast)]

Hallo,
das ist ja interessant dass es so funktioniert. Allerdings hatte ich ich 
bisher nur ADC10 anstelle des ADC12. Gut zu wissen danke.
Grüße,
Sepp