ADC-Messung an mehreren Ports

[subguru (Gast)]

Hallo Leute,

ich versuche gerade an meinem atmega168 abwechselnd die Ports ADC6 und 
ADC7 einzulesen.

So aktiviere ich das ganze:

ADMUX |= (1<<REFS1) | (1<<REFS0); 
ADCSRA = (1<<ADEN)|(1<<ADSC)|(1<<ADIF)|(1<<ADIE)|(1<<ADPS2)|(1<<ADPS1)|(1<<ADPS0); 
// ADC On | ADC Start | Irq after Compare | Irq On | Prescale 128

und so versuche ich das Ganze zuzuweisen:

SIGNAL(SIG_ADC) {
      
  switch(ADMUX & 0x07) {
     case 0x06:   adc0_value = ADCW; 
                  ADMUX |= 0x07;break;
     case 0x07:   adc1_value = ADCW; 
                  ADMUX |= 0x06;break;
     default:     ADMUX |= 0x06;break;
    } 
    
    ADCSRA |= (1<<ADSC);
 }

Mein Problem ist, er springt nur einmal den Port 0x06 an.
Den Port 0x07 aktualisiert er aber regelmäßig und korrekt.

An was kann denn das liegen?

Danke im Voraus!

[Anon Y. (avion23)]

Ich mag diese Bitschreibweise nicht. Benutz doch (1 << ADblahblah).

Wenn ADMUX einmal 0x07 geworden ist, wird es nie wieder 0x06 werden. Du 
löschst kein bit.

[Michael S. (captain-stone)]

Hallo subguru (komischer Vorname),

schau Dir doch mal das Tutorial an:

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

und ich erinnere mich an einen Tread vor kurzem:

Beitrag "ADC: Zweiter Kanal misst nicht"

Sonst bringt die Suchfunktion bei den Foren auch ganz viel brauchbares.

Michael ;-)

[subguru (Gast)]

Oh ja, vielen Dank!!!

Ich ging davon aus er setzt mir die Bits so wie ich sie angebe. Da fehlt 
mir wohl die Praxis ;-)

[Michael U. (amiga)]

Hallo,

subguru schrieb:
> Oh ja, vielen Dank!!!
>
> Ich ging davon aus er setzt mir die Bits so wie ich sie angebe. Da fehlt
> mir wohl die Praxis ;-)

Was hat das mit Praxis zu tun? ;-)

Schreib es doch mal nicht verkürzt:
     default:     ADMUX |= 0x06;break;

     default:     ADMUX = ADMUX | 0x06;

Wenn jetzt in ADMUX bereits 0x07 steht und mit 0x06 oder verknüpft wird, 
was ändert sich dann wohl?

   00000111
OR 00000110
------------
   00000111

Gruß aus Berlin
Michael

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

subguru schrieb:
> Ich ging davon aus er setzt mir die Bits so wie ich sie angebe.
Er setzt sie schon, aber er lässt die anderen Bits gesetzt.
So wäre das besser:

     case 0x06:   adc0_value = ADCW; 
                  ADMUX = 0x07; break;
     case 0x07:   adc1_value = ADCW; 
                  ADMUX = 0x06; break; // diese Zeile könntest du sogar ganz weglassen ;-)
     default:     ADMUX = 0x06; break;

[Michael U. (amiga)]

Hallo,

Lothar Miller schrieb:
> subguru schrieb:
>> Ich ging davon aus er setzt mir die Bits so wie ich sie angebe.
> Er setzt sie schon, aber er lässt die anderen Bits gesetzt.
> So wäre das besser:
>

>      case 0x06:   adc0_value = ADCW;
>                   ADMUX = 0x07; break;
>      case 0x07:   adc1_value = ADCW;
>                   ADMUX = 0x06; break; // diese Zeile könntest du sogar
> ganz weglassen ;-)
>      default:     ADMUX = 0x06; break;
>

auch diese Lösung führt nicht unbedingt zum gewünschten Ergebnis,
in ADMUX sind noch andere Bits, REFS0, REFS1 und ADLAR beim Mega8 als 
Beispiel.

Die würden dann mit auf 0 gesetzt...

Richtig wäre:

      default:     ADMUX = (ADMUX & 0xF8) | 0x06; break;
Noch besser wäre eine Maske zu definieren:

#define MUX_MASK 0b11111000 // MUX-Bits auf 0
#define MUX6     0b00000110
#define MUX7     0b00000111

und dann
      default:     ADMUX = (ADMUX & MUX_MASK) | MUX6; break;

oder ähnlich.
Die Definitionen ins passende Headerfile gepackt und man hat bei 
Umbauten nur eine Stelle zum Ändern.

PS: meine Fehler bitte durch C-Programmierer korrigieren,
ich mache immernoch sehr wenig C und viel Assembler...

Gruß aus Berlin
Michael