Atmega8 Timer1 Interrupt Routine wird nicht angesprungen

[M. H. (siddd)]

Hallo,

Ich möchte eine PWM erzeugen. Am Ausgang OCA1 ist eine LED 
angeschlossen. Der AD-Wandler ADC0 soll im 10-Bit Modus laufen.

Die Interrupt Routine wird nicht angesprungen. Ich finde den Fehler 
nicht. Wenn mir jemand helfen könnte dem wäre ich dankbar.

#define F_CPU 160000000
#include <avr/io.h>
#include <stdint.h>
#include <avr/interrupt.h>


volatile uint16_t buffer;
volatile int8_t flag;

ISR(TIMER1_COMPA_vect)
{
  //Hilfsport setzen
  PORTD |= 1<<PD0;
  
  // Wandlung vom analogen Eingang starten
  ADCSRA |= (1<<ADSC);

  // Warten bis die AD-Wandlung abgeschloßen ist
  while ( !(ADCSRA & (1<<ADIF)) );
  
  //Hilfsport zurücksetzen
  //  PORTD &= ~(1 << PD0);
  PORTD = 0;
  
  buffer = ADC;
  OCR1A=buffer * 4;
}



int main()
{  
  //Alle Interrupts ausschalten
  cli();
  
  //Hilfsport als Ausgang definieren und zuruecksetzen
  DDRD = DDRB | (1 << PD0);
  PORTD = 0;
  
  
  // Timer für PWM initialisieren
  // OC1A auf Ausgang
  DDRB = (1 << PB1 );
  TCCR1A = (1<<COM1A1);
  TCCR1B = (1<<WGM13)  | (1<<CS10);
    
  // PWM Frequenz auf 2000 Hz stellen bei 16 Mhz Oscillator
  ICR1 = 4000;
    
  //Timer Interrupt initialisieren und globale Interrupts enable  
  TIMSK = TIMSK | (1 << OCIE1A);
  sei();
  
  // AD-Wandler initialisieren und auf 10 Bit stellen
  ADMUX = (1<<REFS0);
  ADCSRA = (1<<ADEN);
  
  //Startwert für PWM festlegen
  OCR1A = 1000;

  while( 1 )
  {  
    //Hauptschleife
  }
}

Gruss
Michael

[Högli (Gast)]

Was mißt denn der ADC?
Im gewählten Timermodus ist ICR1=1000 der TOP Wert. Hier:

buffer = ADC;
OCR1A=buffer * 4;

kann der Comparewert für den Interrupt aber locker über 1000 gesetzt 
werden, so daß der Interrupt sich quasi selber an der Ausführung 
hindert.

[M. H. (siddd)]

Auf ADC0 ist am Mittelabgriff eines Potentiometer mit 5 V Versorgung 
angeschlossen.

Wenn ich den Code aus der Interrupt-Routine in der Hauptschleife des 
Hauptprogramms ausführe, dann wird die PWM mit 2kHz Frequenz ausgeführt. 
Über das Potentiometer läßt sich dann die Pulsbreite der PWM stufenlos 
einstellen. Nur der Atmega8 führt dann die AD-Wandlung permanent aus.

Ich möchte allerdings, dass die AD-Wandlung nur einmal pro Interrupt 
Zyklus ausgeführt wird.

[Högli (Gast)]

Högli schrieb:
> Im gewählten Timermodus ist ICR1=1000 der TOP Wert. Hier:

Sorry, muß ICR1=4000 heißen. Kann aber immer noch vom ADC * 4 überboten 
werden.

> Ich möchte allerdings, dass die AD-Wandlung nur einmal pro Interrupt
> Zyklus ausgeführt wird.

In der ISR die ADC Messung starten. Dann in der Hauptschleife "// Warten 
bis die AD-Wandlung abgeschloßen ist" oder den ADC Interrupt nutzen. In 
der ISR auf Abschluß der Wandlung warten ist selten sinnvoll.

[C. W. (chefkoch)]

