ATmega8 ADC problem

[Rob E. (newgenera2589)]

Hi

Das Problem was ich habe ist folgendes:

Ich will mit einem Atmega8 einen Widerstand ermitteln können und auf 
einem Display angezeigt bekommen.
Die Schaltung ist aufgebaut und "funktioniert" in dem Sinne, dass das 
Display angesteuert und programmiert werden kann.. doch kriege ich 
keinen ADCH Wert auf ein LCD Display ausgegeben.

Hier ist einmal der Quellcode... ich hoffe einer kann mir erklären wo 
der Fehler oder mein Verständnisproblem liegt.

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


void ADC_Init()
{
  
  uint16_t result;
  ADMUX = (0<<REFS0)|(0<<REFS1);                  //Referezspannung auf Extern (bei uns Vcc=5V)
  //free runnung nochmal gucken
  ADCSRA = (1<<ADPS1);          // Frequenzvorteiler auf 50MHz Teiler durch8 !!!!!!!!!!ungefähr 120mhz!!!!!!!!!
  ADCSRA |= (1<<ADEN) | (1<<ADLAR);                // ADC aktivieren
  
  // ADC wandler Warlaufen lasse
  ADCSRA |= (1<<ADSC) | (1<<ADFR);                // eine ADC-Wandlung
  while (ADCSRA & (1<<ADSC) )
  {                          // auf Abschluss der Konvertierung warten
  }
  result =ADCL;
  result |=(ADCH<<8);                    //ADCW muss einmal gelesen werden, sonst wird Ergebnis der nächsten Wandlung nicht übernommen.
}
//-----------------------------------------------ADC Einzelmessung------------------------------------------------------
uint8_t ADC_Read( uint8_t channel )        //
{
  uint16_t result;
  
  // Kanal waehlen, ohne andere Bits zu beeinflußen
  ADMUX = (ADMUX & (~(0x1F))) | (channel & 0x1F);      //!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
  ADCSRA |= (1<<ADSC);            // eine Wandlung "single conversion"
  while (ADCSRA & (1<<ADSC) )
  {        // auf Abschluss der Konvertierung warten
  }
  //result =ADCL;                // ADC auslesen und zurückgeben
  //result |=(ADCH<<8);
  
  return ADCH;
}

int main(void)
{
  int zahler=0;
  uint8_t mess=0;
  uint8_t mess2=0;
  uint16_t test;
  lcd_init();
  lcd_string("Bauteilmessgerat");
  lcd_setcursor( 0, 2 );
  lcd_string("Version 0.1");
  _delay_ms(150);
  
  while(1)
  {
    lcd_clear();                    //Display Löschen
    lcd_setcursor( 0, 1 );                //Springe mit dem Curser in die erste Zeile
    lcd_string("Widerstands-");              //Schreibe auf Display
    lcd_setcursor( 0, 2 );                //Springe mit dem Curser in die zweite Zeile
    lcd_string("messung");  
    _delay_ms(100);


while(1)
{
char messstring[20];
char Buffer[20];
      zahler++;
      lcd_clear();
      lcd_setcursor(0,1);

      itoa( zahler, Buffer, 10 );
      lcd_string( Buffer );
      lcd_setcursor(0,2);
    //  mess= ADC_Read(4);
      mess = ADCH;
    //  mess2 = 4.8876 * mess;
   
   itoa( mess, messstring, 10 );
   lcd_string( messstring );   
  _delay_ms(500);
}


  }
  return 0;
}

[spess53 (Gast)]

Hi

>ADCSRA |= (1<<ADSC) | (1<<ADFR);

Du willst Free Running?

MfG Spess

[chris (Gast)]

Rob El schrieb:
> // Frequenzvorteiler auf 50MHz Teiler
> durch8 !!!!!!!!!!ungefähr 120mhz!!!!!!!!!

dieser Kommentar kann mit der Realität irgendwie garnix zu tun haben...

[Detlef K. (adenin)]

Vielleicht müsste ja mal irgendwo ADC_Init() aufgerufen werden?

Und das hier geht so nicht:

  ADCSRA |= (1<<ADSC) | (1<<ADFR);                // eine ADC-Wandlung
  while (ADCSRA & (1<<ADSC) )
  {                          // auf Abschluss der Konvertierung warten
  }

Wenn Du ADFR setzt, dann wird automatisch immer wieder eine neue 
Wandlung gestartet.
ADSC wird in dem Fall nie zurückgesetzt.
Du könntest aber hier auf ADIF=1 warten, musst es aber von Hand 
zurücksetzten. ACHTUNG! Zurückgesetzt wird ADIF, indem man eine 1 
hineinschreibt!

[Karl H. (kbuchegg)]

Detlef Kunz schrieb:
> Vielleicht müsste ja mal irgendwo ADC_Init() aufgerufen werden?
>
> Und das hier geht so nicht:
>

>   ADCSRA |= (1<<ADSC) | (1<<ADFR);                // eine ADC-Wandlung
>   while (ADCSRA & (1<<ADSC) )
>   {                          // auf Abschluss der Konvertierung warten
>   }
>

