ADC Programmcode! Richtig?

[Sebastian (Gast)]

#include <avr/io.h> 


/* ADC initialisieren */
void ADC_Init(void) {
 
  uint8_t i;
  uint16_t result;                
 
  ADMUX = (1<<REFS0);              // Referenzspannug= intern/Avcc
  ADMUX = (1<<REFS1) | (1<<REFS0);      // interne Referenzspannung nutzen
  ADCSRA = (1<<ADPS1) | (1<<ADPS0);     // Frequenzvorteiler = 32
  ADCSRA |= (1<<ADEN);                  // ADC wird aktiviert
 
  /* nach Aktivieren des ADC liest man einen Wert ein 
  und verwirft diesen, um den ADC "warmlaufen zu lassen" */
 
  ADCSRA |= (1<<ADSC);                  // eine ADC-Wandlung 
  while (ADCSRA & (1<<ADSC) ) {}        // auf Abschluss der Konvertierung warten
  
  result = ADCW;            //gemessenen Werte werden hier gespeichert 

//Eigentliches Programm startet

  uint16_t ADC_Read( uint8_t MUX4 )

  result = 0; 
  for( i=0; i<4; i++ )
  {
    ADCSRA |= (1<<ADSC);            // eine Wandlung "single conversion"
    while ( ADCSRA & (1<<ADSC) ) {
      ;   // auf Abschluss der Konvertierung warten
    }
    result += ADCW;        // Wandlungsergebnisse aufaddieren
  }
  ADCSRA &= ~(1<<ADEN);             // ADC deaktivieren (2)
 
  result /= 4;                     // Summe durch vier teilen = arithm. Mittelwert
 
  return result;
}

Ich möchte von ADC6 (PA6) den Wert meines Temperatur Sensors einlesen.
Mein Atmel läuft mit Externer ACC von +5V, mit 16 Mhz. Nun hab ich den 
Teilungsfaktor 32 Eingestellt, sowie auch die Referenzspannung auf ACC 
eingestellt, stimmt das in etwa?

Ist das mit dem " uint16_t ADC_Read( uint8_t MUX4 )" richtig oder muss 
das raus?

Danke im vorraus.

[Jörg W. (dl8dtl) (Moderator)]

Sebastian schrieb:

>   ADMUX = (1<<REFS0);              // Referenzspannug= intern/Avcc
>   ADMUX = (1<<REFS1) | (1<<REFS0);      // interne Referenzspannung

Die zweite Zeile hebt die Zuweisung der vorherigen auf.

>   ADCSRA = (1<<ADPS1) | (1<<ADPS0);     // Frequenzvorteiler = 32
>   ADCSRA |= (1<<ADEN);                  // ADC wird aktiviert

Kann in eine einzige Zeile 'rein.

>   uint16_t ADC_Read( uint8_t MUX4 )

Sieht aus wie eine Funktionsdefinition (allerdings innerhalb einer
Funktion, könnte GCC theoretisch unterstützen, ist aber hier sicher
nicht sinnvoll), aber dann müsste danach eine öffnende geschweifte
Klammer mit dem Inhalt der Funktion kommen.

Compiliert der GCC dir das denn?

> Ich möchte von ADC6 (PA6) den Wert meines Temperatur Sensors einlesen.

Wo stellst du denn im ADMUX diesen Kanal ein?

> Nun hab ich den
> Teilungsfaktor 32 Eingestellt, sowie auch die Referenzspannung auf ACC
> eingestellt, stimmt das in etwa?

Kann sein, muss nicht.  Habe jetzt die REFSn-Bits nicht nachgelesen,
ob du damit AVcc wirklich ausgewählt hast, aber lesen wirst du ja
können. ;-)

Der ADC soll vorrangig mit 50 ... 200 kHz betrieben werden, bei einem
Teilerfaktor von 32 müsste dein CPU-Takt also irgendwo zwischen 1,6
und 6,4 MHz liegen.

[Lutz (Gast)]

Sebastian schrieb:
> while (ADCSRA & (1<<ADSC) ) {}        // auf Abschluss der Konvertierung warten

Unabhängig vom dort fehlenden Semikolon: Wer soll zum unbekannten 
Prozessor was sagen können? Oder was macht uint16_t ADC_Read( uint8_t 
MUX4 ) wann und wie?

[Karl H. (kbuchegg)]

Sebastian schrieb:

> Ich möchte von ADC6 (PA6) ....

Du möchtest dir vor allen Dingen als erstes einmal ein C Buch kaufen und 
da die ersten paar Kapitel durcharbeiten (nicht durchlesen ... 
durcharbeiten!)

[Sebastian (Gast)]

Anfänger werden hier standardmäßig verarscht oder rausgemobbt!!!

Es ist noch kein Meister vom Himmel gefallen.

So ein neuer Code mit 20 Mhz:

#include <avr/io.h> 


