ATMEGA644 ADC

[Marco G. (grmg2010)]

Hallo,

ich versuche gerade den ADC des ATMEGA644 mit Interrupts in Betrieb zu 
nehmen. Dazu habe ich mich bei der Initialisierung an dem Tutorial hier 
im Forum orientiert.

Mein Problem ist, dass der Interrupt nie auftritt. Hier mal mein Code:

void AD_init(void)
{
  int first_result = 0;
  
  DIDR0 = (1<<ADC1D); //Abschalten der Digitalstufe für ADC1
  
  ADMUX = (1<<REFS0); //Einschalten der Referenzspannung: Vref = AVCC
  ADMUX |=(MUX1);    //Waehlen des ADC1
  
  ADCSRA = (1<<ADEN) | (1<<ADIE); // ADC aktivieren und Interrupt einschalten
  ADCSRA |= (ADPS2) | (1<<ADPS1) | (1<<ADPS0); //Prescaler 128
  
  
  _delay_ms(1);
  
  ADCSRA |= (1<<ADSC);    //Starten der Messung
  
  while (ADCSRA & (1<<ADSC))
  {
    
  }
  
  first_result = (ADCL + (ADCH<<8));
  
}


ISR(ADC_vect)
{
  PORTD ^=(1<<PD6);
  adc_result = (ADCL + (ADCH<<8));
  next = 1;
}


int main(void)
{  

  avr_init();
  UART_Init();
  _delay_ms(500);
  
  sei();

    while(1)
    {

    if (next == 1)
    {
      ADC_value = calculate_adc(adc_result);
      
      dtostrf(ADC_value, 5, 3, buffer_adc);
      uart_puts("Read Voltage: ");
      Send_UART_Char(buffer_adc);
      
      start_converting();
      next = 0;
      
    }
    
    _delay_ms(1000);
    

    
    }
}

Gruß

[c-hater (Gast)]

Marco G. schrieb:

> Mein Problem ist, dass der Interrupt nie auftritt.

Wie sollte er auch? Du rufst AD_init in deinem Code niemals auf. 
Dementsprechend wird auch niemals ADIE gesetzt.

[der chef (Gast)]

next muss volatile sein damit der compiler das nicht wegoptimiert. (wenn 
dann der adc initialisiert ist)

[Ingo L. (corrtexx)]

Es fehlt auch noch ein

 sei();

[Marco G. (grmg2010)]

c-hater schrieb:
> Wie sollte er auch? Du rufst AD_init in deinem Code niemals auf.

Stimmt nicht offensichtlich, AD_init wird in der Funktion avr_init 
aufgerufen.

Wieso fehlt das sei? Steht doch als letzter Befehl vor der 
endlosschleife in meiner Main.

Wieso wurde der Compiler die Variable next wegoptimieren?

[Marco G. (grmg2010)]

Ich habe meinen Code jetzt mal simuliert. Es wird alles in der 
ADC-Sektion so gesetzt wie erwartet, außer dem Bit ADSC. Woran könnte 
das liegen?

[Karl H. (kbuchegg)]

Marco G. schrieb:
> Ich habe meinen Code jetzt mal simuliert. Es wird alles in der
> ADC-Sektion so gesetzt wie erwartet, außer dem Bit ADSC. Woran könnte
> das liegen?

3 Möglichkeiten
* An dem Code, den du nicht gezeigt hast
* Du hast im Atmel-Studio die Freigabe beim Debuggen für ISR nicht 
eingeschaltet, so dass ein Breakpoint in der ISR nicht anspricht
* Du hast mit F10 nicht weit genug durchgesteppt. ADSC kommt in deiner 
AD_init() (bei eingeschaltetem Optimizer) erst dann, wenn man das erste 
mal F10 drückt, während die Zeilenanzeige des Debuggers schon auf der 
Zeile mit dem while steht. Aber es kommt.

Wenn ich deinen geposteten Code compilierbar mache und im Simulator 
durchlaufen lassen, ist alles in Ordnung und verhält sich so wie 
erwartet.

[Marco G. (grmg2010)]

Dies ist der gesammte Code. Wie schalte ich die Freigabe der Isr denn 
ein beim Debugger? Auch wenn ich f10 drücke, bei mir setzt er das 
adsc-bit nicht.
In der Praxis merke ich das auch, pb7 wird nicht getoggelt also kein 
Sprung in die isr

[Stefan E. (sternst)]

Marco G. schrieb:
> Dies ist der gesammte Code.

Garantiert nicht. Wo sind z.B. calculate_adc und start_converting?
Und wenn in letzterem auch etwas in dieser Art steht:

ADMUX |=(MUX1);

ADCSRA |= (ADPS2) ...

wäre es kein Wunder, dass es keine weitere Wandlung gibt.

[Karl H. (kbuchegg)]

Marco G. schrieb:
> Dies ist der gesammte Code.

So so.
Und warum musste ich dann Variablen nachdefinieren und die Hälfte der 
Hauptschleife auskommentieren, wenn das der komplette Code ist?


> Wie schalte ich die Freigabe der Isr denn
> ein beim Debugger?


Siehe Bild

> Auch wenn ich f10 drücke, bei mir setzt er das
> adsc-bit nicht.

Siehe ebenfalls Bild

[Marco G. (grmg2010)]

Habe mir meinen geposteten Code noch mal angesehen. Ich hatte doch nicht 
alles gepostet. Sorry :(
Im Simulator setzt er das Bit, aber bei meinem realen uC funktioniert es 
weiterhin nicht.
Edit: Ok, jetzt funktioniert es. Ursächlich war anscheinend, dass ich 
meine Funktion start_converting(); mit in der Schleife stehen hatte. 
Schreibe ich die Funktion außerhalb der Schleife geht es.

Ich habe den Code noch ein wenig umgeschrieben, um einen Mittelwert zu 
bilden, bekomme jetzt jedoch folgenden Fehler, den ich nicht so ganz 
nachvollziehen kann. Ich hatte nur´eine Header-Datei neu eingefügt

[Marco G. (grmg2010)]

Ok, der Fehler ist behoben. Es fehlte ein Komma in eine der 
Header-Dateien...