ADC des ATmega128

[Jens K. (mister232)]

Hallo Leute,

könnt ihr mal über folgenden Code schauen:

// Function to initialize the microcontroller's internal ADC
void init_ADC(void)
{
  ADMUX = (1<<REFS0) | (1<<ADLAR); // Select AVCC as reference, Left adjust ADC result to allow easy 8 bit reading
  ADCSRA = (1<<ADPS1) | (1<<ADPS0) | (ADPS2); // Prescaler = 128 -> 62.5kHz at 8MHz clock
  ADCSRA |= (1<<ADEN); // Enable ADC
  
  ADCSRA |= (1<<ADSC); // Dummy conversion, required in datasheet
  while (ADCSRA & (1<<ADSC)){} // Wait for conversion to be done
  (void) ADCW; // Read conversion result and throw away
}

// Function which does one ADC conversion at the selected ADC channel
uint16_t internal_ADC(uint8_t channel)
{
  ADMUX = (ADMUX & ~(0x1F)) | (channel & 0x1F); // Select channel for AmpSig1
  ADCSRA |= (1<<ADSC); // Convert signal
  while (ADCSRA & (1<<ADSC)){} // Wait for conversion to be done
  
  return ADCW;
}

Iregendwie will der ADC nicht. Er gibt mir immer nur Werte zwischen 15V 
und 16V aus, obwohl der Eingang auf GND liegt.

An AVCC liegen 5V aus einer Referenzspannungsquelle und die im 
Datenblatt angegebene LC-Schaltung

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Jens K. schrieb:
> ADCSRA = (1<<ADPS1) | (1<<ADPS0) | (ADPS2); // Prescaler = 128 ->
> 62.5kHz at 8MHz clock

kleiner Tippfehler

[M. K. (sylaina)]

Jens K. schrieb:
> Iregendwie will der ADC nicht. Er gibt mir immer nur Werte zwischen 15V
> und 16V aus, obwohl der Eingang auf GND liegt.

Also für 15 und 16V bräuchte man etwas mehr infos, was aber auffällt: Du 
hast ADLAR gesetzt und liest das komplette ADC-Register aus. Das ist 
bestimmt nicht so gewollt. Bei ADLAR benutzt man eigentlich nur das 
ADCH-Byte, das ADCL-Byte interessiert dabei eigentlich nicht.

[Jens K. (mister232)]

Vielen Dank, jetzt klapts!

[Jens K. (mister232)]

Zu früh gefreut :-(

Jetzt kommt nur noch 0V raus, auch wenn am Eingang 5V liegen

[Jens K. (mister232)]

Der Aufruf der Funktion sieht momentan so aus:

// *********** Amperometric sensor 1 **********************************
    AmpSig1 = (internal_ADC(0) * 5) / 1024; // Convert ADC value to volt value
    
    // *********** Amperometric sensor 2 **********************************
    AmpSig2 = (internal_ADC(1) * 5) / 1024; // Convert ADC value to volt value
    
    dtostrf(AmpSig1, 6, 2, value_as_string);
    sprintf(output,"%s V\n\r", value_as_string);
    debug_message(output);
    dtostrf(AmpSig2, 6, 2, value_as_string);
    sprintf(output,"%s V\n\n\r", value_as_string);
    debug_message(output);

[M. K. (sylaina)]

Aufbau auch OK?
Ah, und mach mal Float-Zahlen in deiner Berechnung. Nicht dass du in INT 
rechnest.

also *5.0 und /1024.0

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Du hast leider die Definition für deine AmpSig1 und 2 Variablen nicht 
gepostet, und auch den Buffer 'value_as_string' nicht. dstrtof() will da 
double sehen und der Buffer muss gross genug sein.

Zum Testen ist es hilfreich, dir ADCW einfach mal mit itoa auf den 
Output zu werfen.

[Jens K. (mister232)]

Also das mit dem 5.0 und 1024.0 bringt zumindest schonmal andere Werte 
als 0V auf den Schirm. Am Anfang wird für die erste Variabel 0.31V and 
für die zweite 0.32V ausgegeben. Der erste Wert geht dann auf 0.33V hoch 
und bleibt da, egal welche Spannung ich am Eingang anlege.

AmpSig1 und AmpSig2 sind float Variablen, der Buffer für dstrof() ist 
groß genug.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Jens K. schrieb:
> AmpSig1 und AmpSig2 sind float Variablen,

Warum verwendest du dann dtostrf()? Float ist float und double is 
double.
Float Variablen gibst du ohne Umschnurzel-Trara direkt mit sprintf aus:

sprintf(value_as_string,"%f V\n\r", AmpSig1);

[M. K. (sylaina)]

Jens K. schrieb:
> Also das mit dem 5.0 und 1024.0 bringt zumindest schonmal andere Werte
> als 0V auf den Schirm. Am Anfang wird für die erste Variabel 0.31V and
> für die zweite 0.32V ausgegeben. Der erste Wert geht dann auf 0.33V hoch
> und bleibt da, egal welche Spannung ich am Eingang anlege.
>
> AmpSig1 und AmpSig2 sind float Variablen, der Buffer für dstrof() ist
> groß genug.

Hast du schon versucht dir mal stumpf den ADC-Wert ausgeben zu lassen, 
also  statt deinen Code mal

uint16_t AmpSig1;
uint16_t AmpSig2;
AmpSig1 = internal_ADC(0);
AmpSig2 = internal_ADC(1);

itoa(AmpSig1, value_as_string, 10);
sprintf(output,"%s V\n\r", value_as_string);
debug_message(output);
itoa(AmpSig2, value_as_string, 10);
sprintf(output,"%s V\n\n\r", value_as_string);
debug_message(output);

zu probieren? Und bist du dir sicher, dass die Spannung, die du angelegt 
hast, auch am Pin anliegt? (direkt mal auf dem Pin messen)

Die Änderungen von oben hast du schon umgesetzt (u.a. das "Problem" mit 
ADLAR und ADSP2)?

Matthias S. schrieb:
> Float Variablen gibst du ohne Umschnurzel-Trara direkt mit sprintf aus:

Und dann bekommt er, je nach Compiler-Version (wie das Makefile 
aufgebaut ist usw.), ein ? raus. Ich nehme dabei auch immer dtostrf();