ADC-Messung an mehreren Ports

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.

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 ;-)

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

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:

1

     case 0x06:   adc0_value = ADCW; 

2

                  ADMUX = 0x07; break;

3

     case 0x07:   adc1_value = ADCW; 

4

                  ADMUX = 0x06; break; // diese Zeile könntest du sogar ganz weglassen ;-)

5

     default:     ADMUX = 0x06; break;

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:
>

1

>      case 0x06:   adc0_value = ADCW;

2

>                   ADMUX = 0x07; break;

3

>      case 0x07:   adc1_value = ADCW;

4

>                   ADMUX = 0x06; break; // diese Zeile könntest du sogar

5

> ganz weglassen ;-)

6

>      default:     ADMUX = 0x06; break;

7

>

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