Atmega88 ADC auf Display ausgeben

[Christian (Gast)]

Hallo zusammen,

ich habe mir ein kleines Programm geschrieben, bei dem ich zwei AD-Werte 
auslese und auf einen LCD ausgeben will. Es funktioniert alles, bis auf 
die aktualisierung der AD-Werte. Wenn ich Spannung an die Schaltung lege 
wird am Display der aktuelle AD-Wert angezeigt, drehe ich am Poti des 
AD-Wandlers bleibt der Wert am Display gleich. Nehme ich die Spannung 
weg und klemme sie wieder an, wird wieder der momentan eingestellte 
AD-Wert angezeigt, aber während der Laufzeit wird er nicht mehr 
aktualisiert.

Hier der Code:

#define F_CPU 8000000

#include <avr/io.h>
#include <util/delay.h>
#include <lcd-routines.h>
#include <lcd-routines.c>



uint16_t ADC_C0;
uint16_t ADC_C1;
uint8_t ADCL_C0;
uint8_t ADCH_C0;
uint8_t ADCL_C1;
uint8_t ADCH_C1;
char ADC_StringC0 [16];
char ADC_StringC1 [16];



void ADC_Init (void)
{
  
  ADMUX |= (1<<REFS0);
  ADCSRA |= (1<<ADPS2) | (1<<ADPS1) | (1<<ADEN) | (1<<ADSC);
  
  while (ADCSRA & (1<<ADSC));
  ADCL_C0 = ADCL;
  ADCH_C0 = ADCH;
  
}


void ADC_Read (void)
{
  
  ADMUX &= (0<<MUX0);
  
  while (ADCSRA & (1<<ADSC)){}
    ADCL_C0 = ADCL;
    ADCH_C0 = ADCH;
  
  ADMUX |= (1<<MUX0);  
  while (ADCSRA & (1<<ADSC)){}
    ADCL_C1 = ADCL;
    ADCH_C1 = ADCH;
  
  ADC_C0 = ADCL_C0 + (ADCH_C0 * 256);
  ADC_C1 = ADCL_C1 + (ADCH_C1 * 256);
  
}



void LCD_Write (void)
{
  lcd_clear();
  
  itoa (ADC_C0, ADC_StringC0, 10);
  itoa (ADC_C1, ADC_StringC1, 10);
  
  lcd_setcursor(0,1);
  lcd_string("ADC0      ADC1");
  lcd_setcursor(0,2);
  lcd_string(ADC_StringC0);
  lcd_setcursor(9,2);
  lcd_string(ADC_StringC1);
    
  
}





int main(void)
{
  ADC_Init();
  lcd_init();
  
  
  
  
  
    while(1)
    {
        ADC_Read();
  LCD_Write();
  _delay_ms(100)
    }
}

[chris (Gast)]

void ADC_Read (void)
{
  
  ADMUX &= (0<<MUX0);
  
  while (ADCSRA & (1<<ADSC)){}
    ADCL_C0 = ADCL;
    ADCH_C0 = ADCH;
  
  ADMUX |= (1<<MUX0);  
  while (ADCSRA & (1<<ADSC)){}
    ADCL_C1 = ADCL;
    ADCH_C1 = ADCH;
  
  ADC_C0 = ADCL_C0 + (ADCH_C0 * 256);
  ADC_C1 = ADCL_C1 + (ADCH_C1 * 256);
  
}

In dieser Funktion liegt das Problem (eigentlich mehrere Probleme).

ADMUX &= (0<<MUX0);

Diese Zeile löscht ADMUX komplett.
Entspricht also
ADMUX = 0x00;

Das willst du aber sicherlich nicht

Bits löschen geht so:
ADMUX &= ~(1<<MUX0);


Nächstes Problem:
while (ADCSRA & (1<<ADSC)){}

du wartest hier zwar brav, bis die Messung beendet ist, hast aber 
überhaupt keine Messung gestartet!


teste mal die veränderte Funktion:

void ADC_Read (void)
{
  
  ADMUX &= ~(1<<MUX0);
  
  ADCSRA |= (1<<ADSC);   // start conversion
  while (ADCSRA & (1<<ADSC)){}
    ADCL_C0 = ADCL;
    ADCH_C0 = ADCH;
  
  ADMUX |= (1<<MUX0);  
  ADCSRA |= (1<<ADSC);   // start conversion
  while (ADCSRA & (1<<ADSC)){}
    ADCL_C1 = ADCL;
    ADCH_C1 = ADCH;
  
  ADC_C0 = ADCL_C0 + (ADCH_C0 * 256);
  ADC_C1 = ADCL_C1 + (ADCH_C1 * 256);
  
}

