ADC auslesen

[Markus S. (knutschkuh)]

Hallo zusammen,

ich habe mir die ADC Ansteuerung im Wiki angeschaut und dies auch so 
Programmiert.
Nun msus ich ja nach dem Starten einer Messung warten bis diese 
durchgeführt wurde.

 ADCSRA |= (1<<ADSC);            // eine Wandlung "single conversion"
  while (ADCSRA & (1<<ADSC) ) {   // auf Abschluss der Konvertierung warten
  }
  return ADCW;                    // ADC auslesen und zurückgeben

Jedoch bleibt er dann bei mir immer im Programm hängen. Ändere ich es ab 
in

 ADCSRA |= (1<<ADSC);            // eine Wandlung "single conversion"
  while (!(ADCSRA & (1<<ADSC) )) {   // auf Abschluss der Konvertierung warten
  }
  return ADCW;                    // ADC auslesen und zurückgeben

funktioniert es wunderbar. Ist hier ein Fehler meinerseits oder ein 
Fehler im Wiki?? Ich steh da gerade im Wald...

Gruß Markus

[Thomas E. (thomase)]

Markus S. schrieb:
> Ich steh da gerade im Wald...
Genauso wie dein ADC. Der läuft gar nicht.
Wie wird der denn initialisiert?

mfg.

[Cyblord -. (cyblord)]

Welcher Controller? Bei mir sind die beiden ADC Result Bytes im Makro 
"ADC" zu finden und nicht "ADCW".

[Thomas E. (thomase)]

cyblord ---- schrieb:
> "ADC" zu finden und nicht "ADCW".
Doch. Das passt schon, das ist beides definiert.

mfg.

[Markus S. (knutschkuh)]

Also mein Programm sieht so aus:

#include <stdlib.h>
#include <avr/io.h>
#include <avr/version.h>


int main(void)
{
  DDRB = 0xff;                    //Definiere Port B als Ausgang
  DDRA = 0x00;                    //Definiere Port A als Eingang
  PORTB = 0xff;                    //Setze Port B auf 0xff (Alle Ausgänge High = LED´s aus)
  uint8_t ad_channel=0,ad_wert=0,ad_ausgabe=0;    //Variablen definieren
  ADMUX = (1<<ADLAR);                  //Ausgabe des ADC-Wertes Linksbündig!!
  ADCSRA = (1<<ADPS2);                //Teilungsfaktor 32
  ADCSRA |= (1<<ADEN);                //ADC Aktivieren
  ADCSRA |= (ADMUX & ~(0x1f)) | (ad_channel & 0x1f);  //Setze ADC Kanal (Auswahl durch die Variable "ad_channel")
  
  ADCSRA |= (1<<ADSC);                //Eine Messung starten
  while (!(ADCSRA & (1<<ADSC)))            //Endlosschleife bis ADC Messung fertig
    {
    }
  uint8_t temp = ADCH;                //Lesen des ADC Wertes und verwerfen desselben.
  
    while(1)
    {
    ADCSRA |= (ADMUX & ~(0x1f)) | (ad_channel & 0x1f);  //Kanalauswahl
    ADCSRA|=(1<<ADSC);                //Messung starten
    while(!(ADCSRA & (1<<ADSC))){}          //Endlosschleife bis ADC Messung fertig
  //  uint8_t temp3 = ADCL;              //Schreiben der niedristen Bits in Temporäre Variable (Wird verworfen)
    ad_wert = ADCH;                  //Schreiben der höchsten 8 Bits in die Variable "ad_wert" zur weiterverarbeitung
    
    //Hier wird geprüft wie Hoch der ADC Wert ist und dazu passend die Ausgabe für Port B vorbereitet.
    if(0 <= ad_wert && ad_wert < 2){ad_ausgabe=0;}
    if(2 <= ad_wert && ad_wert < 40){ad_ausgabe=1;}
    if(40 <= ad_wert && ad_wert < 70){ad_ausgabe=3;}
    if(70 <= ad_wert && ad_wert < 100){ad_ausgabe=7;}
    if(100 <= ad_wert && ad_wert < 130){ad_ausgabe=15;}
    if(118 <= ad_wert && ad_wert < 160){ad_ausgabe=31;}
    if(160 <= ad_wert && ad_wert < 193){ad_ausgabe=63;}
    if(193 <= ad_wert && ad_wert < 223){ad_ausgabe=127;}
    if(223 <= ad_wert && ad_wert <= 255){ad_ausgabe=255;}
    PORTB = ad_ausgabe;              //Hier erfolgt die negierte (da active LOW) ausgabe für die Balkenanzeige.
        }
}

Ich lese nur ADCH in meinem Programm aus da ich Linksbündig ausgebe auf 
8 bit.  Ich nutze einen ATmega16, laut Datenblatt reicht es ADCH 
auszulesen wenn ich nur 8 bit nutze. Oder ist hier mein Denkfehler?

[Cyblord -. (cyblord)]

Thomas Eckmann schrieb:
> cyblord ---- schrieb:
>> "ADC" zu finden und nicht "ADCW".
> Doch. Das passt schon, das ist beides definiert.
>
> mfg.

Ahso sieh an. Man lernt nie aus. Danke.

[Thomas E. (thomase)]

Markus S. schrieb:
> ADCSRA |= (ADMUX & ~(0x1f)) | (ad_channel & 0x1f);
Welchen Sinn hat das, daß du das ADCSRA-Register mit ADMUX verknüpfst?
Damit sorgst du zufällig dafür, daß du den ADC auf Free Running stellst 
und er ständig wandelt und das ADSC-Bit immer gesetzt ist.
Wo ist deine Referenzspannung?

mfg.

[Markus S. (knutschkuh)]

Ähm ok da soll natürlich ADMUX stehen. Referenzspannung ist AREF daher 
sind die Bits dafür 0.
Danke!!

[Thomas E. (thomase)]

Markus S. schrieb:
> Ähm ok da soll natürlich ADMUX stehen.
Dann sollte das auch laufen.

> ADCSRA |= (ADMUX & ~(0x1f)) | (ad_channel & 0x1f);
Und schreib das nicht so kompliziert. Da muß man ja mindestens dreimal 
hingucken, bis man sieht, was das soll.
ADMUX &= ~0x1F;
ADMUX |= ad_channel;
Nur weil es jetzt 2 Zeilen sind, wird das im Code nicht länger. Der 
Compiler kriegt das schon richtig hin.

mfg.