Atmega8 mehrere ADC Kanäle auslesen

[Asd J. (digim)]

Hey Leute
Ich möchte gerne mit einem Atmega8 zwei Akkus mittels ADC auslesen und 
den jeweiligen Wert auf einem Display anzeigen lassen.

ich hab folgendes Problem:
-ADC funktioniert nur an Pin-PC0 und zeigt diesen Wert als adc(0) und 
adc(1) an ( also auf zwei Zeilen gleizeitig )
-ADC1 zeigt den Wert 0 dauerhaft an an wenn ich einen Akku an PC1 klemme


das ist der Code:

#include <stdlib.h>
#include <stdio.h>        
#include <avr/io.h>
#include <avr/pgmspace.h>
#include <stdint.h>
#define F_CPU 3686400UL
#include <util/delay.h>

#include "lcd_162.h"


uint16_t adc(uint8_t mux)
{
     uint16_t result;
 
    ADMUX |= (mux & 0b00000111);   // Kanal waehlen
 
  ADMUX = (0<<REFS1)|(1<<REFS0);
  

  //Frequenzvorteiler setzen und ADC aktivieren
  ADCSRA = (1<<ADPS1)|(1<<ADPS0);            //Frequenzvorteiler
  ADCSRA |= (1<<ADEN)|(1<<ADIE);            //ADC aktivieren
      
    // Dummy-readout
    ADCSRA |= (1<<ADSC);                // eine ADC-Wandlung 
    while ( ADCSRA & (1<<ADSC) )
    {  ;             // auf Abschluss der Konvertierung warten 
    }
  
  result = ADCW;    
    
  ADCSRA &= ~(1<<ADEN);       // ADC deaktivieren
   ADMUX &= ~0b00000111; // alten Kanal loeschen
  return result;
}




int main(void)
{  
  char buffer[20];            
  DDRB=0xFF;                   //Port B als Ausgänge definieren
  DDRC=0x00;                    //Port C als Eingänge definieren
  lcd_init();

  uint16_t akku1;
  uint16_t akku2;
while(1)
{    akku1=adc(0);
    akku2=adc(1);
  lcd_write_zeile(itoa(akku1, buffer,10),1); // Adc Wert von PC0 in Zeile 1 anzeigen 
  lcd_write_zeile(itoa(akku2, buffer,10),2); // Adc Wert von PC1 in Zeile 2 anzeigen 

  _delay_ms(50);
}
}

[Fritz G. (fritzg)]

Du setzt den Kanal und löscht ihn in der nächsten Zeile.

[Asd J. (digim)]

wo meinst du genau ?

[Magnus M. (magnetus)]

Fritz Ganter schrieb:
> Du setzt den Kanal und löscht ihn in der nächsten Zeile.

Asd Jul schrieb:
> wo meinst du genau ?

Asd Jul schrieb:

ADMUX |= (mux & 0b00000111);   // Kanal waehlen
ADMUX = (0<<REFS1)|(1<<REFS0);

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

@Asd Jul:
Bitte 1. keine Doppelpostings
und 2. Fragen nicht NICHT im Projekte&Code Forum. Dort steht klar und 
leserlich:

Hier könnt ihr eure Projekte, Schaltungen oder Codeschnipsel 
vorstellen und diskutieren. Bitte hier keine Fragen posten!

Zudem hat dein Problem mit dem GCC an sich nichts zu tun. Du hast kein 
Problem mit dem Compiler, sondern mit dessen Anwendung...

EDIT: und um den Quelltext herum kannst du die pasenden Tokens machen:

[c]
  C-Quelltext
[/c]

[Asd J. (digim)]

cool danke für den Hinweis :-)
Die mageren Infos bringen mich leider nicht weiter.

Kann man das auch noch ein bisschen mit Leben füllen :-) ?

[schwach (Gast)]

Magnus M. schrieb:
> Asd Jul schrieb:
>

> ADMUX |= (mux & 0b00000111);   // Kanal waehlen
> ADMUX = (0<<REFS1)|(1<<REFS0);
>

@Asd Jul: Weißt du nicht was die beiden Anweisungen machen? und welche 
Bits danach im ADMUX stehen?

[Dietrich L. (dietrichl)]

Asd Jul schrieb:
> Die mageren Infos bringen mich leider nicht weiter.
>
> Kann man das auch noch ein bisschen mit Leben füllen :-) ?

Mit der Zuweisung

ADMUX =

werden alle Bits des Registers überschrieben, also auch die in der 
Zeile vorher eingestellten Bits des Kanals.
Wenn Du nicht alle Bits ändern willst, brauchst Bitmanipulation: 
https://www.mikrocontroller.net/articles/Bitmanipulation

Gruß Dietrich

[Asd J. (digim)]

Anscheinend nicht richtig :D

Also ich dachte das ich in der ersten Zeile die Kanäle abwähle und in 
der zweiten die Referenz für den Adc

Aber klär mich auf :-)

[schwach (Gast)]

schwach schrieb:
> ... und welche Bits danach im ADMUX stehen?

@Asd Jul: Ich kläre Denkfaule nicht auf. Da gibt es genug Schriften 
drüber.

[Asd J. (digim)]

@schwach
Also so Kommentare kannst du dir auch echt sparen. Sowas macht mich 
sauer. Da kommt man ma nicht direkt auf die Lösung und hat Vll auch noch 
Wissesnlücken und wird dann als Denkfaul bezeichnet.
Das is echt das letzte.Dann musste auch nix in Foren kommentieren.

@Dietrich
Danke für den Tipp. Ich werde ihn gleich mal ausprobieren