ADC to UART Problem C/C++

[Max K. (max_blanket)]

Hey Leute,

ich habe ein Problem mit dem ADC meines AtMega2560.
Ich will ein analoges Signal am Kanal 0 des ADC einlesen und den Wert 
über UART an ein PC-Terminal zurückgeben.
UART läuft ohne Probleme mit einzelnen Zeichen, Zeichenketten oder 
variablen Inhalten. Also das Problem der Übertragung muss schon irgendwo 
im ADC liegen.
Die Konfiguration habe ich nach Tutorials die hier rumfliegen gemacht 
und da "sollten" keine Fehler drin sein. Da lass ich mich aber gerne 
belehren. Ich habe schon 3 Leute drüberschauen lassen, die aber alle 
nichts gefunden haben. (Naja, waren leider alle keine guten 
Programmierer, so wie ich^^)

Ich hoffe jemand hat mal Zeit drüberzuschauen und sieht vielleicht 
meinen Fehler.


Hier mein Code

////////////////////////////////////////////////////////////
///////////////////////Vordefinitionen//////////////////////
////////////////////////////////////////////////////////////
#define F_CPU 16000000    // Systemtakt in Hz

#define BAUD 9600      // Baudrate

#include <avr/io.h>
#include <inttypes.h>
#include <util/delay.h>
#include <stdio.h>
#include <stdlib.h>
#include <util/setbaud.h>

///////////////////////////////////////////////////////////
/////Deklarieren aller Funktionen//////////////////////////
void usart_init();
int usart_send(unsigned char zeichen);
void usart_sput(char *string);
void usart_var(int x);
///////////////////////////////////////////////////////////
void ADC_Init(void);
uint16_t ADC_Read(uint8_t channel);
uint16_t ADC_Read_Avg(uint8_t channel, uint8_t average);
///////////////////////////////////////////////////////////

int main() {
  
  _delay_ms(2000);
  _delay_ms(2000);
  
  usart_init();
  
  //uint16_t adcval;
  uint8_t channel = 0;

  ADC_Init();
    
  char text1[5] = "test";
  char text2[2] = "!";
  
  uint16_t x = ADC_Read(channel);
  
  char text3[8] = {};
  char text4[8] = {};
  
  usart_sput(text1);
  usart_sput(text2);  
  _delay_ms(1000);
  
        while(1)
        {
      *text3 = (uint8_t)x;
            *text4 = (uint8_t)(x>>8);
                        usart_sput(text3);
                        usart_sput("-");
      usart_sput(text4);
                        usart_sput("_");
      _delay_ms(1000);
        }
    
  return 0;
}


//USART-Schnittstelle konfigurieren
void usart_init()
{
  
  //Baudrate einstellen in Baudraten-Register
  UBRR0H = UBRRH_VALUE;
  UBRR0L = UBRRL_VALUE;
  
  #if USE_2X
  // U2X-Modus erforderlich
  UCSR0A |= (1 << U2X0);
  #else
  // U2X-Modus nicht erforderlich
  UCSR0A &= ~(1 << U2X0);
  #endif
  
  //Enable Recceiver and Transmitter
  UCSR0B = (1<<TXEN0);
  
  //Einstellen des Frame-Formates auf 8data-, 1stop-bits
  UCSR0C = (0<<USBS0) | (1<<UCSZ00) | (1<<UCSZ01);
}


//Übergeben einzelner Zeichen des Strings
void usart_sput(char *string)
{
  while(*string)
  {
    usart_send(*string);
    string++;
  }  
}


//Ausgeben einzelner Zeichen
int usart_send(unsigned char zeichen)
{
  while (!(UCSR0A & (1<<UDRE0)))    // warten bis Senden moeglich ist
{}
  UDR0 = zeichen;
  
  return 0;
}


//ADC initialisieren
void ADC_Init(void) {
 
  uint16_t result;
 
  ADMUX = (1<<REFS0);      // interne Referenzspannung nutzen
  ADCSRA = (1<<ADPS1) | (1<<ADPS0);     // Frequenzvorteiler

 
  /* nach Aktivieren des ADC wird ein "Dummy-Readout" empfohlen, man liest
     also einen Wert und verwirft diesen, um den ADC "warmlaufen zu lassen" */
 
 
  ADCSRA |= (1<<ADSC);                // eine ADC-Wandlung 
    ADCSRA |= (1<<ADEN);                  // ADC aktivieren
  
  while (ADCSRA & (1<<ADSC) ) {}      // auf Abschluss der Konvertierung warten
  /* ADCW muss einmal gelesen werden, sonst wird Ergebnis der nächsten
     Wandlung nicht übernommen. */
 
  result = ADCL;
  result = (8<<ADCH);
}


//ADC Einzelmessung
uint16_t ADC_Read(uint8_t channel)
{
  //ADCSRA |= (1<<ADEN);  
  // Kanal waehlen, ohne andere Bits zu beeinflußen
  ADMUX = (ADMUX & ~(0x1F)) | (channel & 0x1F);
  ADCSRA |= (1<<ADSC);            //eine Wandlung "single conversion"
  while (ADCSRA & (1<<ADSC)){}    //auf Abschluss der Konvertierung warten
  
  return ADCW;                    //ADC auslesen und zurückgeben
}

 
//ADC Mehrfachmessung mit Mittelwertbbildung
uint16_t ADC_Read_Avg( uint8_t channel, uint8_t average )
{
  uint32_t result = 0;
 
  for (uint8_t i = 0; i < average; ++i )
    result += ADC_Read( channel );
 
  return (uint16_t)( result / average );
}

Danke für eure Zeit.

MfG
Max

[Karl H. (kbuchegg)]

Max K. schrieb:

Und was 'genau' ist jetzt eigentlich dein Problem?

> belehren. Ich habe schon 3 Leute drüberschauen lassen, die aber alle
> nichts gefunden haben. (Naja, waren leider alle keine guten
> Programmierer, so wie ich^^)

das müssen aber miserable Programmierer sein, wenn sie die Funktion utoa 
nicht kennen und keiner gesehen hat, dass das hier ...

>       *text3 = (uint8_t)x;
>             *text4 = (uint8_t)(x>>8);
>                         usart_sput(text3);
>                         usart_sput("-");
>       usart_sput(text4);
>                         usart_sput("_");

... ausser Unsinn nur noch Unsinn ist.
Entweder du gibst die Bytes direkt aus, dann ist aber usart_sput die 
offenbar falsche Funktion dafür. Oder aber du willst auf dem Terminal 
den Zahlenwert in Textform sehen, dann musst du dir erst mal einen 
String generieren lassen, der den Zahlenwert in x in Textform beschreibt

         utoa( x, text3, 10 );
         usart_sput( text3 );
         usart_sput( " " );

>       _delay_ms(1000);
>         }


Ausserdem wird es eher hilfreich sein, wenn du dann innerhalb der 
Schleife den ADC immer wieder abfrägst. Sonst ändert sich der WErt von x 
so schlecht, wenn sich die externe Spannung ändert.

[Max K. (max_blanket)]

Erstmal dakne für deine schnelle Antwort.

Ja, es sind alles blutige Anfänger wie ich, aber ich habe den 
Mikrocontroller vor ner Woche erhalten und hab die Aufgabe gleich 
aufgehalst bekommen.

Ich bin nach deiner Beschreibung vorgegangen und hat alles geklappt.

Die Abfrage des ADC-Wertes außerhalb der Schleife ist natürlich sehr 
doof. Ist mir das peinlich. -.-

Vielen Dank für deine Hilfe.

PS:Wo darf ich Blumen+Pralinen oder den Kasten Bier hinschicken?^^