Atmega2560, ADC Wert auslesen und versenden

[GS (chromosoma)]

Abend. Im Moment versuche ich den ADC Wert an ADCO auszulesen und via 
USART an PC  zu versenden. Das versenden von Daten per USART 
funktioniert, aber ich habe noch Promleme mit ADC.
Irgendwie funktioniert es nicht. Hier ist meine Code
Die SerialMonitor bekommt immer  <NULL>
 zugesendet.

Wasmache ich falsch?:)

#include <avr/io.h>
#define F_CPU 16000000
#define USART_BAUDRATE 9600
#define BAUD_PRESCALE (F_CPU / 16 / USART_BAUDRATE )

int main (void)
{
  int Res;
  DDRF = 0<<PF0;
  PORTF=1<<PF0;
 ADCSRA |= (1 << ADPS2) | (1 << ADPS1) | (1 << ADPS0); // Set ADC prescaler to 128 - 125KHz sample rate @ 16MHz

 ADMUX |= (1<< REFS0);
  // Set ADC reference to AVCC
 ADMUX |= (1 << ADLAR); // Left adjust ADC result to allow easy 8 bit reading

 // No MUX values needed to be changed to use ADC0


 ADCSRA |= (1 << ADEN);  // Enable ADC
 ADCSRA |= (1 << ADSC);  // Start A2D Conversions
  UCSR0B |= (1 << RXEN0) | (1 << TXEN0);   // Turn on the transmission and reception circuitry
  UCSR0C |= (0<<UMSEL00)|(1 << UCSZ00) | (1 << UCSZ01); // Use 8-bit character sizes

   // Load upper 8-bits of the baud rate value into the high byte of the UBRR register
  UBRR0 = BAUD_PRESCALE; 
  for (;;) // Loop forever
  {
    Res = ADCL;
      
      UDR0= Res;
    
    
  
   }   
}

[troll (Gast)]

Vielleicht solltest du vorm Auslesen von ADCL darauf warten das die 
Conversion abgeschlossen ist?

[Stefan E. (sternst)]

Cross-Posting:
http://www.avrfreaks.net/index.php?name=PNphpBB2&file=viewtopic&t=124944

[Eumel (Gast)]

Stefan, vielleicht ist deine Nähe noch ne Stelle als Blockwart frei. 
Schau doch mal nach.

[Stefan E. (sternst)]

Eumel schrieb:
> Stefan, vielleicht ist deine Nähe noch ne Stelle als Blockwart frei.
> Schau doch mal nach.

Es geht darum, die hilfsbereiten Menschen hier davor zu bewahren, ihre 
Zeit damit zu verschwenden etwas zu schreiben, was dort längst 
abgehandelt wurde.

[GS (chromosoma)]

troll schrieb:
> Vielleicht solltest du vorm Auslesen von ADCL darauf warten das die
> Conversion abgeschlossen ist?


hm..vielleicht:)




To Stefan.
Ja, CrossPosting erhöht die Wahrscheinlichkeit einer Lösung:)

[Stefan E. (sternst)]

Anton R. schrieb:
> Ja, CrossPosting erhöht die Wahrscheinlichkeit einer Lösung:)

Cross-Posting ist eine große Respektlosigkeit gegenüber jenen, von denen 
du dir Hilfe erwartest. Du sagst mit einem Cross-Posting nämlich nichts 
anderes, als:
"Ist mir doch egal, ob ihr möglicherweise eure Zeit mit Antworten 
verschwendet, die anderswo längst gegeben wurden. Mir ist nur wichtig, 
dass ich möglichst schnell eine Lösung bekomme."

[GS (chromosoma)]

Mensch, beruhige dich.
Die Lösung wurde noch nicht gefunden, und falls es eine Lösung geben 
wird, werde ich  auch hier/bei AVRfreaks bescheid sagen.
Und ja, je mehr Menschen  an mein Problem arbeiten, desto schneller wird 
es gelöst, aber ich zwinge ja keinen mir zu helfen, alles freiwillig 
hier.

[Thomas E. (thomase)]

Anton R. schrieb:
> Die Lösung wurde noch nicht gefunden,
Nicht?
Die steht doch da:
troll schrieb:
> Vielleicht solltest du vorm Auslesen von ADCL darauf warten das die
> Conversion abgeschlossen ist?
ergänzen könnte man noch:...und der UART fertig ist.

Oder was erwartest du von so einem Scheiß:
Anton R. schrieb:
> for (;;) // Loop forever
>
>   {
>     Res = ADCL;
>       UDR0= Res;
>    }

mfg.

[Karl H. (kbuchegg)]

Anton R. schrieb:

> Ja, CrossPosting erhöht die Wahrscheinlichkeit einer Lösung:)

Ohne respektlos sein zu wollen, aber die Lösung liegt eher darin, zu 
lernen. Wenn ich mir ansehen, welcher Müll auf AVR-Freaks geschrieben 
wurde, dann kann ich nur auf das AVR-GCC-Tutorial verweisen. Und 
zwar sowohl beim Thema UART als auch beim Thema ADC.

[GS (chromosoma)]

Okey.
nach einigen tutorials und beispielen sieht meine Code jetzt so aus.
Aber es funktioniert trotzdem nicht, wo ist den das problem...?

 #include <avr/io.h>
#include <util/delay.h>
#define F_CPU 16000000
#define USART_BAUDRATE 115200
#define BAUD_PRESCALE (F_CPU / 16 / USART_BAUDRATE )
uint8_t ReadADC(){
      
      ADCSRA|= (1<<ADSC);

      //Wait for conversion to complete
      while (!(ADCSRA & (1<<ADIF)));

      // Clear bit ADIF
      ADCSRA|= (1<<ADIF);

      return (ADCH);
   }   
   void USARTWriteChar(unsigned char data)
   {
      // Wait until the transmitter is ready
      while(!(UCSR0A &(1<<UDRE0)))
      {
         // Do nothing
      }

      // Write data to USART buffer
      UDR0= data;
   }
int main (void)
{
   

unsigned char data;

   DDRE = 1<<PE4;
   DDRF = 0<<PF0;
   
 ADCSRA |= (1 << ADPS2) | (1 << ADPS1) | (1 << ADPS0); // Set ADC prescaler to 128 - 125KHz sample rate @ 16MHz

 ADMUX |= (1<< REFS0);
  // Set ADC reference to AVCC
 ADMUX |= (1 << ADLAR); // Left adjust ADC result to allow easy 8 bit reading

 // No MUX values needed to be changed to use ADC0
ADCSRA |=1<<ADATE;
 ADCSRA |= (1 << ADEN);  // Enable ADC
   UCSR0B |= (1 << RXEN0) | (1 << TXEN0);   // Turn on the transmission and reception circuitry
   UCSR0C |= (0<<UMSEL00)|(1 << UCSZ00) | (1 << UCSZ01); // Use 8-bit character sizes
// Load upper 8-bits of the baud rate value into the high byte of the UBRR register
   UBRR0 = BAUD_PRESCALE;
   for (;;){



data = ReadADC();

USARTWriteChar(data);
}
}