ADCMUX Atmega328P

[Frank W. (frankydaniels)]

Hallo Zusammen,

ich habe mal wieder ein Problem mit meinem ADC beim Atmega328P.

Nachdem jetzt nun die Initialisierung korrekt ist wird der Interrupt 
durch den ADC aufgerufen. Ich möchte nun für einen Kanal 200 Werte und 
dann für den anderen Kanal 200 Werte aufnehmen. Dies soll immer im 
Wechsel im Freerunningmode passieren. Am Ende soll dies eine 
Effektivwertmessung für zwei unterschiedliche Sinussignale geben.

Hier die Initialisierung:

//ADC Init
  ADMUX = (1 << REFS0);//interne Referenz = Betriebsspannung
  
  ADCSRA = (1<<ADATE) | (1 << ADEN) | (1 << ADIE) | (1 << ADPS2) | (1 << ADPS1);
  //Free Running mode, ADC=Ein, ADC Interrupt enable, Prescaler 64 = 125kHz bei F_CPU = 8Mhz
    
  sei();                            // Set the I-bit in SREG

  ADCSRA |= (1<<ADSC); //AD-Wandler starten (Free Running Mode)

Nun habe ich wohl ein Problem mit meinem Mux. Wenn ich Wandlungen auf 
einen Kanal durchführe funktioniert alles.
Wenn ich nun jedoch das oben Geschilderte umsetzen möchte gibt der 
Wandler immer nur den Wert 1024 aus. Ich weis aber nicht was ich falsch 
mache. Die Signale liegen an den Pins ADC1 und ADC0.

Hier meine ADC Interrupt routine:

uint16_t ADC1_Data[200]; 
uint16_t ADC2_Data[200];
uint8_t ADC_Zaehl=0;
uint8_t toggle = 0;

ISR(ADC_vect)
{
  if (toggle)
  {
    ADC1_Data[ADC_Zaehl] = ADCW&0x03FF; 
    ADC1_Data[ADC_Zaehl]= ADC1_Data[ADC_Zaehl];
    if (ADC_Zaehl>=199)
    {
      ADMUX = (1<< MUX0);
      ADFlag = 1;
      ADC_Zaehl=0;
      toggle=0;
    }
    else
    {
      ADC_Zaehl++;
    }
  }
  else
  {
    //cli();
    ADC2_Data[ADC_Zaehl] = (ADCW&0x03FF);
    //sei();
    ADC2_Data[ADC_Zaehl]= ADC2_Data[ADC_Zaehl];
    if (ADC_Zaehl>=199)
    {
      ADMUX = 0x00;
      ADFlag = 1;
      ADC_Zaehl=0;
      toggle=1;
    }
    else
    {
      ADC_Zaehl++;
    }
  }  
}

Die variable Toggle gibt an welcher Kanal gerade dran ist. Die Variable 
ADFlag gibt an wann ein Datenarray fertig beschrieben ist um somit die 
spätere Auswertung zu starten.

Muss ich beim ADMUX auf irgendetwas achten? Im Datenblatt habe ich 
nichts nennenswertes gefunden.

Danke schon mal für eure Antworten

Mfg Franky :)

[Frank W. (frankydaniels)]

Bitte nicht an den unnötigen Zeilen stören wie:

ADC2_Data[ADC_Zaehl]= ADC2_Data[ADC_Zaehl];

Das sind noch Überreste von verschiedenen Versuchen :-P

[Felix P. (fixxl)]

Gewöhne dir bitte an - denn das war schon im ersten Thread das Problem, 
Register nicht mit der Holzhammermethode

REGISTER = WERT

 zu beschreiben, sondern setze oder lösche die jeweiligen Bits gezielt 
durch

REGISTER |= 1<<BITNUMMER // setzen
REGISTER &= ~(1<<BITNUMMER) // löschen

Wenn du ADMUX = 1<<MUX0 setzt, löschst du dabei die Einstellung für die 
Referenzspannung aus der Initialisierung, versuche es mal mit ADMUX |= 
1<<MUX0

[Frank W. (frankydaniels)]

Felix Pflaum schrieb:
> Gewöhne dir bitte an - denn das war schon im ersten Thread das Problem,
> Register nicht mit der Holzhammermethode

REGISTER = WERT

 zu
> beschreiben, sondern setze oder lösche die jeweiligen Bits gezielt durch
>

REGISTER |= 1<<BITNUMMER // setzen
> REGISTER &= ~(1<<BITNUMMER) // löschen

>
> Wenn du ADMUX = 1<<MUX0 setzt, löschst du dabei die Einstellung für die
> Referenzspannung aus der Initialisierung, versuche es mal mit ADMUX |=
> 1<<MUX0

Ohman danke, es funktioniert. Zweimal hintereinander den gleichen Fehler 
-.-

Danke nochmals :) Ich werde jetzt in Zukunft versuchen es immer so zu 
machen :)