Warum genau lässt Du nicht den ADC frei laufen und machst das PWM-Update 
in der ISR "ADC Conversion complete"?

[M. H. (siddd)]

Vielen Dank für die Kommentare!

Högli schrieb:
> In der ISR die ADC Messung starten. Dann in der Hauptschleife "// Warten
> bis die AD-Wandlung abgeschloßen ist" oder den ADC Interrupt nutzen. In
> der ISR auf Abschluß der Wandlung warten ist selten sinnvoll.

Ich habe den Kommentar umgesetzt, und siehe da es funktioniert. Dabei 
habe ich der Übersicht halber noch einzelne Änderungen durchgeführt. Der 
neue C-Code lautet folgendermassen:

#define F_CPU 160000000
#include <avr/io.h>
#include <stdint.h>
#include <avr/interrupt.h>


volatile uint16_t buffer;
volatile int8_t flag;

ISR(TIMER1_COMPA_vect)
{
  flag=1;
}



int main()
{  
  //Alle Interrupts ausschalten
  cli();
  
  //Hilfsport als Ausgang definieren und zuruecksetzen
  DDRD = DDRB | (1 << PD0);
  PORTD = 0;
  
  
  // Timer für PWM initialisieren
  // OC1A auf Ausgang
  DDRB = DDRB | (1 << PB1 );
  TCCR1A = TCCR1A | (1 << COM1A1);
  TCCR1A = TCCR1A | (0 << COM1A0);
  TCCR1B = TCCR1B | (1 << WGM13);
  TCCR1B = TCCR1B | (0 << WGM12);
  TCCR1B = TCCR1B | (1 << CS10);
  TCCR1B = TCCR1B | (0 << CS11);
  TCCR1B = TCCR1B | (0 << CS12);
  TCCR1A = TCCR1A | (0 << WGM10);
  TCCR1A = TCCR1A | (0 << WGM11);
    
  // PWM Frequenz auf 1953 Hz stellen bei 16 Mhz Oscillator
  ICR1 = 4095;
    
  //Timer Interrupt initialisieren und globale Interrupts enable  
  TIMSK = TIMSK | (1 << OCIE1A);
  sei();
  
  // AD-Wandler initialisieren und auf 10 Bit stellen
  ADMUX = (1<<REFS0);
  ADCSRA = (1<<ADEN);
  
  //Startwert für PWM festlegen
  OCR1A = 1000;

  while( 1 )
  {  if (flag==1)
    {
        //Hilfsport setzen
        PORTD |= 1<<PD0;
        
        // Wandlung vom analogen Eingang starten
        ADCSRA |= (1<<ADSC);

        // Warten bis die AD-Wandlung abgeschloßen ist
        while ( !(ADCSRA & (1<<ADIF)) );
        
        //Hilfsport zurücksetzen
        //  PORTD &= ~(1 << PD0);
        PORTD = 0;
        
        buffer = ADC;
//        OCR1A = buffer * 4;
        OCR1A = (buffer<<2);
        flag = 0;
    }

  }
}

Den ADC möchte ich nicht frei laufen lassen, um für spätere Anwendungen 
Rechenkapazität zu sparen.

[C. W. (chefkoch)]

??? Du willst Rechenzeit sparen indem Du durch den Timer ein Flag setzt, 
in der Mainschleife und dann wartest bis der ADC fertig ist???

[M. H. (siddd)]

So ist es! Vielleicht mache ich noch einen Denkfehler.

Was ist denn aus Deiner Sicht der Vorteil, wenn man den ADC frei laufen 
läßt ?

[Uwe (de0508)]

Guten Tag,

es gibt noch ein weiteres Problem mit diesem Codeabschnitt:

// Wandlung vom analogen Eingang starten
ADCSRA |= (1<<ADSC);

// Warten bis die AD-Wandlung abgeschloßen ist
while ( !(ADCSRA & (1<<ADIF)) );

Wenn man nach einen atMega Datenblatt geht, dann ist es so richtig:

// Wandlung vom analogen Eingang starten
ADCSRA |= (1<<ADSC);

