Timer1 bei ATmega16 - Interrupt klappt nicht

[Jürgen H. (misteret)]

Hi!
Ich suche schon seit 2 Stunden meinen Fehler, vielleicht ist es an der 
Zeit, dass mal jemand anders den Quelltext anschaut. Ich suche schon wie 
blöde...

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

unsigned int g_uiTest = 0;

ISR (TIMER1_COPMPA_vect)
{
  g_uiTest++;
}


int main(void)
{
      

    // void init_timer1(void)    
    TCCR1B = (0<<CS02) | (0<<CS01) | (1<<CS00) | (1<<WGM12);
      // Zähltakt einstellen
  // clk = 16 MHz
  // WGM12:  CTC-mode aktiviert


    TCNT1 = 0;
  // Zählreggister auf 0 setzen

    OCR1A = 20000;
        

    TIMSK = (1<<OCIE1A);
  // Freigabe T/C1 Interrupt

    TIFR = 0;

    sei();



    while{1}
    {
       if(g_uiTest > 20000)
       {
          g_uiTest = 0;
       }
    }
    return 1;
}

Im Debug-Modus kann ich genau alles verfolgen, die Register sind korrekt 
gesetzt. Die Variable g_uiTest müsste doch jetzt hochlaufen...tut sie 
aber nicht

[Jürgen H. (misteret)]

Außerdem wundert mich:

Das Output Compare Flag 1 B (OCF1B) wird immer gesetzt, obwohl es zu 
Beginn mit 0 initialisiert wurde. Es bleibt dann auch auf 1.
Und Output Compare Flag 1 A (OCF1A) ist immer 1.

Was hat OCF1B auf einmal damit zu tun?

[holger (Gast)]

volatile unsigned int g_uiTest = 0;

[Jürgen H. (misteret)]

holger schrieb:
> volatile unsigned int g_uiTest = 0;

Stimmt danke...das habe ich vergessen!


Aber auch mit dieser Korrektur klappt es nicht.

[Magnetus (Gast)]

Was wassiert, wenn du den Optimizer ausschaltest? Womöglich wurde da ein 
wenig mehr wegoptimiert, als dir lieb ist. Schließlich wird die Variable 
g_uiTest nicht wirklich sinnvol verwendet. Hier könnte der Optimizer 
also einiges wegoptimieren.

Gruß,
Magnetus

[Jürgen H. (misteret)]

Also das mit der Optimierung, dieses Problem hatte ich, als ich mein 
erstes mal versucht hatte zu debuggen ;-)

Der ist schon ausgeschaltet...auf O0.

Ich sehe ja auch, dass jeder Schritt abgearbeitet wird. Es passiert nur 
nichts mit der Variable

[Magnetus (Gast)]

Wie hast du den Code eigentlich durch den Compiler quetschen können?

ISR (TIMER1_COPMPA_vect)

Das schreibt sich TIMER1_COMPA_vect.
                           ^^

    while{1}

Hier würde ich "while(1)" vorschlagen.
                     ^ ^

Mal sehen, was jetzt der Simulator sagt...

[holger (Gast)]

>    TCCR1B = (0<<CS02) | (0<<CS01) | (1<<CS00) | (1<<WGM12);

    TCCR1B = (0<<CS12) | (0<<CS11) | (1<<CS10) | (1<<WGM12);

Das ist es aber auch nicht.

[Jürgen H. (misteret)]

Hmm also wenn ich das

TIMSK = (1<<OCIE1A);

weglasse, also so

//TIMSK = (1<<OCIE1A);

dann bleibt das OCF1A Flag immer auf 1.


D.h., es muss durch die Software selber auf 0 gesetzt werden.


Mit TIMSK = (1<<OCIE1A); sind die Interrupts enabled. Dadurch wird immer 
beim Eintreten des Interrupts auch das OCF1A-Flag zurückgesetzt. Dieses 
Rücksetzen durch den Interrupt scheint also zu funktionieren. Wieso aber 
wird diese dämliche Variable nicht hochgesazählt???!!

[Jürgen H. (misteret)]

Magnetus schrieb:
> Das schreibt sich TIMER1_COMPA_vect.
>                            ^^

OOOAAAAR DAAAARAN lag es also!!!

Das mit dem while{1} hatte ich nur hier falsch eingetippt....

Aber der Dreher oben war auch im Quelltext....oh man ich sitz einfach zu 
lang davor, um sowas zu sehen....so am besten jetzt aufhören^^

Es kam aber auch keine Fehlermeldung tz tz


Bye bye


Danke!!

[holger (Gast)]

>ISR (TIMER1_COPMPA_vect)

Das wirds wohl sein.
COPMPA hab ich noch nie im Datenblatt gesehen;)

[Magnetus (Gast)]

Büddeschön ;)

[Magnetus (Gast)]

By the way:

Jürgen Hems schrieb:
> Das mit dem while{1} hatte ich nur hier falsch eingetippt....

Hast du schon einmal was von "Copy & Paste" gehört?

[Falk B. (falk)]

@  Jürgen Hems (misteret)

>Das mit dem while{1} hatte ich nur hier falsch eingetippt....

Patsch

Quelltext tippt man nicht ab, man postet das ORIGINAL als Anhang!!!
Nur so kann man den Fehler 100% beurteilen!!!

Leute gibts

>Es kam aber auch keine Fehlermeldung tz tz

Doch, da kommt eine Warnung, man muss halt nur die Augen aufmachen.