Also ich hab den Fehler jetzt gefunden.
Es fehlt lediglich eine ganze Zeile in Code !!
Den Timer Interrupt nur alle 10msec ausführen zu lassen ist keine so gute Idee. Es wird unter anderem geprüft wie lange eine Taste gedrückt wurde. Dazu dient als Zeitgeber dieser Interrupt (Ticks). Bei einer lang gedrückten Taste gibt es z.B. eine variable 60Ticks. Wären bei 10msec dann 600msec. Das ist zu lange.
Aber so funktionierts:

Code :

// Interruptserviceroutine für Timer 0 // hier 1ms ISR( TIMER0_OVF_vect ) // every 1ms { TCNT0 = (uint8_t)(int16_t)-(F_CPU / 1024 * 1e-3 + 0.5); // preload for 1ms volatile static unsigned char count_ovl0; volatile unsigned char ovl0 = count_ovl0 +1; if (ovl0 >= 10) { get_taster (0,PIND & (1<<PD2)); get_taster (1,PIND & (1<<PD3)); get_taster (2,PIND & (1<<PD4)); ovl0 = 0; } count_ovl0 = ovl0; }

Zitat :

Wenn du jede ms einen Interrupt (Aufforderung das Programm zu unterbrechen und in die Interuptroutine zu springen) auslöst kommt die gar nicht dazu auf 10 ms hoch zu zählen.

Wos?


Dein zweiter Quellcode ist besser, wobei ovl0 nicht volatile sein muss sondern nur der Zähler.
ovl0 als temporäre Variable zu benutzen ist aber eine gute Idee, dann muss der Zählerwert nicht jedes mal aus dem RAM zurückgelesen werden (bzw. auch beim Inkrementieren geschrieben werden). Das kostet geringfügig mehr Zeit und Platz.



_________________

Zitat :

Dein zweiter Quellcode ist besser,

...Aber dies ist bei Timern extrem schlecht:

Zitat :

TCNT0 = (uint8_t)(int16_t)-(F_CPU / 1024 * 1e-3 + 0.5); // preload for 1ms

Da man nicht genau weiss, wie lange vor der Ausführung dieses Befehls der Überlauf passiert ist, lädt man den Wert für den Timer nicht einfach in das Register, sondern man addiert ihn besser zum aktuellen Registerinhalt.

Hab diese Zeile nicht verstanden.
Erklär mir doch bitte mal was da passiert ?
den zweiten Teil kann ich ja noch ausrechnen, doch was bedeuten hier die
Deklariationen uint8_t und uint16_t ?
Ist das die Subtraktion des aktuellen Zählerstandes von TCNT0 vom max Zählerstand ?

Zitat :

doch was bedeuten hier die Deklariationen uint8_t und uint16_t ?

Das siehst du am besten in der Dokumentation deines C-Komplizierers nach (und lernst es), oder du lässt es übersetzen und schaust dir den erzeugten Maschinencode an (sicherer, wenn man nah an der Hardware programmiert).

@Perl
Hab letztendlich doch noch eine Erklärung, sogar vom Autor persönlich verfasst, im Netz gefunden.
Für mich als Anfänger wäre es fast unmöglich diese Notation zu verstehen.
Es handelt sich um einen CAST, bzw zwei CASTS. Der Ausdruck rechts wird erst in eine 16bit unsigned int und dann in eine 8bit unsigned int gewandelt. Timer 0 hat nur 8bit.
Nach der Umwandlung wird der Wert ins TCNT0 Register geladen (hier 6).
Ich werde die Routine so abändern, daß ich vorher noch den Wert des Registers TCNT0 auslese und dann den Rest zuaddiere.
Das hilft mir jetzt weiter.
LG
Peterschrott

Zitat :

Nach der Umwandlung wird der Wert ins TCNT0 Register geladen (hier 6)

Ja, der ganze lange Ausdruck lädt nur eine Konstante ins Register, d.h. zur Laufzeit wird da gar nichts mehr gerechnet.

Zitat :

Ich werde die Routine so abändern, daß ich vorher noch den Wert des Registers TCNT0 auslese

Mach das lieber nicht explizit, sondern addiere einfach die Konstante zum Register.
Mit etwas Glück, d.h. wenn der aktuelle Wert von TCNT0 nicht noch woanders gebraucht wird, optimiert der Compiler das vorige Auslesen sowieso weg.