Problem beim Vergleich von zwei ADC Werten

[Benedikt K. (benek)]

Hallo,
dieser Code soll bewirken, dass die LED an PB5 erst angehen soll, wenn 
die Werte der Potis an ADC0 und ADC1 höher als 200 sind. Jedoch dimmt 
die LED nur, wenn die beiden Werte unterschiedlich sind (ADC1 > 200, 
ADC0 < 200). Dabei sollte sie ausgehen. Wo liegt mein Fehler?

/*
 * PotentiometerCompare.c
 *
 * Created: 28.03.2013 13:54:33
 *  Author: Benedikt_2
 */ 


#include <avr/io.h>

int main(void)
{
  DDRB |= (1<<5);
  
  ADMUX |= (1<<REFS0); //sets Reference to AVCC
  ADMUX |= (1<<ADLAR); //sets ADC Result to ADCH
  ADCSRA |= (1<<ADPS2) | (1<<ADPS1) | (1<<ADPS0); //Prescaler = 128 -> ADC Clock = 125 kHz
  ADCSRA |= (1<<ADEN);
  
    while(1)
    {
        int ADC_0 = getADC0();
    int ADC_1 = getADC1();
    
    if (ADC_0 > 200 && ADC_1 > 200)
    {
      PORTB |= (1<<5);
    }
    else
    {
      PORTB &= ~(1<<5);
    }
    
}

int getADC0(void){
  int ADC0Result;
    ADMUX &= ~(1<<MUX0); //ADC Pin = ADC0
  ADCSRA |= (1<<ADSC); //start Conversion
  ADC0Result = ADCH; //ADC Result
  return ADC0Result;
}

int getADC1(void){
  int ADC1Result;
  ADMUX |= (1<<MUX0); //ADC Pin = ADC1
  ADCSRA |= (1<<ADSC); //start Conversion
  ADC1Result = ADCH; //ADC Result
  return ADC1Result;
}

Vielen Dank schonmal im vorraus,
Benedikt

[Peter II (Gast)]

du solltest mal warten bis der ADC fertig ist.

[Benedikt K. (benek)]

Peter II schrieb:
> du solltest mal warten bis der ADC fertig ist.

Wie bewerkstellige ich das?

[Bernd S. (bernds1)]

Benedikt K. schrieb:
> Peter II schrieb:
>> du solltest mal warten bis der ADC fertig ist.
>
> Wie bewerkstellige ich das?

while ( ADCSRA & ( 1<<ADSC ) )
{
   ;
}

Außerdem sollte erstmal eine Auslesung des ADC erfolgen, die
verworfen wird, damit der ADC sich stabilisiert.

[Benedikt K. (benek)]

Okay, ich denke ich habe es selber herausgefunden. Ein kurzer delay von 
20 ms vor jeder Conversion hat gerreicht.

MfG Benedikt

[Bernd S. (bernds1)]

Benedikt K. schrieb:
> Okay, ich denke ich habe es selber herausgefunden. Ein kurzer delay von
> 20 ms vor jeder Conversion hat gerreicht.

Nein, ein Delay ist erstens Zeitverschwendung, und zweitens was soll es 
vor der Conversion machen? Du wartest erst, und wenn du mit Warten 
fertig bist, startest du die Conversion und liest sofort das Ergebnis 
ein. Ohne zu wissen, ob er überhaupt schon fertig damit ist.

Für diesen Zweck ist das Flag da (ADSC), das zeigt dir an, ob er noch 
arbeitet oder schon fertig ist. Das ist die einzige Möglichkeit, es 
genau zu wissen.

[Benedikt K. (benek)]

Bernd S. schrieb:
> Benedikt K. schrieb:
>> Okay, ich denke ich habe es selber herausgefunden. Ein kurzer delay von
>> 20 ms vor jeder Conversion hat gerreicht.
>
> Nein, ein Delay ist erstens Zeitverschwendung, und zweitens was soll es
> vor der Conversion machen? Du wartest erst, und wenn du mit Warten
> fertig bist, startest du die Conversion und liest sofort das Ergebnis
> ein. Ohne zu wissen, ob er überhaupt schon fertig damit ist.
>
> Für diesen Zweck ist das Flag da (ADSC), das zeigt dir an, ob er noch
> arbeitet oder schon fertig ist. Das ist die einzige Möglichkeit, es
> genau zu wissen.

Danke. Verstanden :)

MfG Benedikt

[Rolf M. (rmagnus)]

Bernd S. schrieb:
> Für diesen Zweck ist das Flag da (ADSC), das zeigt dir an, ob er noch
> arbeitet oder schon fertig ist. Das ist die einzige Möglichkeit, es
> genau zu wissen.

Es gibt noch eine andere: Man kann den ADC auch so einstellen, daß er 
einen Interrupt auslöst, wenn er fertig ist.

[Bernd S. (bernds1)]

Rolf Magnus schrieb:
>
> Es gibt noch eine andere: Man kann den ADC auch so einstellen, daß er
> einen Interrupt auslöst, wenn er fertig ist.

Hallo Rolf,

das stimmt, das spart Rechenzeit.
Ich habe aber dem Benedikt nur den Fehler gezeigt, also seine 
Vorgehensweise korrigiert, ohne sein Konzept zu verändern. Es ist eben 
eine Art, wie man es machen kann.
Ich hoffe, du siehst es mir nach? ;-)