[Christian (Gast)]

Erstmal Danke für deine schnelle Antwort.
Das mit der 0 hatte ich nicht gewusst. Ich dachte das wäre rechnerisch 
das selbe.
Zum anderen mit der Messung: Ich habe auf Free Running modus gestellt, 
sollte da nicht automatisch eine neue Messung starten?

[Christian (Gast)]

Bitte nochmal um Antwort

[spess53 (Gast)]

Hi

>Zum anderen mit der Messung: Ich habe auf Free Running modus gestellt,
>sollte da nicht automatisch eine neue Messung starten?

Wo hast du das eingestellt? Ich sehe keine Stelle, an der ADATE gesetzt 
wird.

MfG Spess

[spess53 (Gast)]

Hi

Nachtrag:

>  while (ADCSRA & (1<<ADSC))

Wird dich im Free Running Mode nicht weiter bringen, da ADSC dort 
ständig High ist. Dort musst du ADIF abfragen und das auch wieder per 
Hand löschen. Oder du nutzt den ADC Complete Interrupt.

MfG Spess

[Christian (Gast)]

Ich habe mir das Datenblatt mal angesehen und habe weder ADATE noch ADIF 
gefunden. Ich habe hoffentlich nich das falsche Datenblatt.

[Karl H. (kbuchegg)]

Ist auch egal.
Solange du 2 Eingänge abfragen willst, bringt dich der free-Running 
Modus sowieso nicht weiter.

Du hast nur EINEN ADC!
Den kannst du auf verschiedene Eingänge schalten. Aber die eigentliche 
ADC Hardware gibt es nur ein eiziges mal. D.h. du willst die volle 
Kontroller darüber haben, welchen Kanal der ADC gerade bearbeitet und 
auch wann er das tut.
In dem Moment, in dem du Free Running Modus und "warten aufs Ergebnis" 
im selben Programm benutzt, machst du etwas grundsätzliches falsch.

[Christian (Gast)]

Das mit der Hardware ist mir bewusst

[Christian (Gast)]

Welchen modus sollte ich demnach verwenden?

[spess53 (Gast)]

Hi

>Ich habe mir das Datenblatt mal angesehen und habe weder ADATE noch ADIF
>gefunden.

ADATE: Bit5 in  ADCSRA
ADIF:  Bit4 in  ADCSRA

>Ich habe hoffentlich nich das falsche Datenblatt.

http://www.atmel.com/Images/doc2545.pdf

MfG Spess

[Karl H. (kbuchegg)]

> Das mit der Hardware ist mir bewusst

Scheint nicht so zu sein.
Denn sonst wüsstest du, dass du die MUX Bits nicht einfach so ändern 
kannst. Eine Änderung der MUX Bits wird nämlich erst dann wirksam, wenn 
der ADC den laufenden Zyklus beendet hat. Sobald du aber auf das Beenden 
des Zykluses wartest um dir das Ergebnis zu holen, startet der ADC schon 
wieder neu während das gerade ermittelte Ergebnis ausgelesen wird. D.h. 
deine danach erfolgende Änderung der MUX bits wirkt sich dann erst auf 
das übernächste Messergebnis aus.
Free Running Modus mit Kanalwechsel ist tricky.

Wie gesagt: benutze ihn konventionell

  MUX Bits einstellen
  ADC Starten
  warten auf Fertigstellung
  Wert holen

und du hast weniger Probleme. Zeitmässig schenkt sich das sowieso 
nichts. Das Programm wird dadurch auch nicht langsamer. Dafür aber 
zuverlässiger :-)

[Karl H. (kbuchegg)]

Karl Heinz schrieb:

> nichts. Das Programm wird dadurch auch nicht langsamer.

Vor allen Dingen dann nicht, solange ein
>   _delay_ms(100)
mit im Programm ist.

[Christian (Gast)]

Der Free running modus ist aber standardmäßig eingestellt. ADTS2 -ADTS0 
= 0.

[Karl H. (kbuchegg)]

Christian schrieb:
> Der Free running modus ist aber standardmäßig eingestellt.

Das wär uns allen aber neu.

Solange ADATE in ACSRA auf 0 bleibt, is nix mit Auto Trigger. Und ohne 
Auto Trigger kein Free Running.

Datenblatt, Mega88 Seite 256 und folgende.

[spess53 (Gast)]

Hi

>Der Free running modus ist aber standardmäßig eingestellt. ADTS2 -ADTS0
>= 0.

Läuft aber nur, wenn ADATE ( AD Auto Trigger Enable) gesetzt ist.

MfG Spess

[Christian (Gast)]

Aha jetzt hab ichs kappiert. Vielen Dank für eure Hilfe.