Tiemer Polling wird nur einmal erkannt

Der Codefetzen ist nicht ausreichend, um das Problem eindeutig zu lösen.

Was passiert sein kann ist, dass der Timer überhaupt nicht angeschaltet 
ist. In dem Fall läuft das *while (!m_timer_stoped)* so lange bis *if 
(m_timer_stop_counter >= 132)* erfüllt ist.

Wie geschrieben, kann man das nicht eindeutig beurteilen, weil die 
Definition der Schlüsselvariablen (m_timer_stop_counter, m_timer_stoped, 
volatile notwendig?) und des Timerinterrupts nicht angegeben ist und der 
grundsätzliche Aufbau von main unbekannt ist. Und weil nicht klar ist, 
wie der Timer3 eineschaltet wird, denn die Definition von 
power_timer3_enable() fehlt. IMHO würde ich auch ein 
power_timer3_enable() nicht vor den Initialisierungszuweisungen 
schreiben bzw. die Zuweisungen würde ich in der Funktion machen. Dito 
für power_timer3_disable().

Stefan "stefb" B. wrote:
> Der Codefetzen ist nicht ausreichend, um das Problem eindeutig zu lösen.
>
> Was passiert sein kann ist, dass der Timer überhaupt nicht angeschaltet
> ist. In dem Fall läuft das *while (!m_timer_stoped)* so lange bis *if
> (m_timer_stop_counter >= 132)* erfüllt ist.

Das Problem ist daß er nur einmal in "if (!(TIFR3 & (1<<TOV3)))" 
reinkommt, und nie bis 132 kommt.


> Wie geschrieben, kann man das nicht eindeutig beurteilen, weil die
> Definition der Schlüsselvariablen (m_timer_stop_counter, m_timer_stoped,
> volatile notwendig?) und des Timerinterrupts nicht angegeben ist und der
> grundsätzliche Aufbau von main unbekannt ist.

alle "m_..." Variablen sind volatile.
es soll kein (ISR) Timerinterrupt geben sondern nur das polling!
im main wird die Funktion "mv1_ein();" aufgerufen

> Und weil nicht klar ist,
> wie der Timer3 eineschaltet wird, denn die Definition von
> power_timer3_enable() fehlt.

die definition ist in der avr-libc und schaltet die versorgung des 
Timers ein
eingeschaltet wird der Timer mit "TCCR3B  = (1<<WGM33) | (1<<WGM32) | 
(1<<CS30);      // Fast PWM"

> IMHO würde ich auch ein
> power_timer3_enable() nicht vor den Initialisierungszuweisungen
> schreiben bzw. die Zuweisungen würde ich in der Funktion machen. Dito
> für power_timer3_disable().

Ich verstehe leider nicht was du damit meinst.

Dein Timer3-Interrupt läuft also nicht (da das global interrupt enable 
mittels sei(), die Interrupt-Routine und - ungeprüft eventuell mehr - 
fehlen) und du fragst in dem if (!(TIFR3 & (1<<TOV3))) nur den 
Startzustand der Timer3-Hardware ab, die dann mit der Anweisung TIFR3 = 
1<<TOV3; von dir geändert wird und so den nächsten Aufruf des if-Falls 
verhindert...

Stefan "stefb" B. wrote:
> Dein Timer3-Interrupt läuft also nicht (da das /global interrupt enable/
> mittels sei(), die Interrupt-Routine und - ungeprüft eventuell mehr -
> fehlen) und du fragst in dem if (!(TIFR3 & (1<<TOV3))) nur den
> Startzustand der Timer3-Hardware ab, die dann mit der Anweisung TIFR3 =
> 1<<TOV3; von dir geändert wird und so den nächsten Aufruf des if-Falls
> verhindert...

eigentlich sollte doch mit "TIFR3 = 1<<TOV3;" das Bit gelöscht werden? 
Oder?

Gelöscht oder getoggelt? Um es genau zu sagen, müsste ich im Datenblatt 
nachsehen. Um welchen µC geht es genau?

Ich habe bei diesem "Polling" Bauchweh. Warum machst du das nicht 
klassisch mit einem laufenden Timer3?

IMHO bringt es nichts, die Timerhardware zu initialisieren, dann aber 
nicht zu starten und dann abzufragen, ob der Timer3 was gemacht hat.

es handelt sich um einen ATmega1281

in den klasischen Interrupts ist zuviel overhead drin (ist mir zulangsam 
bei nur ganz kleiner ausschaltzeit)

> Ich verstehe nicht warum du erst abfrägst ob das Bit NICHT gesetzt ist
> um anschließend das Bit zu löschen?
> Muss es nicht heissen:
> if((TIFR3 & (1<<TOV3)) ....  // Wenn Flag gesetzt...

Danke das war es.

So was blödes, Da hab ich immer wieder drübergelesen.