AVR Atmega32 while mit Timer Interrupt unterbrechen

@ 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;

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

So sieht meine Initialisierung aus.

1

void init_Timer1(void)

2

{

3

  TCCR1A = 0;

4

  TCCR1B = 0;

5

  TIMSK = (1<<TOIE1);  //Timer1 Overflow Interrupt

6

  sei();               //Globale Interrupts avtivieren

7

  TCNT1 = 0;

8

}

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);

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

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!

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?

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