ATMEGA8 - 2 analoge Signale einlesen (2-Ch AD-Wandler)

>   ADMUX |= 0b01100000;    // Einlesen des Kanals 0 usw.

Hier habe ich beim Lesen kapituliert.

Krapao schrieb:
>>   ADMUX |= 0b01100000;    // Einlesen des Kanals 0 usw.
>
> Hier habe ich beim Lesen kapituliert.

Ja, sorry für die schlechte Beschreibung. Ja die letzten vier Bits 
beziehen sich auf den Anschluss zum Poti. Also MUX0-MUX3. Und da alle 
vier Bits Null sind ist es ja der Anschluß ADC0 des µC. War für mich ne 
Bezeichnung wo ich das Kabel anzuschließen habe.

Durch die absoluten Bitmasken ist es sau schwer direkt zu sehen, ob dein 
Code richtig ist.

Mit den symbolischen Bitnamen aus den Includefiles ist es wesentlich 
einfacher zu sehen, ob der Code dem bekannten Code z.B. im 
AVR-GCC-Tutorial entspricht.

Deinen Code kann man nur mit intensivem Lesen des Atmega8 Datenblatts 
überprüfen.

Bei der Stelle

>  while (ADCSRA &0b01100000)  // Überprüfung: Abschluss der Wandlung
>  {}

ist was faul. Ich erwarte da eine Bitmaske mit einem gesetzten Bit, 
nicht zwei wie in deinem Code.

Ich würde auch die bewährte Auftrennung in die einmalige 
ADC-Initialisierung und ggf. regelmäßige ADC-Kanalumschaltung/Auslesen 
beibehalten, so wie sie im Tutorial verwendet wird.

Hi, das wird nicht funktionieren:
> while (ADCSRA &0b01100000)

Du meinst sicherlich:

1

while (ADCSRA &(1<<ADSC));

Grüße,
Ingo

Hubert G. schrieb:
> Wenn du ADMUX|=(1<<REFS0)|(1<<ADLAR); schreibst dann muss sich der Leser
> nicht das Datenblatt heraussuchen um zu verstehen was du meinst.
> Warum aktivierst du das immer wieder, es genügt doch einmal im main.

Ah okay, wird diese Art der Schreibweise "(1<<REFS0)|(1<<ADLAR)" als 
"symbolische Bitnamen" benannt? Hab davor gedacht die absolute 
Schreibweise ist eindeutiger. Ich wollte damit nur ausschließen, dass 
mir durch das Kompilieren oder durch andere Programme die Register 
verändert werden.

Danke für die vielen Hinweise. Werde mich dann jetzt erst nochmals mit 
der Kanalumschaltung ausführlich befassen.

Hallo Freunde,

ich möchte gerne mit einem Atmega16 zwei verschiedene Analogsignale an 
den Pins PA0 und PA1 abtasten.
Leider bin ich noch Anfänger und muss noch viel herum probieren und 
lesen dazu. Aber jetzt komme ich wirklich nicht mehr weiter.
Vielleicht kann mir ja mal einer einen Tip geben und sagen, was an dem 
unten aufgeführten Code nicht richtig ist. Wenn ich nur über den ADC PA0 
einlese funktioniert es, was ich dann über das PWM TImer1 auch sauber 
angezeigt bekomme. Möchte ich jedoch beide Pins abtasten, reagiert er 
wieder nur auf PA0 und lässt PA1 scheinbar außer acht.

Danke im Voraus.
Jens






#include <avr/io.h>
#ifndef F_CPU
#define F_CPU 4000000UL
#define ADATE 5
#endif
#include <util/delay.h>



int main(void)
{

uint16_t wert1;
uint16_t wert2;


  DDRB |= (1 << DDB3);      // Setup PB0 as output

  while(1)
  {

//erstes Abtasten

  ADMUX |= (1<<REFS0);
  ADCSRA |= (1<<ADEN)|(1<<ADATE)|(1<<ADPS2)|(1<<ADSC);

  while(!(ADCSRA&(1<<ADSC))); // Wait until a conversion is finished

  wert1 = ADC;

//zweites Abtasten

  ADMUX |= (1<<REFS1);
        ADCSRA |= (1<<ADEN)|(1<<ADATE)|(1<<ADPS2)|(1<<ADSC);

  while(!(ADCSRA&(1<<ADSC))); // Wait until a conversion is finished

  wert2 = ADC;

//über PWM Timer1 ausgeben


  DDRD |= (1 << DDB4);
  DDRD |= (1 << DDB5);


  TCCR1A = (1<<WGM10)|(1<<WGM11)|(1<<COM1A1)|(1<<COM1B1);

  TCCR1B = (1<<WGM12)|(1<<CS11)|(1<<CS10);

  OCR1A = wert1;
  OCR1B = wert2;


  }

  return 0;

}

Hi

>//erstes Abtasten
>  ADMUX |= (1<<REFS0);

>//erstes Abtasten
>  ADMUX |= (1<<REFS0);

Damit schaltest du die Referenzspannungsquelle aber nicht die Kanäle um. 
Außerdem konfiguriert man den ADC nicht ständig in der while-Schleife. 
das macht man einmal davor. In der Schleife brauchst du nur die 
ADMUX-Bits und ADSC anfassen.

MfG spess

Max fragte:
>Was war an diesem Hienweis nicht verständlich?

Wo ist Deine Hilfe? Ich sehe sie nicht.
Bist Du einer der Typen, die den TO, wenn er wirklich einen neuen 
Beitrag
verfasst hätte, auch nur auf die Suchfunktion hinweisen und nicht helfen
wollen oder können?

Ist es Deiner Meinung nach sinnvoll, 10 gleichartige Threads mit nicht
Einer Lösung des Problemes zu haben?

Das mußte mal raus....

Sorry, ich hatte das verteilt. Ja ich werd dazu was komplett neues 
erstellen.

Verpeilt meinte ich.

@ Max H. (hartl192)
was lässt du dich als registrierter User hier von so einem Blockwart 
denunzieren? setz den auf die Ignoliste und gut ist,
oder suche seine Beiträge hier, der kann nur Rummosern - fachl. 
Tiefflieger

Jens schrieb:
> Sorry, ich hatte das verteilt. Ja ich werd dazu was komplett neues
> erstellen.

Dann schreib doch gleich den Link - wer soll sich da sonst noch zurecht 
finden.

Oder ein Mod erbarmt sich und löscht hier den ganzen, an den alten 
Thread rangeklatschten Mist.