ADC nicht ganz verstanden oder fehl Interpretation

[amle (Gast)]

Guten Tag
Ich habe hier etwas code betreffend einer Abfrage des ADC von Arduino, 
der soweit funktioniert. Jedoch wenn ich 2 ADC Abfrage sind die 
Variablen analgValue0/1 vertauscht, warum?

Siehe demarkieren von !!!!!!!! (ADMUX = ADCH1;)
Kann mir da jemand helfen?
Danke.

#include <Arduino.h>
#define ADC0 0
#define ADC1 1
#define ADCH0 ((1 << REFS0) | ADC0)
#define ADCH1 ((1 << REFS0) | ADC1)

volatile int analogValue0;
volatile int analogValue1;
void setup() {
  Serial.begin(115200);
  ADC_Init();
}

void loop() {
   Serial.print("Value1: ");
   Serial.print(analogValue0);
   Serial.print("   Value2: ");
   Serial.println(analogValue1);
}

// Function:    snaAdcIntSetup()
// Description: This funtion sets up the ADC for generating interrupts.
// Arguments:   None
// Returns:     None
void ADC_Init() {
  // Set for the first ADC channel.
  ADMUX = ADCH1;
  // 0=2,1=2,2=4,3=8,4=16,5=32,6=64, 7=128
  ADCSRA = (1 << ADEN) | (1 << ADATE) | (1 << ADIE) | (1 << ADPS2) | (0 << ADPS1) | (1 << ADPS0); 
  ADCSRB = 0;
  sei();
  ADCSRA |= (1 << ADSC);
}

// The ADC interrupt function.
ISR(ADC_vect) {
  int iCannel = ADMUX & ((1 << MUX3) | (1 << MUX2) | (1 << MUX1) | (1 << MUX0));
  switch (iCannel) {
    case ADC0:
      analogValue0 = ADCL | (ADCH << 8);
      // !!!!!!!!!!!!! für den zweiten kanal unmarkieren, dann sind die werte vertauscht
      //ADMUX = ADCH1;
      break;
    case ADC1:
      analogValue1 = ADCL | (ADCH << 8);
      ADMUX = ADCH0;
      break;
  }
  //ADCSRA |= (1 << ADSC);
}

[Stefan F. (Gast)]

> analogValue0 = ADCL | (ADCH << 8);

Kleiner Tipp: Das kannst du einfach  "analogValue0 = ADC" schreiben.

Nur zur Erklärung deines Phänomens:

Dein ADC läuft im Free-Running Modus, er misst also fortlaufend. Sobald 
eine Messung fertig ist, wird die ISR aufgerufen. Du schaltest dort den 
Kanal um, aber während dessen läuft bereits die nächste Messung. Der 
Kanalwechsel wirkt sich daher erst auf die  Messung danach aus.

Bei zwei Kanälen läuft es also daher auf die von dir beobachtete 
Vertauschung hinaus. Wenn du mehr als zwei Kanäle hättest, würdest zu 
einen Zirkulären Versatz um 1 Position beobachten.

[amle (Gast)]

Ja danke, das dachte ich mir und ja den Versatz um 1 habe ich bemerkt, 
leider.

Aber das vorgehen ist doch richtig oder mache ich da was falsch?
Getrennt kann man die IRQ ja nicht definieren ohne grösseren Aufwand?

[Stefan F. (Gast)]

amle schrieb:
> Aber das vorgehen ist doch richtig oder mache ich da was falsch?

Ja das ist richtig. Gegen den Versatz kannst du im free-running Modus 
nichts machen. Der wäre nur bei Einzelmessungen vermeidbar, die übrigens 
auch Interrupts nutzen könnten.

Wenn du den ADC innerhalb der ISR jedes mal neu starten würdest, hättest 
du weniger Messungen pro Sekunde. Und jedes mal wenn ISR kurz gesperrt 
sind, würde es sich entsprechend verzögern.

Ob das so zu deiner Anwendung passen würde, musst du selbst entscheiden.