ADC mit mega32 funktioniert nicht

[S. L. (slehner)]

Hallo alle zusammen,

seit etwa 3 Monaten programmiere ich jetzt den mega32 und habe soweit 
alle Funktionen in Betrieb nehmen können dich ich brauche (I2C, UART, 
PORTS) Nun möchte ich über den ADC gerne eine Spannung einlesen. Im 
Internet habe ich bereits auch ein beispiel gefunden und teilweise etwas 
angepasst. Ich poste euch erstmal den Code der Funktion und dann den 
Programmteil, der diese aufruft. Der Wert soll über Port C ausgegeben 
werden. (oder testweise und die unteren 8 Bit. Es wäre super wenn mir 
jemand helfen könnte und nen Fehler in dem Code findet.

Funktion:

uint16_t ReadChannel(uint8_t mux)
{
  uint8_t i;
  uint16_t result = 0; //Initialisieren wichtig, da lokale Variablen
  uint8_t temp;
  
  ADCSRA = (1<<ADEN) | (1<<ADPS1) | (1<<ADPS0);   // requenzvorteiler
                          // setzen auf 8 (1) und ADC aktivieren (1)
  ADMUX = mux; // Kanal waehlen
  //ADMUX |= (1<<REFS1) | (1<<REFS0); // interne Referenzspannung nutzen
  ADMUX |= (1<<REFS0);
  /* 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

  while ( ADCSRA & (1<<ADSC) )   {
                  //konvertierung abwarten
                  }


  /* Eigentliche Messung - Mittelwert aus 4 aufeinanderfolgenden Wandlungen */
  for(i=0;i<4;i++)
  {
    ADCSRA |= (1<<ADSC); // eine Wandlung "single conversion"
    while ( ADCSRA & (1<<ADSC) ) 
    {
    ; // auf Abschluss der Konvertierung warten
    }

    result += ADCL; // Wandlungsergebnisse aufaddieren
    temp = ADCH;
  }
  ADCSRA &= ~(1<<ADEN); // ADC deaktivieren (2)
  result /= 4; // Summe durch vier teilen = arithm. Mittelwert
  return result;
}

aufrufender Programmteil:

uint16_t adcval;
  uint8_t ausgabe;
  for(;;){
    adcval = ReadChannel(0); /* MUX-Bits auf 0b0000 -> Channel 0 */
    //adcval = ReadChannel(2); /* MUX-Bits auf 0b0010 -> Channel 2 */
    DDRB = 0xFF;
    ausgabe = adcval | 0xFF;
    PORTB = ausgabe;
    delayloop16(16000);
    }

[yalu (Gast)]

>   ausgabe = adcval | 0xFF;

Sollte da statt des | nicht ein & stehen? Sonst wird immer 0xff an den
Port gegeben.

[Joerg X. (Gast)]

...oder ein "^", um die Bits umzudrehen, falls es um low-active-LEDs 
geht
(Und war da nicht was mit PORTC und JTAG...)
hth. Jörg

[S. L. (slehner)]

Die Wandlung vom ADC funktioniert. Das Problem war ein anderen, nähmlich 
die Ausgabe. Ich möchte die 16bit Zahl ausgeben und dabei macht er 
verrückte Sachen. Ich glaube dass ich die zahl nicht anständig ich 8 bit 
Variablen aufteilen kann. Weiß jemand wie man das macht? Ich habe es nun 
mit folgendem probiert und es klappt nicht(Die Ausgabe über die serielle 
Schnittstelle funktioniert mit bytes bereits):

    zahl = AD_in();      // uint_16
    unsigned char tausender = (unsigned char) (zahl/1000) & 0x00FF;
    unsigned char hunderter = (unsigned char) (mod(zahl,1000)/100) & 0x00FF;
    unsigned char zehner = (unsigned char) (mod(zahl,100)/10)& 0x00FF;
    unsigned char einer = (unsigned char) mod(zahl,10) & 0x00FF;
    //PORTB = zahl_l;
    
    USART_transmit(tausender+48);
    
    USART_transmit(hunderter+48);
    USART_transmit(zehner+48);
    USART_transmit(einer+48);
    USART_transmit(45);// -

[yalu (Gast)]

Ich kann in dem geposteten Programmteil keinen Fehler erkennen. Der
Fehler liegt entweder in USART_transmit (aber du schreibst, dass die
Ausgabe einzelner Zeichen schon funktioniert) oder die Funktion mod
tut nicht das Erwartete.

Was wird denn verrücktes ausgegeben? Wird wenigstens das '-' am Ende
angezeigt?

Woher kommt überhaupt die mod-Funktion? Hast du die selbst
geschrieben? Sonst gibt es für solche Dinge in C auch den
Modulooperator '%'.

Die "& 0x00FF" kannst du dir sparen. Sie haben keinen Effekt.

[S. L. (slehner)]

danke für die Idee mit dem % Operator, den hab ich ganz vergessen.


Die Ausgabe funktioniert auch inzwischen. Der Fehler lag darin, dass ich 
die Stromversorgung des Atmels vor dem UART Anschluss angeschlossen habe 
und dadurch das Hyperterminal mitten im Byte angefangen hat das die 
Zeichenkette aufzunehmen. Deshalb wurden auch wirre Zeichen ausgegeben

[Oliver (Gast)]

????

Das muß man jetzt aber nicht verstehen, oder?

Falls du das meinst, was ich glaube, zu verstehen (aber eigentlich nicht 
glauben kann): Ein Reset-Taster kostet nur ein paar Cent, und ist Gold 
wert.

Oliver

[S. L. (slehner)]

Ja Oliver das stimmt. Es ist ein Projekt von 3 Leuten und ich 
programmiere den Atmel nur. Die Elektronik mach ich nicht. Mit wäre es 
auch lieber gewesen nicht bei jedem überspielen den Programmes den 
Stromstecker ziehen zu müssen und anschließend den Programmerstecker 
abziehen zu müssen. (Von der Masse des Oszis ganz zu schweigen) Aber das 
war dem Elektroniker zu viel Arbeit...