ATMega8 interner ADC

[Tomato (Gast)]

Hallo alle miteinander!

Ich möchte mit meinem ATMega8 Spannungen einlesen und diese dann mittels 
LED Anzeige ausgeben lassen (also bitweise Darstellung).

Hier ist schonmal mein Code:

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

volatile uint16_t c = 0;

void out (volatile uint16_t value);

int main(void)
{
  DDRB |= (1 << PB0);
  DDRC |= 0x3E;
  DDRD |= 0xF0;
  
  ADMUX = 0x0000;
  ADCSRA |= (1<<ADFR) | (1<<ADEN) |  (1<<ADPS2);  
  ADCSRA |= (1<<ADIE);              
  sei();
  
  ADCSRA |= (1<<ADSC);              
  while (ADCSRA & (1<<ADSC))            
  {
  }
  
    while (1) 
    {
    ADCSRA |= (1<<ADSC);
    
    while (ADCSRA & (1<<ADSC))
    {
    }

    c = ADCL;
    c += (ADCH << 8);
    }
  
}

ISR (ADC_vect)
 {
    out(c);
  PORTB |= (1<<PB1);  //Zur Kontrolle, ob er ins ISR geht
  ADCSRA |= (1<<ADSC);
 }

void out (volatile uint16_t value)
{
  PORTB ^= (1<<PB0);
  PORTD &= ~0xF0;
  PORTC &= ~0x3E;
  
  PORTD |= (uint8_t)((value >> 2) & 0xF0);
  PORTC |= (uint8_t)(value & 0x3E);  
}

Das ganze will jetzt aber leider nicht so wie ich will. Anscheinend geht 
er nicht ins ISR, da die Kontroll LED nie angesprochen wird.
Liegt das jetzt an meiner Konfiguration? Obwohl da ja nur angeprochen 
wird, oder ist etwas mit meiner Ausgabe nicht in Ordnung?

Für Hilfe wäre ich sehr dankbar :)

[Dyson (Gast)]

B1 ist nicht als Ausgang konfiguriert.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Tomato schrieb:
> Anscheinend geht er nicht ins ISR
Wofür brauchst du für diese simple Aufgabe einen Interrupt?
Ich würde die erst mal mit Polling lösen. Und ich sehe grade, dass du 
das auch tust. Zusätzlich zum Interupt...

Was passiert denn, wenn du das machst:

:
:
    ADCSRA |= (1<<ADSC);
    while (ADCSRA & (1<<ADSC))
    {
    }
    PORTB |= (1<<PB1);  //Zur Kontrolle, ob er überhaupt mal fertig wird

    while (1) 
:
:
ISR (ADC_vect)
{
   out(c);
//   PORTB |= (1<<PB1);  //Zur Kontrolle, ob er ins ISR geht
   ADCSRA |= (1<<ADSC);
}
:
:

BTW: dir ist schon klar, dass du den ADC nicht unbedingt selber in 2 
Zugriffen auslesen musst. Das macht der Compiler schon richtig, wenn du 
schreibst c = ADC;

[Tomato (Gast)]

Oh nein ... so ein dummer Fehler ... Danke ^^
Jetzt weiß ich zumindest schonmal, dass er ins ISR reingeht.
Aber meine Ausgabe ist leider noch immer nicht ganz korrekt. Es leuchten 
permanent alle LEDs, obwohl ja nur die einzelnen Bits leuchten sollten.

Lothar M. schrieb:
> Was passiert denn, wenn du das machst:

Die Kontoll-LED leuchtet nicht

[HildeK (Gast)]

Tomato schrieb:
> c = ADCL;
> c += (ADCH << 8);

Kleiner Tipp, das geht auch einfacher:
 c = ADCW;
oder
 c = ADC;

Zu deinem Problem:
schreib in der ADC-ISR
 PORTB |= (1<<PB0);  //Zur Kontrolle, ob er ins ISR geht
dann schaltet es auch den richtigen Pin :-)
PB0 statt PB1!

[S. Landolt (Gast)]

Welche Spannung liegt an ADC0, welche an AREF?

[S. Landolt (Gast)]

Wozu wird eigentlich ADFR gesetzt, wenn ohnehin jede Messung mit ADSC 
gestartet wird?

[Tomato (Gast)]

Ich habe meinen Fehler gefunden.
Ich dachte, da ich meine "unbekannte" Spannung einlese benötige ich kein 
AREF. Das war aber eine falsche Annahme ^^'

Vielen Dank für die Antworten.
Solche Schusselfehler sollten jetzt nicht mehr vorkommen.

[HildeK (Gast)]

Wozu verwendest du hier den ADC-Interrupt?
Nach meinem bisherigen Verständnis ist der hauptsächlich dann sinnvoll, 
wenn man mit SLEEP_MODE_ADC den Noise Reduction Mode des ADC verwenden 
oder parallel zur AD-Wandlung was anderes tun will. Beides sehe ich in 
deinem Code nicht.
Vielleicht fehlt mir aber auch ein wenig Wissen über weitere Optionen.