|
Anzeige
|
ADC-Messung an mehreren Ports
Gast
#1388955
Hallo Leute, ich versuche gerade an meinem atmega168 abwechselnd die Ports ADC6 und ADC7 einzulesen. So aktiviere ich das ganze:
und so versuche ich das Ganze zuzuweisen:
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! 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 ;-)
Gast
#1388982
Oh ja, vielen Dank!!! Ich ging davon aus er setzt mir die Bits so wie ich sie angebe. Da fehlt mir wohl die Praxis ;-) 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:
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: >
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
Antwort schreiben |
Anzeige
|
