ATmega168 ADC nur Single Conversion trotz Free Run

[Tungsten (Gast)]

Hallo Forum,

ich bin gerade dabei ein Projekt von ATmega8 (4MHz) auf ATmega168 
(18,432MHz) zu transferieren, aber der ADC will nicht so richtig 
arbeiten.

Ich lese über den ADC-Eingang einen NTC-Wert, verwende die internen 1,1V 
als Referenz und habe dafür die Spannungsteiler (wegen 2,56V beim mega8) 
angepasst.

Nach dem Reset bekomme ich beim ersten Lesen des ADC einen plausiblen 
Wert, jedes weitere Lesen liefert nur 1023.

Habe schon einiges probiert. Vielleicht könnt ihr mich auf eine 
Eigenschaft oder Fehler aufmerksam machen.

Lg,
Tungsten

void init_adc(){
  // Set ADC reference
  ADMUX |= (1 << REFS1) | (1 << REFS0); // Internal 1.1V Voltage Reference with external cap at AREF

  ADCSRA |= (1 << ADPS2) | (1 << ADPS1) | (1 << ADPS0); // Prescaler 128
    
  ADCSRB |= (0 << ADTS2) | (0 << ADPS1) | (0 << ADPS0); // Free Running mode
  //ADCSRA |= (1 << ADATE); // Enable ADC Interrupt
    
  ADCSRA |= (1 << ADEN);// Enable ADC

  ADCSRA |= (1 << ADSC); // Start conversions
}
void setchannel_adc(uint8_t channel)
{
  ADMUX &= 0b11110000; //clean channel
  channel = channel & 0b00001111;
  ADCSRA &= ~(1 << ADEN);
  ADMUX = channel; //set channel
  ADCSRA |= (1 << ADEN);
}

/*
 * read from adc channel
 */
uint16_t read_adc(uint8_t channel) 
{
  setchannel_adc(channel);
  ADCSRA |= (1 << ADSC); // Start conversion
  while( !(ADCSRA & (1<<ADIF)) ); // Wait for conversion to complete 
  uint16_t adc = ADC;
  ADCSRA |= (1 << ADIF); // Clear ADIF by writing one to it
  return(adc);
}

Aufruf:

uint16_t ntcValue = read_adc(1);

[Spess53 (Gast)]

Hi

>  ADCSRA |= (1 << ADPS2) | (1 << ADPS1) | (1 << ADPS0); // Prescaler 128

>  ADCSRB |= (0 << ADTS2) | (0 << ADPS1) | (0 << ADPS0); // Free Running mode

Sieht ulkig aus. Aber der wirkliche Fehler ist, das du Autotrigger 
(ADATE in ADCSRA) gar nicht erlaubst.

MfG Spess

[Karl H. (kbuchegg)]

>   ADMUX = channel; //set channel


gratuliere. Du hast dir gerade die Einstellung der Referenzspannung 
unter dem Allerwertesten weggeschaltet.

[Tungsten (Gast)]

@ Spess, danke für den Hinweis, ADATE habe ich hinzugefügt.
@K.H. .. der Allerwerteste ist wichtig, darauf werde ich jetzt achten.

Ich habe die Änderungen eingebracht, leider gleiches Verhalten. Nach dem 
Reset liefert das erste read_adc einen Wert (z.B.609) und jedes folgende 
read_adc 1023,1023,1023,...

????

void setchannel_adc(uint8_t channel)
{
  ADCSRA &= ~(1 << ADEN);
  ADMUX |= (channel&0x0F); //set channel
  ADMUX |= (1 << REFS1) | (1 << REFS0); // Internal 1.1V Voltage Reference with external cap at AREF
  ADCSRA |= (1 << ADEN);
}

/*
 * read from adc channel
 */
uint16_t read_adc(uint8_t channel) 
{
  setchannel_adc(channel);
  ADCSRA |= (1 << ADSC); // Start conversion
  while( !(ADCSRA & (1<<ADIF)) ); // Wait for conversion to complete 
  uint16_t adc = ADC;
  ADCSRA |= (1 << ADIF); // Clear ADIF by writing one to it
  return(adc);
}

void init_adc(){
  // Set ADC reference
  ADMUX |= (1 << REFS1) | (1 << REFS0); // Internal 1.1V Voltage Reference with external cap at AREF
  ADCSRA |= (1 << ADPS2) | (1 << ADPS1) | (1 << ADPS0); // Prescaler 128  
  ADCSRB |= (0 << ADTS2) | (0 << ADPS1) | (0 << ADPS0); // Free Running mode
  ADCSRB |= (1 << ADATE); // Auto Trigger Enable
    
  ADCSRA |= (1 << ADEN);// Enable ADC
  ADCSRA |= (1 << ADSC); // Start conversions
}

[Karl H. (kbuchegg)]

Mir ist eines nicht klar.
Wozu der ganze Hackmack mit dem Free Running, wenn du dann sowieso in 
der read Funktion den ADC händisch triggerst.

Hast du schon mal die ADC Funktionen aus dem AVR-GCC-Tutorial so wie sie 
sind (mit der Ausnahme der Referenzspannungseinstellung) benutzt?

AVR-GCC-Tutorial/Analoge Ein- und Ausgabe: Nutzung des ADC

[Tungsten (Gast)]

Die Änderungen passen!!!!!

Es geht, war ein nach den Änderungen noch ein Fehler beim Compilieren.

Jetzt geht es, SUPER, DANKE

[Tungsten (Gast)]

K.H. da hast du schon recht, nach einigen Stunden Fehlersuchen probiert 
man alles, egal ob logisch oder nicht...
Jetzt wo es geht werde ich alles redundante weggeben und auch händisch 
triggern.

DANKE.

[Spess53 (Gast)]

Hi

>@ Spess, danke für den Hinweis, ADATE habe ich hinzugefügt.

>  ADCSRB |= (1 << ADATE); // Auto Trigger Enable

Nö. ADATE ist in ADCSRA.

MfG Spess

[Karl H. (kbuchegg)]

Tungsten schrieb:
> K.H. da hast du schon recht, nach einigen Stunden Fehlersuchen probiert
> man alles, egal ob logisch oder nicht...
> Jetzt wo es geht werde ich alles redundante weggeben und auch händisch
> triggern.

Tip:
Lass das ADIF Flag in Ruhe. Du brauchst es nicht, und es macht nur 
Arbeit. Die Sache ist ganz einfach: Du setzt das ADSC Bit und wenn der 
ADC fertig ist, zieht er es wieder auf 0 zurück.

Siehe Tutorial.

[Tungsten (Gast)]

@ Spess, danke für den Hinweis, das war jetzt wirklich schlamping, 
wirklich schlampig...

@K.H. Danke, ich habe das geändert.

Vermutlich könnte ich auch ADC direkt zurückgeben und bräuchte die adc 
Variable auch nicht.

Schönen Abend noch...
Tungsten

uint16_t adc_read(uint8_t channel) 
{
  adc_setchannel(channel);
  ADCSRA |= (1 << ADSC); // Start conversion
  while( IS_BIT_SET(ADCSRA,ADIF) ); // Wait for conversion to complete 
  uint16_t adc = ADC;
  return(adc);
}

[Karl H. (kbuchegg)]

>   while( IS_BIT_SET(ADCSRA,ADIF) ); // Wait for conversion to complete

ADSC, nicht ADIF


Studier doch bitte die Routinen aus dem Tutorial. Es macht keinen 
wirklichen Spass deine ganzen Tippfehler zu korrigieren, wenn du dort 
fix fertige, sauber funktionierende Funktionen finden kannst.