// Warten bis die AD-Wandlung abgeschloßen ist
while ( (ADCSRA & (1<<ADSC)) ) { }

[Bastler (Gast)]

Der ADC kann sich auch selbst melden, wenn er fertig ist:

 ISR( ADC_vect ) {}

[Högli (Gast)]

Man könnte auch die ADC Wandlung wie ursprünglich in der ISR starten. In 
der Hauptschleife ohne irgendeine Abfrage oder auf irgendwas zu warten 
den ADC Wert immer einfach direkt dem OCR1A zuweisen. Wird zwar oft 
zugewiesen, ändert sich ja dann aber ohnehin nur mit der 
Interruptfrequenz mit der ja eine neue Wandlung gestartet wird. Ob das 
in dein Konzept paßt mußt du dann schauen.

[M. H. (siddd)]

>
>

 
> // Warten bis die AD-Wandlung abgeschloßen ist
> while ( (ADCSRA & (1<<ADSC)) ) { }

Uwe S. schrieb:
> // Warten bis die AD-Wandlung abgeschloßen ist
> while ( (ADCSRA & (1<<ADSC)) ) { }

Und was passiert, wenn man die Zeile ganz weg läßt ?

Funktionieren tut es auch ohne dieser Zeile.

[Högli (Gast)]

M. H. schrieb:
> Und was passiert, wenn man die Zeile ganz weg läßt ?

Was soll passieren? Kaputt geht nix. Und ... außerdem ... hab ich doch 
GERADE geschrieben ... :-) :-) :-) 
Beitrag "Re: Atmega8 Timer1 Interrupt Routine wird nicht angesprungen"

[spess53 (Gast)]

Hi

>Und was passiert, wenn man die Zeile ganz weg läßt ?

>Funktionieren tut es auch ohne dieser Zeile.

Das gleiche wie bei

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

Der Controller macht ohne zu warten weiter. ADIF bleibt nämlich 1, wenn 
du es nicht manuell zurück setzt.

ADSC dagegen bleibt so nur lange 1 bis der ADC mit der Conversation 
fertig ist.

MfG Spess

[Uwe (de0508)]

Danke spess53,

für die Klarstellung, wenn man nur das Datenblatt zum  Verständnis lesen 
würden, wenn hätte man ein Problem weniger.

[M. H. (siddd)]

Wenn ich das Datenblatt nicht lesen würde, dann wäre ich nicht soweit 
gekommen!

Eine bessere Frage wäre wohl gewesen, warum der ATMEGA8 die ADC Wandlung 
noch zu Ende führt:
> while ( (ADCSRA & (1<<ADSC)) ) { }

Meine Erwartung wäre gewesen, dass der ATMEGA8 im Code fortschreitet 
(wenn die Zeile nicht vorhanden ist)und die ADC Wandlung abbricht.

[Händeringer (Gast)]

>Wenn ich das Datenblatt nicht lesen würde, dann wäre ich nicht soweit
>gekommen!

Zustimmung!

In Anbetracht der Tatsache, daß es sich um DATENBÜCHER mit mehreren
hundert Seiten handelt, in denen die zusammenhängenden Informationen
sehr gut gestreut vorliegen, kann es passieren, daß man einen Halbsatz
überliest.

Diese Überheblichkeit ist nicht angebracht!

[Oliver S. (oliverso)]

Händeringer schrieb:
> In Anbetracht der Tatsache, daß es sich um DATENBÜCHER mit mehreren
> hundert Seiten handelt, in denen

die Kapitel mir der Beschreibung zu den einzelnen Modulen immer nur 
wenige Seiten umfassen, ist das durch aus machbar.

Oliver

[spess53 (Gast)]

Hi

>Meine Erwartung wäre gewesen, dass der ATMEGA8 im Code fortschreitet
>(wenn die Zeile nicht vorhanden ist)und die ADC Wandlung abbricht.

Warum soll der ADC die Wandlung abbrechen? Dem ADC ist es egal ob ADSC 
abgefragt wird.

MfG Spess