AVR Atmega32 while mit Timer Interrupt unterbrechen

[Luki (Gast)]

Hallo,

ich bin noch relativ neu im uC programmieren und habe folgendes Problem.
Ich möchte eine while Schleife mit dem Overflow Interrupt meines Timers1
stoppen. Der Overflow Interrupt funktioniert wenn ich die while Schleife 
auskommentiere(und ein delay einfüge). Wenn er jedoch in der while 
Schleife ist, springt er nicht in den Interrupt bzw. er ruft ihn nicht 
auf (habs mit Breakpoints versucht).

Hier die Funktionen bzw. der Code:

static volatile char checkForOF;

void initTimer1(){
    TCCR1A &=~((1<<WGM10)|(1<<WGM11));   //normal Mode
    TCCR1B &=~((1<<WGM12)|(1<<WGM13));  
     TIMSK |= (1<<TOIE1);  //activates the overflow interrupt
}

void startMeasuring(){    
    unsigned int value;
    unsigned int oldADC=readADC();  
    unsigned int newADC;
    checkForOF=1;
    
    TCCR1B |= ((1<<CS10)|(1<<CS11));    
    TCCR1B &= ~(1<<CS12);  //start Timer 1
    
    while(checkForOF){  //as long as no interrupt
    newADC=readADC();    
    if(newADC>oldADC){    
      value=TCNT1;
      }
    }
    TCCR1B &=~ ((1<<CS12)|(1<<CS11)|(1<<CS10));//stop Timer 1
     TCNT1=0;            
}

ISR(TIMER1_OVF_vect){
  checkForOF=0;
}

Ich hoffe ihr könnt mir weiterhelfen.
Danke für eure Hilfe
Lukas

[Falk B. (falk)]

@ Luki (Gast)

>Ich möchte eine while Schleife mit dem Overflow Interrupt meines Timers1
>stoppen.

Kann man machen. Das static ist eher nicht nötig.

> Der Overflow Interrupt funktioniert wenn ich die while Schleife
>auskommentiere(und ein delay einfüge). Wenn er jedoch in der while
>Schleife ist, springt er nicht in den Interrupt bzw. er ruft ihn nicht
>auf (habs mit Breakpoints versucht).

Vielleicht fehlt die Interruptfreigabe mit sei()?

>Hier die Funktionen bzw. der Code:

Stehen die alle in EINER Datei? Wenn nicht, ist das static hier 
kontraproduktiv.

>static volatile char checkForOF;

[guest (Gast)]

Dann wird die nicht gezeigte readADC(); wohl die Interrupts sperren.

[Frederik H. (diveturtle93)]

Ich könnte mir vorstellen, dass der Timer 1 falsch initialisiert wurde.

So sieht meine Initialisierung aus.

void init_Timer1(void)
{
  TCCR1A = 0;
  TCCR1B = 0;
  TIMSK = (1<<TOIE1);  //Timer1 Overflow Interrupt
  sei();               //Globale Interrupts avtivieren
  TCNT1 = 0;
}

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Luki schrieb:
> void initTimer1(){
>     TCCR1A &=~((1<<WGM10)|(1<<WGM11));   //normal Mode
>     TCCR1B &=~((1<<WGM12)|(1<<WGM13));
>      TIMSK |= (1<<TOIE1);  //activates the overflow interrupt
> }

Da sehe ich nichts von 'Normal Mode'. Das ist der 'Fast PWM' Mode mit 
OCR1A als TOP. Da OCR1A nicht initialisiert wird, steht es vermutlich 
auf null und erzeugt dadurch entweder gar keinen oder einen Regen von 
IRQs. Das steht aber auch so im Datenblatt.
Es ist übrigens gute Praxis, das TOV1 Flag einmal zu löschen, wenn man 
die Kiste initialisiert:
TIFR = (1 << TOV1);

[Oliver S. (oliverso)]

Matthias S. schrieb:
> Es ist übrigens gute Praxis, das TOV1 Flag einmal zu löschen, wenn man
> die Kiste initialisiert:
> TIFR = (1 << TOV1);

Es ist noch bessere Praxis, den Timer erst zu initialisieren, und dann 
zu starten, und ansonsten dem Datenblatt zu Folgen. Damit erübrigen sich 
alle fragwürdigen "gute Praxis" Tips.

Oliver

[Stefan F. (Gast)]

Woher weisst du, dass der Timer-Interrupt auftritt, wenn du die while 
Schleife durch einen delay ersetzt?

In diesem Programmcode sehe ich keinen Test dafür. Hast du das nur 
geraten?

[Rainer B. (katastrophenheinz)]

Der Codeschnipsel sieht ok aus, der Fehler liegt mMn woanders. Wie 
bereits oben gefragt: Hast du Interrupts überhaupt mal mit _sei() 
zugelassen bzw was macht readADC() ?

Zeig mal den gesamten Code!

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Oliver S. schrieb:
> Es ist noch bessere Praxis, den Timer erst zu initialisieren, und dann
> zu starten, und ansonsten dem Datenblatt zu Folgen. Damit erübrigen sich
> alle fragwürdigen "gute Praxis" Tips.

Blablabla. Wieder mal prima aus dem Zusammenhang heraus zitiert. Zur 
Intialisierung habe ich davor was geschrieben, Dummquatscher.
Und einen pending IRQ vor dem Starten des Timers zu löschen, ist gute 
Praxis. Wie wärs einfach mal mit Schnauze halten, wenn man keine Ahnung 
hat?

[S. Landolt (Gast)]

Matthias S. schrieb:
> Luki schrieb:
>> void initTimer1(){
>>     TCCR1A &=~((1<<WGM10)|(1<<WGM11));   //normal Mode
>>     TCCR1B &=~((1<<WGM12)|(1<<WGM13));
>>      TIMSK |= (1<<TOIE1);  //activates the overflow interrupt
>> }
>
> Da sehe ich nichts von 'Normal Mode'. Das ist der 'Fast PWM' Mode ...

Könnten Sie das näher erläutern?

[S. Landolt (Gast)]

Ihn dünkt', er säh' den "normal mode",
ihm schien das sonnenklar.
Er guckt' noch mal und merkt, dass da
der Matze andrer Meinung war.
"Itzt wüsst ich gar zu gern," sprach er,
"lieg ich hier falsch, was ist nun wahr?"

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Is' ja gut, mein Fehler - Asche auf mein Haupt. Die Schreibweise ist 
allerdings wirklich ungewöhnlich.