Zwei ADC Messungen gleiches Ergebnis ATMega1280

[Bene J. (terfagter)]

Hallo zusammen,

Ich möchte mit meinem ATMEGA1280 zwei Spannungen an zwei verschiedenen 
ADC-Eingängen messen. Ich erhalte aber immer das gleiche Ergebnis. Immer 
die Spannung des ersten.
Als erstes Initialisiere ich den ADC:

void adcinit(void)
{
  uint16_t result;
  
  ADMUX |= (1<<REFS0);                // AVCC als Referenzspannung nutzen
  ADCSRA = (1<<ADPS1) | (1<<ADPS2);         // Frequenzvorteiler 64
  ADCSRA |= (1<<ADEN);                    // ADC aktivieren
  
  ADCSRA |= (1<<ADSC);                     // eine ADC-Wandlung 
  while (ADCSRA & (1<<ADSC) ) {}           // auf Abschluss der Konvertierung warten
  
  result = ADCW;                // ADCW muss einmal gelesen werden, sonst wird Ergebnis der nächsten Wandlung nicht übernommen.

}

Dann lese ich die zwei Spannungen hintereinander in einer Funktion ein:

void spgmessungakkus(void)
{
  float umrechnung = 0;
   
  //Akkuspg Hauptplatine
  ADMUX |= (1<<MUX0) | (1<<MUX1) | (1<<MUX2);   // Kanal wählen; hier: ADC7
  ADCSRA |= (1<<ADSC);                      // eine Wandlung "single conversion"
  while (ADCSRA & (1<<ADSC) ) {}            // auf Abschluss der Konvertierung warten
  
  umrechnung = ADCW;                         // ADC auslesen und speichern
  
  umrechnung = umrechnung*3300/1024;
  reg1[0] = umrechnung * 2;
  sprintf(spgakkuplatine, "%d.%02d", (reg1[0]/1000), (reg1[0] % 1000)/10);
  
  //Akkuspg Heizung
  ADMUX |= (1<<MUX1) | (1<<MUX2);           // Kanal wählen; hier: ADC6
  ADCSRA |= (1<<ADSC);                      // eine Wandlung "single conversion"
  while (ADCSRA & (1<<ADSC) ) {}            // auf Abschluss der Konvertierung warten
  
  umrechnung = ADCW;                         // ADC auslesen und speichern
  
  umrechnung = umrechnung*3300/1024;
  reg1[1] = umrechnung*2;
  sprintf(spgheizung, "%d.%02d", (reg1[1]/1000), (reg1[1] % 1000)/10);
}

Es liegen zur Zeit 0V an ADC6 an, trotzdem wird aber die Spannung vom 
ADC7 angezeigt!
Vielen Dank für eure Hilfe.

[hdd (Gast)]

ADMUX |= (1<<MUX1) | (1<<MUX2);           // Kanal wählen; hier: ADC6
Damit löschst du nicht MUX0, mach mal aus dem |= nur ein =

[Bene J. (terfagter)]

hdd schrieb:
> ADMUX |= (1<<MUX1) | (1<<MUX2);           // Kanal wählen; hier: ADC6
> Damit löschst du nicht MUX0, mach mal aus dem |= nur ein =

Lösche ich damit nicht auch meine Einstellungen zur Referenzspannung?
Ich habs jetzt mal mit den Zusatz ADMUX &= ~(1<<MUX0); gelöst.
Jetzt funktioniert es! Danke. Da hätte ich ja auch selber drauf kommen 
können...

[hdd (Gast)]

Bene Jan schrieb:
> Lösche ich damit nicht auch meine Einstellungen zur Referenzspannung?

Stimmt, hatte nicht mehr im Kopf, dass die ja im selben Register 
stecken. Ist doch schon etwas spät ;)

[Klaus D. (kolisson)]

Hi Bene Jan,
ich verstehe zwar nix von deinem C Quelltext , biete dir aber
folgendes an:



ADMUX = REFS1  REFS0  ADLAR  MUX4  MUX3  MUX2  MUX1 / MUX 0

gewünscht ist Ref = AVCC .. entspricht 01xxxxxx

Zudem gewünscht ist Kanal 6 = xx000110
und alternativ kanal 7      = xx000111

Die Grundeinstellung für ref = Avcc und kanal 0 ist also:
ADMUXBASIS = &b01000000


Wenn jetzt Kanalnummer in var "Kanal" gespeichert ist kannst du den 
Kanal (5Bit)
ja schonmal so isolieren : Kanal = Kanal and &B00011111

.. nun sind nur noch 5Bit aktiv.

admux = admuxbasis or Kanal setzt dir dann den Kanal.

Ich hoffe es hilft.

Gruss Klaus