ADC Rutine vom GCC-Tutorial

@ Richard B. (rbrose)

>es geht um diese Methode vom GCC-Tutorial:

Das ist eine einfache Funktion. Wie sind hier bei C, nicht C++.

>Es wir in der Methode der ADC immer komplett neu initialisiert, bei
>jeder Messung.

Ja.

>Ist das sinvoll?

Nein.

> wird das nicht lagsam?

Ja.

> Ich messe jede 10ms einen Wert
>... ist es nicht besser wenn man den ADC in der main Einstellt und dann
>in der  ReadChannel Methode den ADC nur ein und ausmacht?

Ja.

MfG
Falk

Ok, ich benutze den AtMEga168 und habe ADC so eingestellt:

1

  //init ADC

2

  ADCSRA |= (1<<ADEN) | (1<<ADSC) | (1<<ADPS2) | (1<<ADPS1);    // Frequenzvorteiler 

3

                               // setzen auf 64 (1) und ADC aktivieren (1)   //Free Running Mode     

4

    ADMUX |= (1<<REFS0) | (1<<ADLAR); // AVCC als Referenz //ADC0

Aber wenn ich mit

1

  char buffer[7];

2

  itoa(ADCH, buffer,10);

3

  uart_puts(buffer);

ADCH jede Sekunde ausgeben ist die Variable immer 255 groß, egal welcher 
Wert an PC0 anliegt.

Was ist falsch? Habe ich etwas übersehen? Mit ADSC sollte es jetzt im 
Free Running Mode permanent messen oder?

MMhh der ADC scheint zu funktionieren ... weil wenn ich die Taste Drücke 
und dann der AVR anmachen, dann zeigt er mir ein anderen Wert an .. aber 
alle folgende sind gleich. Das heißt der Free Running Mode klappt nicht 
... Ich dachte er Updated das ADCH automatisch.

Beim ATTiny26 muss man das ADFR Bit aktivieren ... aber der AtMega168 
hat es nicht. Das wird im Register ADCSRB gemacht. Sehe ich das Richtig? 
Aber om Register ADCSRB ist Free Running Mode default ausgewählt, also 
brauche ich da nichts ändern.

Was ist falsch?

@ Richard B. (rbrose)

>ADCH jede Sekunde ausgeben ist die Variable immer 255 groß, egal welcher
>Wert an PC0 anliegt.

Lies das Tutorial. GENAU!

>Was ist falsch? Habe ich etwas übersehen?

Ja. Du musst ADCW lesen, das ist der ADC als 16 Bit Wert. Denn der 
Zugriff auf ADCL ADCH muss VOLLSTÄNDIG in der RICHIGEN Reihenfolge 
erfolgen. Sonst liest du nämlich immer das gleiche.

> Mit ADSC sollte es jetzt im Free Running Mode permanent messen oder?

Ja.

MFG
Falk

@Falk

Wieso muss ich ADCW lesen? Ich habe doch ADLAR aktiviert.
Und Peter Dannegger liesst im seinem Beispiel auch nur ADCH aus:

http://www.mikrocontroller.net/attachment/highlight/21646

Lese jetzt ADCW aus ... ist aber nicht viel besser ... jetzt bekomme ich 
-64^^

@ Richard B. (rbrose)

>Wieso muss ich ADCW lesen?

Warum liest du nicht des Tutorial GENAU! Dann würdes du DAS hier finden.

"
Wenn eine Umwandlung abgeschlossen ist, befindet sich der gemessene Wert 
in diesen beiden Registern. Von ADCH werden nur die beiden 
niederwertigsten Bits verwendet. Es müssen immer beide Register 
ausgelesen werden, und zwar immer in der Reihenfolge: ADCL, ADCH. Der 
effektive Messwert ergibt sich dann zu:

x = ADCL;       // mit uint16_t x
x += (ADCH<<8); // in zwei Zeilen (LSB/MSB-Reihenfolge und
                // C-Operatorpriorität sichergestellt)

oder

x = ADCW; // je nach AVR auch x = ADC (siehe avr/ioxxx.h)
"

MfG
Falk

Wieso widerspricht ihr euch hier im Forum??


Beitrag "ADC des ATMEGA168"
>Autor: Andreas Kaiser (a-k)
>Datum: 01.03.2008 16:28
>
>ADLAR setzen und nur ADCH verwenden.
>

Dennach kann ich nur ADCH lesen ....

Aber das ich nicht der Fehler ... man musste ADATE aktivieren.
Aber irgendwie will er trotzdem nicht richtig funktinieren.
Ich glaube es liegt am AtMega168 ...

@Falk
Du hast dich geirrt, ich muss nicht ADCL vor ADCH auslesen wenn ADLAR 
gesetzt ist.

Habe jetzt in die Schaltung einen ATMega8 getan ... und da klappt es 
problemlos mit 8bit und 10bit.

Ich glaube der AtMega168 hat ein Bug. Finde irgendwie auch nicht viel 
über den 168 im Free  Running mode.

@Jörg
was meintest du in dem anderen Thread mit

>> ADCSRB = (0<<ADTS2) | (0<<ADTS1) | (1<<ADTS0);   // Free running mode.
>Das stimmt nicht...

Mit diesem Pattern triggert der ADC auf den AC.

DANKE DIR JÖRG!! Es lag an der reihenfolge ;-)
Erst ADMUX und dann ADCSRA.
Jetzt funktioniert es.

ADCSRB = (0<<ADTS2) | (0<<ADTS1) | (1<<ADTS0);   // Free running mode.

Es stimmt so ... 0 0 0 ist für Free running mode. Steht im Datenblatt 
von dem AtMega168.

Da steht aber nicht 0 0 0, da steht 0 0 1.

Aber wieso wird dann im dem AVR Tutorial erst ADCSRA beschaltet und dann 
ADMUX ?
Man sollte es mal überarbeiten. Ist ja ein sehr schlechtes Beispiel. 
Nicht allgemein gültig!

@Andreas?
Bin ich blind? Oder habe ich ein anderes Datenblatt?

Table 23-5. ADC Auto Trigger Source Selections
ADTS2 ADTS1 ADTS0 Trigger Source
0 0 0 Free Running mode
0 0 1 Analog Comparator
0 1 0 External Interrupt Request 0
0 1 1 Timer/Counter0 Compare Match A
1 0 0 Timer/Counter0 Overflow
1 0 1 Timer/Counter1 Compare Match B
1 1 0 Timer/Counter1 Overflow
1 1 1 Timer/Counter1 Capture Event


0 0 1 wäre der Analog Comparator.

Putz die Brille, wisch dir die Augen und lass mal jemand aus deiner 
Umgebung drauf gucken. Vielleicht siehst du dann, dass da (1<<ADTS0) 
steht, nicht etwa (0<<ADTS0).

Alles klar ;-) Im Code ist es falsch .. nicht im Datenblatt. Ich merke 
es jetzt, sitze schon zu lange vor dem PC :-) Sehe den Wald vor lauter 
Bäume nicht.