Timer macht nicht das, was er soll?

[Jonathan K. (burgerohnealles)]

Hallo,

ich wollte zuerst einen einfachen Timer programmieren, der jede Sekunde 
eine Variable auf 1 setzt. Dazu habe ich den 8-Bit Timer2 genommen, den 
Prescaler auf 1024 gesetzt und den Overflow Interrupt eingeschaltet. 
Jetzt sollte dieser Interrupt alle 1 / (16.000.000 / 1.024 / 256) = ca. 
61 Millisekunden ausgelöst werden. Dies geschieht aber bei mir nur ca. 
alle 3 bis 5 Millisekunden. Mein Code:

#define SET_BIT(x,y) x|=(1<<y)
#define CLR_BIT(x,y) x&=~(1<<y)
#define TGL_BIT(x,y) x^=(1<<y)
#define IS_BIT(x,y)  (x&(1<<y))


volatile int changed = 0;


void setup () {
  Serial.begin(9600);
  
  SET_BIT(TCCR2B, CS20);
  SET_BIT(TCCR2B, CS22);
  SET_BIT(TIMSK2, TOIE2);
}

void loop () {
  long t = millis();
  while(changed == 0);
  t = millis() - t;
  Serial.println(t);
  changed = 0;
}

ISR (TIMER2_OVF_vect) {
  changed = 1;
}

Was mache ich falsch?


Danke schon mal
burgerohnealles

[Thomas E. (thomase)]

Jonathan K. schrieb:
> Jetzt sollte dieser Interrupt alle 1 / (16.000.000 / 1.024 / 256) = ca.
> 61 Millisekunden ausgelöst werden

Jonathan K. schrieb:
> SET_BIT(TCCR2B, CS20);
> SET_BIT(TCCR2B, CS22);

Dann stell deinen Prescaler auch auf 1024.

mfg.

[Electronics'nStuff (Gast)]

Nicht sonderlich gut gelesen, oder?

[Uwe (Gast)]

1/16000000=62.5ns Pro Takt
Er braucht 1024*256 Takte a 62.5ns
Also 62.5ns*1024*256=16,384ms
Also alle ca. alle 16ms

[Jonathan K. (burgerohnealles)]

Thomas Eckmann schrieb:
> Dann stell deinen Prescaler auch auf 1024.

Hab ich das nicht gemacht? Laut 
http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Die_Timer_und_Zähler_des_AVR#8-Bit_Timer.2FCounter 
muss ich CS20 und CS22 setzen. Und das hab ich doch gemacht!? Wo ist 
mein Fehler?

[Jonathan K. (burgerohnealles)]

Warum steht da: 
http://www.mikrocontroller.net/attachment/173218/table.png was anderes 
als da: 
http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Die_Timer_und_Zähler_des_AVR#8-Bit_Timer.2FCounter 
?

[Walter S. (avatar)]

Jonathan K. schrieb:
> Wo ist
> mein Fehler?

du verwendest timer2, dann musst du auch die Werte für timer2 nehmen

[Electronics'nStuff (Gast)]

Vor allem schaut man diese Dinge im DB nach und nicht im Tutorial. Die 
Garantie, dass es kompatibel ist hast du nie zu 100%.

[Jonathan K. (burgerohnealles)]

Walter S. schrieb:
> du verwendest timer2, dann musst du auch die Werte für timer2 nehmen

Ahh. Ok. Ich habe jetzt dieses Bit noch gesetzt, aber es funktioniert 
trotzdem noch nicht ganz. Nach meinem Programm wird der Interrupt ca. 
alle 32ms ausgelöst. Es müsste doch ca. alle 16ms sein. Wo ist jetzt der 
Fehler?

[Electronics'nStuff (Gast)]

Naja, dann hast du offensichtlich die falsche Taktquelle.

[Karl H. (kbuchegg)]

Jonathan K. schrieb:
> Warum steht da:
> http://www.mikrocontroller.net/attachment/173218/table.png was anderes
> als da:
> 
http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Die_Timer_und_Zähler_des_AVR#8-Bit_Timer.2FCounter
> ?

Weil Atmel zig-verschiedene µC produziert mit jeweils unterschiedlicher 
Ausstattung an Timern.
Ein Tutorial zeigt immer die Dinge beispielsweise anhand eines 
spezifischen µC (und in diesem Fall anhand eines spezfischen Timers, den 
dieser im Tutorial benutzte µC besitzt).

Für dich ist das Tutorial ein Überblick, was die Timer so können und wie 
man sei konfiguriert. Aber deine Bibel, die einzig für dich relevante 
Dokumentation, ist das Datenblatt zu deinem Prozessor.

[Electronics'nStuff (Gast)]

Electronics'nStuff schrieb:
> Naja, dann hast du offensichtlich die falsche Taktquelle.

EDIT: Ich tippe mal auf interne Taktquelle, 8MHz.

[Karl H. (kbuchegg)]

Mach da mal

void loop () {

  while(changed == 0)
    ;
  changed = 0;

  long t = millis();
  while(changed == 0)
    ;
  t = millis() - t;

  Serial.println(t);
  changed = 0;
}

draus.
je nach Baudrate braucht auch der println seine Zeit. Wenn du da einen 
Timer-Interrupt verpasst, dann misst du die Zeit zwischen 2 Interrupts.

[Jonathan K. (burgerohnealles)]

Electronics'nStuff schrieb:
> Electronics'nStuff schrieb:
>> Naja, dann hast du offensichtlich die falsche Taktquelle.
>
> EDIT: Ich tippe mal auf interne Taktquelle, 8MHz.

Hab mal die Fuses gecheckt: FF DE 05 -> Das heißt Ext. Crystal Osc.
Der Arduino läuft mit 16Mhz. Das heißt daran kann es schon mal nicht 
liegen.

Karl Heinz Buchegger schrieb:
> Mach da mal
> void loop () {
>
>   while(changed == 0)
>     ;
>   changed = 0;
>
>   long t = millis();
>   while(changed == 0)
>     ;
>   t = millis() - t;
>
>   Serial.println(t);
>   changed = 0;
> }
>
> draus.
> je nach Baudrate braucht auch der println seine Zeit. Wenn du da einen
> Timer-Interrupt verpasst, dann misst du die Zeit zwischen 2 Interrupts.

Das geht leider auch nicht. Der Arduino gibt immer noch 32 aus.

[Jonathan K. (burgerohnealles)]

Hab jetzt nochmal ein bisschen herumprobiert. Ich dachte schon evtl. ist 
mein Arduino kaputt. Aber mit der Library "TimerOne" funktionierts 
perfekt. Der folgende Code sollte ca. alle 4 Sekunden die LED auf dem 
Arduino ein-/ausschalten (1 / (16.000.000Hz/1024/65536) = ca. 4,2s). 
Warum funktioniert der folgende Code nicht (die LED blink gar nicht)?

#define SET_BIT(x,y) x|=(1<<y)

void setup () {
  pinMode(13, OUTPUT);

  // prescaler: 1024
  SET_BIT(TCCR1B, CS12);
  SET_BIT(TCCR1B, CS11);
  SET_BIT(TCCR1B, CS10);
  
  // overflow interrupt enable
  SET_BIT(TIMSK1, TOIE1);
  
  // enable global interrupts
  sei();
}

ISR (TIMER1_OVF_vect) {
  digitalWrite(13, digitalRead(13)^1);
}

void loop () {
  // nothing
}