/* ADC initialisieren */
uint16_t ADC_Init(void) {
 
  uint8_t i;
  uint16_t result;                
 
  ADMUX  = 0x06;               //Kanal 6 gewählt 
  ADMUX  = (1<<REFS1) | (1<<REFS0);        // interne Referenzspannung nutzen
  ADCSRA = (1<<ADPS1) | (1<<ADPS0)|(1<<ADPS2);       // Frequenzvorteiler = 128
  ADCSRA|= (1<<ADEN);                    // ADC wird aktiviert
 
  /* nach Aktivieren des ADC liest man einen Wert ein 
  und verwirft diesen, um den ADC "warmlaufen zu lassen" */
 
  ADCSRA |= (1<<ADSC);                  // eine ADC-Wandlung 
  while (ADCSRA & (1<<ADSC) ) {}        // auf Abschluss der Konvertierung warten
  
  result = ADCW;            //gemessene Werte werden hier gespeichert 

//Eigentliches Programm startet

  
  result = 0; 
  for( i=0; i<4; i++ )
  {
    ADCSRA |= (1<<ADSC);            // eine Wandlung "single conversion"
    while ( ADCSRA & (1<<ADSC) ) {
      ;   // auf Abschluss der Konvertierung warten
    }
    result += ADCW;        // Wandlungsergebnisse aufaddieren
  }

 
  result /= 4;                     // Summe durch vier teilen = Mittelwert
 
  return result;
}

[holger (Gast)]

>  ADMUX  = 0x06;               //Kanal 6 gewählt
>  ADMUX  = (1<<REFS1) | (1<<REFS0);        // interne Referenzspannung nutzen

Tja, und damit ist Kanal 6 wieder tot.

>Anfänger werden hier standardmäßig verarscht oder rausgemobbt!!!

Dann geh doch woanders hin.

[Helfer (Gast)]

Für die Anfänger wurde das AVR-GCC-Tutorial geschrieben und da sind die 
ADC Funktionen als getesteter Code inkl. Beispielanwendung enthalten. 
Wenn Anfänger ohne Anmerkungen oder Rücksicht auf das AVR-GCC-Tutorial 
hier eigenen Code präsentiert, muss er damit rechnen, dass der ohne 
Rücksicht auseinander genommen wird.

Bei deinem Code sieht man nicht, wie er aufgerufen wird, d.h. die 
Funktion ist per se unprüfbar. Und es macht C-typisch (1. Code) und 
programmlogisch (2. Code) keinen Sinn die Messfunktionalität in der 
Initialisierung zu implementieren. Der Hinweis auf "C-Lernen!" ist also 
nicht unberechtigt bzw. keine Verarsche oder Mobbing.

[Karl H. (kbuchegg)]

Sebastian schrieb:
> Anfänger werden hier standardmäßig verarscht oder rausgemobbt!!!
>
> Es ist noch kein Meister vom Himmel gefallen.
>


Das hat mit Meister vom Himmel fallen nichts zu tun.
Schon eher mit der Überheblichkeit mancher Anfänger und der Denkweise, 
dass sie die Grundlagen nicht lernen müssten.

Wenn du in einem Physikforum nach der Relativitätstheorie fragst und 
beireits beim Hebelgesetz aussteigst, wird man dir auch erst mal ein 
Physikbuch auf Einsteigerniveau nahelegen.

Ein Programm zu schreiben ist schwieriger als alles andere, was du 
bisher in deinem Leben gemacht hast. Gott sei Dank wissen das auch viele 
Leute und haben all die großen und kleinen Details in Büchern 
geschrieben. Die haben im Falle von C nicht selten mehr als 200 Seiten, 
von denen du die ersten 50 bis 70 intus haben solltest, ehe du dann ins 
Lager der µC Programmierer wechseln kannst.

[Karl H. (kbuchegg)]

Helfer schrieb:

> Bei deinem Code sieht man nicht, wie er aufgerufen wird,


Ich hege den Verdacht, dass Sebastian der Ansicht ist, dass wäre bereits 
alles was notwendig ist. Also: Das wäre das komplette vollständige 
Programm.

[Jörg W. (dl8dtl) (Moderator)]

Lutz schrieb:
> Sebastian schrieb:
>> while (ADCSRA & (1<<ADSC) ) {}        // auf Abschluss der Konvertierung warten
>
> Unabhängig vom dort fehlenden Semikolon:

Nö, das fehlt nun gerade nicht, zwei geschweifte Klammern bilden sehr
wohl eine leere Anweisung.

Aber, paar Zeilen später:

    while ( ADCSRA & (1<<ADSC) ) {
      ;   // auf Abschluss der Konvertierung warten
    }

Sieht aus wie copy, paste'n die.  Wenn man sich für eine bestimmte
Ausdrucksweise entschieden hat, sollte man diese innerhalb eines
Werks auch konsistent durchziehen, statt dasselbe immer wieder anders
aufzuschreiben.

[Frank S. (franksanderdo)]

Helfer schrieb:
> Wenn Anfänger ohne Anmerkungen oder Rücksicht auf das AVR-GCC-Tutorial
> hier eigenen Code präsentiert, muss er damit rechnen, dass der ohne
> Rücksicht auseinander genommen wird.

Auch wenn ich die Ansicht unterstütze das Tutorials gelesen werden 
sollen:

In keinem Umfeld sollte eine Person ohne Rücksicht auseinander genommen 
werden!!

Grüße
Frank

[Helfer (Gast)]

Das der in dem Satz bezieht sich auf den Code und nicht auf die 
Person. Bzgl. der Person gebe ich dir Frank uneingeschränkt Recht.

[Frank S. (franksanderdo)]

Hallo Helfer,

sorry dann habe ich das falsch verstanden.

Grüße
Frank