AD Wandler auf dem ATmega48

[charly (Gast)]

Hi ich habe ein Problem mit meinem ADWandler mit meinem ATmega48. Ich 
möchte 2 Kanäle auslesen und zwar 0 und 1 diese habe ich probehalber 
einmal mit VCC und den anderen mit GND beschaltet. Doch egal was ich 
mache ich bekomme für beide Werte immer die selbe Ausgabe, wenn ich mir 
den Wert über die Serielle Schnittstelle ausgeben lasse.

Ich hoffe ihr habt eine Idee wo mein Problem liegt. Ich bin total 
überfragt.

Hier habt ihr meine Register Konfiguration und die entsprechenden 
Funktionen und ein paar Eckdaten.

VCC = 5V
F_CPU = 1MHz interner Takt

void adcInit(){

  ADMUX = 0;
  
  DIDR0 |= (1 << ADC0D) | (1 << ADC1D);
  
  //ADC ist freigeschaltet, automatisch getriggert, interrupt ist enabled und prescaler ist bei 8 also 125kHz -> 104µs pro Wandlung
  ADCSRA = (1 << ADEN) | (1 << ADIE) | (1 << ADPS1) | (1 << ADPS0);
  ADCSRB = 0;
  
}

void adcChannel(uint8_t c){
  if(channel >= 0 && channel <= 7){
    //wähle den Channel aus aber lasse linksbündige einstellung drin 64 setzt ADLAR und die channel Variable die MUX Bits
    ADMUX = channel;
  }
}

ISR(ADC_vect){  
  uint8_t valueHigh = ADCH;
  uint8_t valueLow = ADCL;
  uint16_t value = valueLow + (valueHigh << 8);
  
  
  uartPutS("Channel ");
  uartPutC(channel + 48);
  uartPutS(" = ");
  uartPutC(value/1000 + 48);  //1000er Stelle
  uartPutC(value%1000/100 + 48);  //100er Stelle
  uartPutC(value%100/10 + 48);  //10er Stelle
  uartPutC(value%10 + 48);  //1er Stelle
  uartPutC('\n');
  //*/
  
  if(channel == 1){
    channel = 0;
  }else{
    channel = 1;        
  }
  
  adcChannel(channel);
  
  ADCSRA |= (1 << ADSC);
}

Die Ausgabe im Terminal:

Channel 1 = 0450<\n>
Channel 0 = 0450<\n>

Ich hoffe ihr könnt mir weiterhelfen :) Danke für eure Aufmerksamkeit.

charly

[Karl H. (kbuchegg)]

Du hast die REFS Bits im ADMUX auf 0. Bist du sicher, dass du eine 
extern über AREF eingespeiste Spannung als Referenz benutzen willst?

Ausserdem: Welche Werte kriegst du, wenn du nicht dauernd den Kanal 
wechselst, sondern immer nur denselben (entweder 0 oder 1)?

[Thomas (kosmos)]

Im ADMUX Register steht nicht nur die Kanalauswahl da musst du auch die 
Referenz einstellen.

[charly (Gast)]

Also wenn ich den Kanal wechsel dann, bleibt es bei einem Wert um die 
400 +- 10

VCC und AVcc sind beide direkt an 5V angeschlossen. Keine Kondensatoren. 
Und ich habe noch einen 1k Widerstand zwischen Klemme und dem mega48 das 
hätte ich noch erwähnen sollen.

Danke schonmal.

[Karl H. (kbuchegg)]

charly schrieb:
> Also wenn ich den Kanal wechsel dann, bleibt es bei einem Wert um die
> 400 +- 10
>
> VCC und AVcc sind beide direkt an 5V angeschlossen.

das ist erst mal egal. Denn weder Vcc noch AVcc werden benutzt. So wie 
du ADMUX eingestellt hast, wird die SPannung an AREF als 
Referenzspannung benutzt und nichts anderes.

> Keine Kondensatoren.

Kein Kommentar (zumindest jetzt noch nicht)

