Atmega48 & ADC

[Johannes (Gast)]

Hallo Leute,

ich habe ein Board entwickelt für die Ansteuerung von LED-Treibern. 
Soweit hat alles funktioniert...PWM, Timer, I²C, SPI und UART. Jedoch 
eine wichtige Komponente (ADC) funktioniert halt überhaupt nicht.

void adc_init (void)
{
  DDRC   &= ~(1<<port);            /*Den Port auf einang schalten*/

  ADMUX  = (1<<REFS0);            /*Referenz auf AREF - interne Ref aus */

  ADCSRA = 0x07;                /*Teiler auf 128 - höchste Genauigkeit*/
  
  ADCSRB = 0x00;                // Einfach so aus Verzwiflung auf null

  ADCSRA |= (1<<ADEN);            /*Analogwandler einschalten*/
}


unsigned int adc_read (unsigned char port)
{
  unsigned int value;

  ADMUX  = (0x0F & port);    // Mux auf Port ausrichten

  ADCSRA |= (1<<ADSC);    // Wandlung starten
  while (!(ADCSRA & ADSC));  // Warten bis wandlung ende
  
  value = ADCH;
  value += ADCL<<8;

  return value;
}

Mein Controller... Atmega48 - AVCC & VCC=3,3 - AREF = AVCC.
Returnwert der Funktion ist immer 0.

Was mach ich falsch? Sieht einer von euch einen Fehler?

Vielen Dank im voraus!

Gruß
Johannes

[Johannes B. (Firma: SSB-Computer) (ssb-computer)]

Sorry! Falscher Code. Alter Stand.

Hier der neue. ;)

#include <avr/io.h>

void adc_init (void)
{
  ADMUX  = 0x00;            /*Referenz auf AREF - interne Ref aus */

  ADCSRA = 0x07;                /*Teiler auf 128 - höchste Genauigkeit*/
  
  ADCSRB = 0x00;                // Einfach so aus Verzwiflung auf null

  ADCSRA |= (1<<ADEN);            /*Analogwandler einschalten*/
}


unsigned int adc_read (unsigned char port)
{
  unsigned int value;

  ADMUX  = (0x0F & port);    // Mux auf Port ausrichten

  ADCSRA |= (1<<ADSC);    // Wandlung starten
  while (!(ADCSRA & ADSC));  // Warten bis wandlung ende
  
  value = ADCH;
  value += ADCL<<8;

  value = ADCL;

  return value;
}

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

Johannes Boss schrieb:
> ADMUX  = (0x0F & port);    // Mux auf Port ausrichten

Ist das nicht der alte Fehler, wo die Referenzauswahl gelöscht wird? 
Kann jemand mal das Tutorial endlich korrigieren?
Richtig wäre :
ADMUX  = ((ADMUX & 0xF0) | (0x0F & port));    // Mux auf Port ausrichten

[Krapao (Gast)]

Im AVR-GCC-Tutorial steht bereits:

>  // Kanal waehlen, ohne andere Bits zu beeinflußen
>  ADMUX = (ADMUX & ~(0x1F)) | (channel & 0x1F);

[Johannes B. (Firma: SSB-Computer) (ssb-computer)]

Hy funktioniert wunderbar!

Ich kann mich nur für meine blindheit entschuldigen :(

Hab ich in dem Projekt einfach nicht drüber nachgedacht. Sorry!