ADC Werte ändern sich nicht

[zwergenstadt (Gast)]

Hallo zusammen,
nachdem ich seit kurzen Assembler programmiere wollte ich jetzt mal das 
ganze in C versuchen. Testumgebung ist ein Atmega8. Jetzt habe ich aber 
das Problem, dass mein ADC sehr komische Werte leifert. Ich will dabei 
nur das High bit haben. In Assembler hat das wunderbar funktioniert und 
in C überhaupt nicht...
Der ADC pendelt sich einfach sofort auf 84/85 ein und ändert den Wert 
nicht mehr, egal welche Quelle als Eingang, egal wie sich das Signal 
ändert. Zuerst Low dann High auslesen hat nicht funktioniert, es mit 
ADCW auslesen auch nicht, mit und ohne Dummy, Mittelwert, An/Abschalten, 
Warten usw alles erfolglos. Werte lass ich mir über den UART ausgeben 
(wird im Bootloader initialisiert). Hat jemand noch einen Tipp, was an 
dem C Code falsch ist?

#define F_CPU 8000000UL //define 8MHz

#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>

void myinit();
uint8_t get_adc_value();
void send(uint8_t);



int main(void)
{
  myinit();
  uint8_t left_adcvalue_on = 0;
  ADMUX |= (1 << REFS0)|(1 << MUX1) |(1 << MUX0)  |(1 << ADLAR); //set left to adc
  PORTD |= (1<<PD6); //switch led on
  
    while(1)
    {
    _delay_ms(500);
    left_adcvalue_on = get_adc_value();
    send(left_adcvalue_on);    
    }
}

uint8_t get_adc_value()
{
  uint8_t i;
  uint16_t result = 0;
  ADCSRA |= (1 << ADEN); //enable adc
  
  //dummy readout
  ADCSRA |= (1<<ADSC);
  while(ADCSRA & (1<<ADSC));

  //read real value
  for(i=0; i<3; i++) 
  {
    ADCSRA |= (1<<ADSC);
    while(ADCSRA & (1<<ADSC));
    result += ADCH;
  }
  result /= 3  ;
  
  ADCSRA &= ~(1<<ADEN); //disable adc
  return result;
}

void send(uint8_t send_this)
{
    while (!(UCSRA & (1<<UDRE))) ;  //wait until you can send
    UDR = send_this;
  return;
}

void myinit()
{
  //init leds
  DDRD |= (1<<PD7)| (1<<PD6);
  DDRC |= (1<<PC1)| (1<<PC0);
  PORTD &= ~(1<<PD7);
  //init USART
  UCSRB |= (1<<TXEN)|(1<<RXEN);
  //init ADC
  ADCSRA |= (1 << ADEN) | (1 << ADPS2) | (1 << ADPS1);
  //init PWM
  TCCR1A |= (1 << WGM10) | (1 << COM1A1) | (1 << COM1B1);
  TCCR1B |= (1 << CS11);
  return;
}

[auch c Beginner (Gast)]

zwergenstadt schrieb:
> }
>
> uint8_t get_adc_value()
> {
>
>   uint16_t result = 0;
>   return result;
> }
>

Rückgabetyp und Variablentyp passen nicht zusammen?

Dauerplus an PC3 (ADC3)?

[zwergenstadt (Gast)]

Der Rückgabetyp stand noch falsch drin, weil ich es davor mit ADCW 
versucht hatte. Hat aber leider nichts geändert, außer dass er jetzt 
zwischen 33/34 schwankt.
Ein Dauerplus am Eingang ist eigentlich auch nicht möglich. Da ist ein 
Phototrans davor, der den Wert je nach Helligkeit ja dann schön ändern 
sollte. Außerdem tritt das Problem ja leider bei jedem Eingang auf

[Walter S. (avatar)]

zwergenstadt schrieb:
> der den Wert je nach Helligkeit ja dann schön ändern
> sollte.

"sollte" sollte man nicht sagen sondern den Spannungswert am Eingang 
überprüfen

[zwergenstadt (Gast)]

Da hast du natürlich vollkommen recht. Multimeter zeigt an, dass es 
zwischen 0V und 5V Schwankt (ziemlich schlechtes Multimeter, gerade kein 
anders zur Hand)

[zwergenstadt (Gast)]

Habe das ganze eben auf anderer Hardware getestet, da ist der Wert auch 
immer gleich :(

[zwergenstadt (Gast)]

zwergenstadt schrieb:
> ADMUX |= (1 << REFS0)|(1 << MUX1) |(1 << MUX0)  |(1 << ADLAR);

Hab den Fehler gefunden...
Das sollte = nicht |= sein. Also:
ADMUX = (1 << REFS0)|(1 << MUX1) |(1 << MUX0)  |(1 << ADLAR);

[auch c Beginner (Gast)]

zwergenstadt schrieb:
> zwergenstadt schrieb:
>> ADMUX |= (1 << REFS0)|(1 << MUX1) |(1 << MUX0)  |(1 << ADLAR);
>
> Hab den Fehler gefunden...
> Das sollte = nicht |= sein. Also:
> ADMUX = (1 << REFS0)|(1 << MUX1) |(1 << MUX0)  |(1 << ADLAR);

Versteh ich nicht. Nach Einschalten ist ADMUX mit 0x00 initialisiert. Im 
Programm wird ADMUX nur einmal beschrieben. Da sollte es keine Rolle 
spielen, ob der Inhalt verodert wird oder nicht?