> Und ich habe noch einen 1k Widerstand zwischen Klemme und dem mega48 das
> hätte ich noch erwähnen sollen.

Zwischen welcher Klemme und welchem Anschluss des Mega48?

[Karl H. (kbuchegg)]

http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Analoge_Ein-_und_Ausgabe#Der_interne_ADC_im_AVR

http://www.mikrocontroller.net/articles/AVR-Tutorial:_ADC

[charly (Gast)]

Karl Heinz schrieb:
>> Und ich habe noch einen 1k Widerstand zwischen Klemme und dem mega48 das
>> hätte ich noch erwähnen sollen.
>
> Zwischen welcher Klemme und welchem Anschluss des Mega48?

zwischen dem Anschlusspin des Controllers und der Klemme an der später 
das Poti dran ist, welches zwischen VCC und GND geschaltet ist.

Karl Heinz schrieb:
> das ist erst mal egal. Denn weder Vcc noch AVcc werden benutzt. So wie
> du ADMUX eingestellt hast, wird die SPannung an AREF als
> Referenzspannung benutzt und nichts anderes.

AREF ist ebenfalls auf 5V geschaltet.

[Thomas E. (thomase)]

charly schrieb:
> uint8_t valueHigh = ADCH;
>   uint8_t valueLow = ADCL;
>   uint16_t value = valueLow + (valueHigh << 8);

ADCL muss vor ADCH ausgelesen werden!

Also überlass das dem Compiler, anstatt dir selbst etwas falsches 
zusammen zu basteln:

uint16_t value = ADC;

Die UART-Ausgaben in der ISR zu kommentieren, erspare ich mir jetzt.

mfg.

[charly (Gast)]

Ich habe jetzt erst das Low byte gelesen und auch deinen Rat mit value = 
ADC befolgt. Beides führt zu keinem Ergebnis. Ich bekomme immernoch nur 
seltsame Werte. Hat jemand noch einen Tipp?

Gruss charly

[Karl H. (kbuchegg)]

Können wir uns darauf einigen, mal ein ganz simples Testprogramm ohne 
Interrupt und sonstigen Schnickschnack, nur mit den Tutorial Routinen zu 
schreiben?

...
hier die Routinen aus dem Tutorial, wobei du die Referenzspannung so einstellst, wie du das brauchst.
...

int main()
{
  uint16_t adcval;

  ADC_Init();

... was immer du noch für die UART benötigst
...
 
  while( 1 ) {

    adcval = ADC_Read(0);  // Kanal 0

    sprintf( uart_buff, "ADC 0: %u\n", adcval );
    uartPutS( uart_buff );
  }
}

Dann variierst du mal die Spannung am ADC Eingang 0 (mit Voltmeter 
kontrollieren) und siehst nach, was die ausgegebenen Werte machen.

Wenn sich die wie erwartet verhalten, dann kommt das nächste 
Testprogramm

...
  while( 1 ) {

    adcval = ADC_Read(0);

    sprintf( uart_buff, "ADC 0: %u\n", adcval );
    uartPutS( uart_buff );

    adcval = ADC_Read(1);

    sprintf( uart_buff, "ADC 1: %u\n", adcval );
    uartPutS( uart_buff );
  }

[Ingo (Gast)]

Es sollte mich wundern wenn damit nicht jeden PC abschießt. Mach hinter 
dem Senden nochmal 500ms Pause mit nem Delay_ms(500).


Ingo

[flavius (Gast)]

charly schrieb:
> Karl Heinz schrieb:
>>> Und ich habe noch einen 1k Widerstand zwischen Klemme und dem mega48 das
>>> hätte ich noch erwähnen sollen.
>>
>> Zwischen welcher Klemme und welchem Anschluss des Mega48?
>
> zwischen dem Anschlusspin des Controllers und der Klemme an der später
> das Poti dran ist, welches zwischen VCC und GND geschaltet ist.

Jetzt ist Zeit für einen Schaltplan.