Timer beim ATMega32 in C

[Bumbum]

Hallo,

drei Dinge fallen mir spontan auf:

- die Laufvariable muss volatile sein (volatile long int Laufvariable = 0 Warum ist die überhaupt Long? Wenn die nur bis 100 zählt reicht ein Smallint bis 127 oder zumindest ein Byte. Controller-Speicher ist kostbar!
- in die ISR müsste noch ein OCR1A = 15625; oder?
- ist die Display-Ausgabe in der Main auch in einer Endlosschleife?

Viele Grüße
Andreas

[Bumbum]

Nachtrag: Ich habe gerade das Register OCR1A nachgeschlagen. Das ist ja der Compare-Vergleichswert. Da reicht es, wenn er einmal gesetzt wird. In der ISR kannst du ihn weglasen.

Die Initialisierung des Timers habe ich nicht geprüft. Ich hoffe die Bits stimmen alle, und auch der Interrupt wird richtig aktiviert. Wenn es immer noch nicht läuft, würde ich eine LED anschalten, wenn die Interrupt-Routine ausgeführt wird. (Vor dem sei natürlich ausschalten)

[tom77?]

hmm.. danke dir erst mal für deine mühen. code hab ich versucht --> erfolglos

zum datentyp. in der praxis zählt er höher als bis 100 (100 war nur zur vereinfachten darstellung im forum - aber auch mit 100 funktioniert es nicht)

led werde ich testen.

[tom77?]

led wird nicht angesteuert - ergo wird die routine gar nicht angesprochen. aber warum ?

[wkrug]

Warum rätselst Du da rum?
Nimm den Simulator vom AVR Studio und simuliere das Prog Schritt für Schritt.
Dann siehst Du schon wo es hakt.
Du kannst ja das OCRA1 Comparematch Flag auch händisch setzten und musst dann nicht ewig im Code warten.

[Bumbum]

Hallo,

wenn die ISR gar nicht angesteuert wird würde ich sagen ist die Initialisierung des Timers falsch. Das sind nur 3 Zeilen, da muss man halt ein bisschen spielen, wenn man nicht weiß was man da tut.

Nach lesen des Datenblattes frage ich mich, warum du z.B.WGM10 setzt? Das hat mit Timer 1 gar nichts zu tun. Wenn überhaupt heißt das Bit WGM12. Aber wäre nicht Normal Operation besser? Du müsstest dann in der ISR den Timer nur manuell wieder auf 0 setzen.

Viele Grüße
Andreas

[wkrug]

Du müsstest dann in der ISR den Timer nur manuell wieder auf 0 setzen

Das Thema hatten wir schon ein paar Antworten weiter oben.

[Bumbum]

Guten Morgen,

Das Thema hatten wir schon ein paar Antworten weiter oben.

jetzt habe ich es auch gelesen.

Trotzdem sollte das entsprechende Bit laut Datenblatt WGM12 heißen. CTC gibt es nicht, CTC1 auch nicht, deswegen hat das kompilieren in den Antworten weiter oben nicht funktioniert. Aber WGM10 ist laut Datenblatt eindeutig falsch! In der Tabelle für die WGM-Bits des Registers TCCR1A sieht man, dass nur WGM12 gesetzt sein muss für CTC.

Viele Grüße
Andreas

[MagicWSmoke]

Weder ein syntaktisch noch ein semantisch Fehler wird vom Compiler angezeigt

Ist auch ein wildes Durcheinander.

Im letzten geposteten Code:

//Timer1 (16Bit) Initialisieren
TCCR1A |= (1<<CS02) | (1<<CS00) | (1<<WGM10);
OCR1A = 15625;
TIMSK |= (1<<OCIE1A);
sei();

TCCR1A enthält keine Prescalerbits, ein paar Posts vorher war das noch ein bisserl besser:

TCCR1B |= (1<<CS02) | (1<<CS00);

Lies das Datenblatt statt programming-by-guessing.
Prescalerbits und auch einige andere Bitnamen haben bei Atmel die Timernummer im Namen, ein CS02 ist ein Timer0-Bit, hat hier nur keinen Einfluss, da das Register sowieso das Falsche ist und die Bits für Timer0 und Timer1 ander gleichen Position stehen.
Außerdem ver-odert man bei der Initialisierung nicht:

TCCR1A |= ...

sondern weist direkt zu, damit die Register einen definierten Zustand erhalten.

Mach das alles richtig und es wird gehen.

[tom77?]

ich danke euch allen für die posts. werde mich wohl selbst etwas mehr einlesen müssen und hoffen dass ich etwas mehr finde.