ATMega32 ADC hängt

[Robert S. (socketa)]

Hallo,
Beim Umstieg vom Easy-Mode Arduino auf avr-gcc bin ich auf ein Problem 
beim ADC lesen gestossen. Es scheint als wueder der ADC haengen bleiben.

Die erste Wandlung nach einem Reset ist korrekt, aber alle weiteren 
geben immer den selben Wert zurück wenn man den Eingang verändert.
Nach einem Reset wird wieder einmal richtig gewandelt, und dann hängts.

Ist auf einem ATMEGA32 der mit 16MHz betrieben wird. AVCC ist auf VCC 
(+5V), Kondensator von AREF auf GND. Um AVCC als Referenz zu nutzen muss 
laut Datenblatt REFS0 auf 1 sein und REFS1 auf 0.

Mein ADC Code sieht wie folgt aus:

void main(){
    ADCSRA = (1<<ADPS2) | (1<<ADPS1) | (1<<ADPS0) | (1<<ADEN); //prescaler 128 
    ADMUX = (1<<REFS0); //AVCC, channel 0
    _delay_ms(10);      //nicht notwendig 
    while(1){
      ADCSRA |= (1<<ADSC); 
      while(!(ADCSRA & (1<<ADIF)));
      UARTSend(ADCH);
      UARTSend(ADCL);
      ADCSRA |= (1<<ADIF); //Interruptflag loeschen
      _delay_ms(500);      
   }
}

Soweit ich das richtig verstanden habe wird ADIF gesetzt wenn die 
Wandlung abgeschlossen ist bzw. neue Daten in ADCH/L sind. Man loescht 
es wieder indem man es auf 1 setzt. Der ADC sollte dann wieder eine neue 
Wandlung nach dem Setzen von ADSC durchfuehren koennen.

Ich nehme an dass ich etwas grundsaetzlich falsch verstanden habe und 
bin sehr dankbar ueber jeden Hinweis dazu was das wohl sein koennte.

[Spess53 (Gast)]

Hi

Vergiss ADIF und teste, ob ADSC wieder zurückgesetzt wird. Dann ist der 
ADC fertig.

MfG Spess

[Karl H. (kbuchegg)]

>       UARTSend(ADCH);
>      UARTSend(ADCL);

Beim Auslesen von 16-Bit Pseudo-Register:
Erst das Low-Register, dann das High-Register. Die Reihenfolge ist NICHT 
egal.

[Spess53 (Gast)]

Hi

Nachtrag: Beim Lesen erst Das L-Register und danach das H-Register.

MfG Spess

[Cyblord -. (cyblord)]

> When ADCL is read, the ADC Data Register is not updated until ADCH is
> read. Consequently, if
> the result is left adjusted and no more than 8-bit precision is required,
> it is sufficient to read
> ADCH. Otherwise, ADCL must be read first, then ADCH.

Also erst ADCL lesen!

Oder gleich:

uint16_t x=ADC;

Damit liest du direkt in eine 16 Bit Variable und musst dich nicht mehr 
drum kümmern.

[Robert S. (socketa)]

MEINE GUETE, wie peinlich.
Vielen Dank, die falsche Reihenfolge beim Registerlesen wars.
"When ADCL is read the ADC register is not updated until ADCH is read" - 
Laut Doku.