> Wenn Du ADFR setzt, dann wird automatisch immer wieder eine neue
> Wandlung gestartet.
> ADSC wird in dem Fall nie zurückgesetzt.
> Du könntest aber hier auf ADIF=1 warten, musst es aber von Hand
> zurücksetzten. ACHTUNG! Zurückgesetzt wird ADIF, indem man eine *1*
> hineinschreibt!

Er könnte aber auch ganz einfach auf die Beendingung der Messung pfeifen 
und überhaupt nicht warten. In der Zeit, in der die obligatorische 
Copyright-Meldung zur Belustigung aller ausgegeben wird, macht der ADC 
eigenständig ein paar Hundert Messungen, so dass es völlig wurscht ist, 
ob der erste Wert nicht ganz so genau ist, er wird sowieso nicht 
verwendet.


Allerdings erhebt sich die Frage: Wozu eigentlich der Free-Running?

[Rob E. (newgenera2589)]

hi vielen dank schonmal für eure hilfe :)


Rob El schrieb:
> // Frequenzvorteiler auf 50MHz Teiler
> durch8 !!!!!!!!!!ungefähr 120mhz!!!!!!!!!

dieser Kommentar kann mit der Realität irgendwie garnix zu tun haben...

chris, ich habe den frequenzteilre auf 50mhz gehabt und mir wurde 120mhz 
empfohlen.... deswegen der kommentar

danke detlef das war mir nicht klar...

karl ich hatte zuerst die einzelmessung aber es hatte nicht 
funkioniert... dementsprechend habe ich gedacht versuch es doch einfach 
mal nur zum test im free-running modus...

[Karl H. (kbuchegg)]

Rob El schrieb:
> hi vielen dank schonmal für eure hilfe :)
>
>
> Rob El schrieb:
>> // Frequenzvorteiler auf 50MHz Teiler
>> durch8 !!!!!!!!!!ungefähr 120mhz!!!!!!!!!
>
> dieser Kommentar kann mit der Realität irgendwie garnix zu tun haben...
>
> chris, ich habe den frequenzteilre auf 50mhz gehabt und mir wurde 120mhz
> empfohlen.... deswegen der kommentar

Aber auch du musst zugeben, dass 50 durch 8 nicht 120 ergibt.

Der Kommentar ist im besten Fall einfach nur missverständlich.

Und da ist noch gar nicht drinnen, dass der Vorsatz 'm' milli (also 1 
Tausendstel) bedeutet, der Vorsatz 'M' aber Mega (also 1 Million).

> karl ich hatte zuerst die einzelmessung aber es hatte nicht
> funkioniert...

dann hättest du dem nachgehen sollen.
Die Routinen sehen in etwa so aus wie die aus dem Tutorial. Und die 
funktionieren. D.h. du musst bei deinen 'Anpassungen' irgendetwas 
zerstört haben.

> dementsprechend habe ich gedacht versuch es doch einfach
> mal nur zum test im free-running modus...

Free Running macht eigentlich nur dann wirklich Sinn, wenn man sich das 
Ergebnis mit einem Interrupt abholt und gleich wegspeichert.
Free Running macht hingegen Probleme, wenn man mal mehr als nur einen 
ADC Kanal benötigt. In deiner Anwendung gibt es keinen wirklichen Grund, 
warum man Free Running benutzen sollte. Wichtiger wäre es, wenn dein 
generelles Setup für gezielte ADC Einzelmessungen sauber funktioniert. 
Denn das brauchst du viel häufiger als einen Free Running Modus zur 
schnellstmöglichen Massenwandlung von Werten.

Zumal auch der mutwillige Verzicht auf 10 Bit und statt dessen 8 Bit zu 
benutzen in einem Messgerät eher etwas seltsam anmutet. Das letzte Bit 
in einem ADC wird zwar ein wenig klappern, aber auch das kann man durch 
Mittelung wieder in den Griff kriegen. Ob du in einem Schleifendurchlauf 
500ms wartest, oder ob du 32 mal misst und dann mit dem Durchschnitt der 
Messwerte das 'rauschen' des Eingangs ausgleichst, ist für den µC 
gehupft wie gesprungen.

[Uwe (Gast)]

> // Frequenzvorteiler auf 50MHz Teiler
> durch8 !!!!!!!!!!ungefähr 120mhz!!!!!!!!!
AArgh!
Schon mal das Datenblatt gelesen ... nein ... sieht man!

[Эраст Петрович Фандорин (Gast)]

Uwe nölte:AArgh!
>Schon mal das Datenblatt gelesen ... nein ... sieht man!

Leg Dich wieder hin. Das Alles war gestern Abend um 22:30 Uhr schon 
bekannt.

[Uwe (Gast)]

Ja und heute um 16:49 kam:
> chris, ich habe den frequenzteilre auf 50mhz gehabt und mir wurde 120mhz
> empfohlen.... deswegen der kommentar
wiederhinleg

[Uwe (Gast)]

Ich empfehle ihm deshalb 870MHz ...