Hi,
ich habe zum Lernen und Verstehen des ATMega8's auf einem myAVR-Board
ein Programm geschrieben, das die 3 Leuchtdioden binärmäßig
sekundnweise schalten soll:
#include <avr/io.h>
#include <avr/signal.h>
#include <avr/interrupt.h>
#define TL 0xf7
#define TH 0xc2
unsigned char i=0;
int main(void)
{
DDRD = 0x00;
PORTD |= (1<<PD2) | (1<<PD3);
DDRB = 0xff;
PORTB = 0x00;
TCCR1B |= (1<<CS00) | (1<<CS02);
TCNT1L = TL;
TCNT1H = TH;
TIMSK |= (1<<TOIE1);
TCNT1L = TL;
TCNT1H = TH;
sei();
while (1) {}
return 0;
}
SIGNAL(SIG_OVERFLOW1)
{
i = PINB;
i++;
if (i==8) i=0;
PORTB =i;
TCNT1L = TL;
TCNT1H = TH;
}
Meine Frage bezieht sich auf die Zeilen
TCNT1L = TL;
TCNT1H = TH;
TIMSK |= (1<<TOIE1);
TCNT1L = TL;
TCNT1H = TH;
Es kommt mir dämlich vor, zweimal die TCNT1x initialisieren zu müssen,
ansonsten (egal, welche der beiden ich weglasse) zählt der Timer
erstmal von FFFF runter, d.h. die erste LED geht (bei 16MHz) erst nach
ca. 4s an.
Mache ich was falsch?
Gruß, Axel
Gast
#285394
Hi, der Teil außerhalb der while(1)-Schleife deines main-Programmes läuft ja nur einmal. Dort wird der Timer mit TL/TH vorgeladen und beim Setzen des Prescalers läuft der Timer los. Nach 1 Sekunde kommt der gewollte Überlauf. Zu diesem Zeitpunkt ist der aktuelle Zählerstand (TL/TH) = 0. Wenn du innerhalb der Überlauf-ISR (SIGNAL(OVERFLOW)) den Timer nicht wieder mit den Standardwerten lädst, zählt er keine Sekunde mehr sondern von 0...65535 (0...0xFFFF). In deinem Programm setzt du den Timer bei einem Überlauf immer wieder auf 0xC2F7 (49911). Von diesem Wert zählt der Timer dann wieder bis 0xFFFF (65535). Du wirst mir Recht geben, dass hier der Überlauf schneller kommt, als wenn er von 0 bis 65535 zählen müsste. Mfg Tommy
Gast
#285395
Sorry, war etwas zu schnell. Ich habe mir gerade nochmal deine Quelle angeschaut. Hast du das Programm selber geschrieben oder irgendwo abgetippt? Ich gehe immer wie folgt vor. (1) Timer stoppen (Prescaler ausschalten) (2) Zählerwert (TH/TL) setzen (3) Konfigurationen in den Registern setzen (4) Prescaler-Bits via "OR-Verknüpfung" setzen -> Timer wird gestoppt Also denke ich, dass du problemlos bei deinem Quelltext die letzten beiden Zeilen auskommentieren. TCNT1L = TL; TCNT1H = TH; TIMSK |= (1<<TOIE1); //TCNT1L = TL; //TCNT1H = TH; Also einfach mal probieren... Viele Grüße Tommy
Das klappt eben nicht -> wenn ich das so mache, zählt der Timer einmal bis FFFF bis die erste LED angeht, danach gehts korrekt im Sek-Takt weiter. Verstehen tue ich das auch nicht; aber vielleicht kann mir das jd. erklären. Gruß, Axel
Mal diverse Anmerkungen in wahlloser Reihenfolge.
1 | |
2 | |
3 | |
4 | |
5 | |
Sorry, aber das würde ich als ,,Assemblerprogrammierung mit C-Syntax'' bezeichnen. (Selbst in Assembler kann man das besser formulieren.) Ich würde das so schreiben:
1 | |
2 | |
3 | |
4 | |
5 | |
6 | |
7 | |
8 | |
9 | |
10 | |
11 | |
12 | |
13 | |
14 | |
15 | |
1 | |
Diese Variable sollte static innerhalb der ISR sein, denn nur dort wird sie gebraucht.
1 | |
Schönheitsfehler: du setzt Timer 1, aber nimmst die Definitionsbits von Timer 0. Meine Variante:
1 | |
2 | |
3 | |
4 | |
5 | |
6 | |
7 | |
8 | |
9 | |
10 | |
11 | |
12 | |
13 | |
14 | |
15 | |
16 | |
17 | |
18 | |
19 | |
20 | |
21 | |
22 | |
23 | |
24 | |
25 | |
26 | |
27 | |
28 | |
29 | |
30 | |
31 | |
32 | |
33 | |
34 | |
35 | |
36 | |
37 | |
38 | |
39 | |
40 | |
41 | |
42 | |
43 | |
44 | |
45 | |
(Für avr-libc < 1.4.x musst du ISR durch SIGNAL ersetzen und den alten Vektornamen benutzen.) Im Simulator blinkt zumindest die LED im Sekundentakt. Das Ganze hat natürlich rein akademischen Wert: in der Praxis würde ich statt der umständlichen und mit Jitter behafteten Methode des Overflow-Interrupts den CTC-Modus des Timers nehmen.
Hi, Danke; ich übe noch und bin für alle Tips sehr dankbar; ich werde es ausprobieren. Gruß, Axel
Antwort schreiben
Bitte melde dich an, um einen Beitrag zu schreiben.