Immer gleicher Wert am ADC (ATMEGA8)

[Dennis (Gast)]

Guten Abend alle zusammen,
ich habe folgendes Problem mit meinem ADC:
Das ADC-Werte Register bleibt irgendwie immer beim ersten gelesenen Wert 
"hängen", es ändert sich also nicht mehr.

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

uint16_t wert;
ISR(ADC_vect)
{
  wert = ADCL;       
  wert += (ADCH<<8);
  if(wert < (uint16_t)500)  PORTC |= (1<<PC5);
  else if(wert > (uint16_t)524)  PORTC &= ~(1<<PC5);
}

void ADC_Init(void) {


  // interne Referenzspannung als Refernz für den ADC wählen:
  ADMUX = (1<<REFS0);

  ADCSRA = (1<<ADPS1) | (1<<ADPS2)| (1<<ADPS0);     // Frequenzvorteiler
  ADCSRA |= (1<<ADEN) | (1<<ADFR) | (1<<ADIE);
  ADCSRA |= (1<<ADSC);
  _delay_ms(100);
  wert = ADCL;       
  wert += (ADCH<<8);
  ADCSRA |= (1<<ADSC);

   // ADC aktivieren

}



int main()
{
  DDRC |= (1<<PC5);
  PORTC &= ~(1<<PC5);
  sei();
  ADC_Init();

}

Der erste eingelesene Wert (siehe Interrupt) scheint sich nicht mehr zu 
ändern und, denn der Port bleibt etwa HIGH oder LOW, egal, was für eine 
Spannung ich am ADC-Eingang anlege.
Ich hab schon wie doof das Datenblatt studiert und überlegt, ob ich 
irgendwas falsch gemacht habe, aber ich wüsste nicht was.

Hat jemand ne Idee?
LG

[Stefan E. (sternst)]

Dennis schrieb:
> Hat jemand ne Idee?

Deine main-Funktion endet. Nach main() kommt eine Endlosschleife bei 
ausgeschalteten Interrupts.

[cppler (Gast)]

Dennis schrieb:
> int main()
> {
>   DDRC |= (1<<PC5);
>   PORTC &= ~(1<<PC5);
>   sei();
>   ADC_Init();
>
> }

Wie schon erwähnt fehlt die Endlosschleife:

int main()
{
  DDRC |= (1<<PC5);
  PORTC &= ~(1<<PC5);
  sei();
  ADC_Init();
  while(1);
}

[Hubert G. (hubertg)]

Es macht auch keinen Sinn den ADC bei jedem Durchlauf neu zu 
initialisieren. Initialisieren genügt einmal im main, Start und Abfrage 
dann in der Endlosschleife.
Das delay ist auch nicht sehr sinnvoll, besser wäre ein

while (!(ADCSRA & (1<<ADIF)))

da wird nicht länger gewartet als notwendig. Noch besser wäre die 
zugehörige ISR.

[Malte S. (maltest)]

Nebenbei:

> wert = ADCL;
>   wert += (ADCH<<8);

GCC kann das doch alleine...
wert = ADC;

[Dennis (Gast)]

Die while Schleife... Wie peinlich...
Aber danke, vor lauter Stress übersieht man diese triviales... 
funktioniert alles, danke euch ;)