Probleme beim Analogeingang einlesen Atmega32

[stephan b. (stu)]

Hallo nochmal,
ich bräuchte nochmal hilfe bei meinem Projekt.
Ich möchte mit einem Atmega32 an einem Spannungsteiler eine Spannung 
0-5V messen. Eigentlich sollte das ganze an einem LCD-Diplay angezeigt 
werden aber mir reicht erstmal irgendeine anzeige. Leider bekomme ich es 
nicht mal hin, mir den umgewandelten Wert an einer LED Reihe anzeigen zu 
lassen.

#include <avr/io.h>
#include <avr/lcd-pollin-jumper.h> //Bitte lib in ordner kopieren!
#define F_CPU 16000000UL
#include <util/delay.h> //Standardverzögerungs Lib
#define F_CPU 16000000UL

uint16_t result;

void ADC_init(void)
{


ADMUX = (1<<REFS1) | (1<<REFS1); //int. Referenzspannung
ADCSRA = (1<<ADPS1) | (1<<ADPS0); //Vorteiler setzen
ADCSRA |= (1<<ADEN); //ADC aktivieren

ADCSRA |= (1<<ADSC); //einmal auslösen
while(ADCSRA & (1<<ADSC)){} //warte bis fertig
result = ADCW; //Ergebnis
}

uint16_t ADC_Read (uint8_t channel)
{
ADMUX = (ADMUX & ~(0x1f)) | (channel & 0x1f);
ADCSRA |= (1<<ADSC); 
while (ADCSRA & (1<<ADSC)){} //warte bis fertig
return ADCW; //ADC ausgeben und zurückgeben
}

int main(void)
{
DDRA = 0b00000000;
DDRB = 0b11111111;
DDRC = 0b11111111;
DDRD = 0b11111111;

while(1)  
{
  ADC_init;
        PORTD = ADC_Read (0);

}
}

ich habe es auch schon mit dieser version probiert:

while(1)  
{
  ADC_init;
        ADC_Read (0);
        PORTD = result;
}

oder mit dieser:

while(1)  
{
  ADC_init;
        ADC_Read (0);
        PORTD = ADCW;
}

aber egal wie groß die Spannung am Eingang ist, die LEDS an Port D 
bleiben dunkel.

[Frank L. (franklink)]

Hallo Stefan,
schau Dir mal im Datenblatt vom Mega32 die Seite 214 die Tabelle "ADC 
Multiplexing Selection Register - ADMUX" an und vergleiche mit Deinen 
Daten, ich denke, dann fällt Dir Dein Fehler direkt auf.

Ein Tipp, der Fehler liegt in dieser Zeile:

ADMUX = (ADMUX & ~(0x1f)) | (channel & 0x1f);

Gruß
Frank

[stephan b. (stu)]

Mein Fehler lag in dieser Zeile:

ADMUX = (1<<REFS1) | (1<<REFS1); //int. Referenzspannung

es müsste aber:

ADMUX = (1<<REFS1) | (1<<REFS0); //int. Referenzspannung

heißen.
Die Referenzspannung ist zu klein das habe ich mittlerweile auch 
gemerkt. Aber das bekomme ich durch änderung des Spannungsteilers in den 
Griff. Danke trotzdem für die Mühe

[Frank L. (franklink)]

Hallo Stefan,
stimmt und ich hatte gerade ein boolsches Brett vor dem Kopf...

Gruß
Frank

[Karl H. (kbuchegg)]

Mag alles sein. Allerdings ist das alles nicht der Hauptfehler.

Der Hauptfehler ist, dass hier

while(1)  
{
  ADC_init;
        PORTD = ADC_Read (0);

}

kein Funktionsaufruf der Funktion ADC_init steht. C ist nicht Pascal. 
Die Funktionsaufrufklammern fehlen.

Und btw. Es macht nicht viel Sinn, innerhalb der Schleife den ADC 
ständig neu zu initialisieren. Initialisierungen macht man nur einmal:

  ...

  ADC_init();

  while(1)  
  {
    PORTD = ADC_Read (0);
  }
}