Sekunden Timer nicht synchron

[Jo S. (tigermatze)]

Hi,

ich verwende einen AT90USB128 und möchte auf dem Timer/Counter 2 einen 
Sekunden Timer laufen lassen.
Leider läuft die Zeit nicht synchron, das heißt die Sekunde kommt nicht 
genau was ich mir irgendwie nicht erklären kann.
Ich habe den MC folgendermaßen konfiguriert:

ASSR = 0x00;    // Asynchronous Status Register;
TCCR2A = 0x00;    // Normal port operation, OC2A disconnected.
TCCR2B  = (1<<WGM21) | (1<<CS22) | (1<<CS21);  // CTC, prescaler 256 -> 8MHz / 256 = 31250Hz
//TCCR2B = 0x00;  // disabled
TCNT2 = 0x00;    // Timer/Counter Register;
OCR2A = 0xFA;    // (FA=250) 31250Hz / 250 = 125Hz -> jede 125-tel Sekunde ein Interrupt
      // 125Hz / 125 = 1Hz = 1
OCR2B = 0x00;    // Output Compare Register B;
TIMSK2 = 0x00;    // Timer/Counter Interrupt Mask Register;
TIFR2 = 0x00;    // Timer/Counter 2 Interrupt Flag Register;
GTCCR = 0x00;    // General Timer/Counter Control Register;

TIMSK2 = (1<<OCIE2A);    // Timer/Counter2 Interrupt Mask Register; the Timer/Counter2 Compare Match A interrupt is enabled.

Meine ISR wird also alle 125-tel Sekunde aufgerufen

ISR(TIMER2_COMPA_vect)
{
  Disable_interrupt();

  tmp_sek++;

  tmp_sek %= 125;
  if ( tmp_sek < last_tmp_sek )
    Second++;
  
  last_tmp_sek = tmp_sek;

  Enable_interrupt();
}

In der main-loop wird die Sekunde dann als Uhrzeit auf einem Display 
ausgegeben. Nur leider geht die Uhr falsch, weil die Sekunde nicht genau 
kommt.

Könnt ihr einen Fehler entdecken?


Vielen Dank

[Karl H. (kbuchegg)]

Was heißt 'nicht genau'? Wieviel Abweichung hast du?

[Karl H. (kbuchegg)]

ZUm Programm

OCR2A = 0xFA;    // (FA=250) 31250Hz / 250 = 125Hz -> jede 125-tel 
Sekunde ein Interrupt
      // 125Hz / 125 = 1Hz = 1

Erstens.
Wenn du dir im dir gewohnten Dezimalsystem ausrechnest, wann der Timer 
den Reset machen muss, warum schreibst du das dann in Hex hin?

  OCR2A = 250;

Zweitens.
Du musst 1 weniger nehmen. Du brauchst 250 Zähl-ZYKLEN. Auch das 
Rücksetzen vom Wert auf 0 ist ein Zyklus. Also

  OCR2A = 250 - 1;

[Karl H. (kbuchegg)]

Drittens
Das hier

ISR(TIMER2_COMPA_vect)
{
  Disable_interrupt();

  tmp_sek++;

  tmp_sek %= 125;
  if ( tmp_sek < last_tmp_sek )
    Second++;
  
  last_tmp_sek = tmp_sek;

  Enable_interrupt();
}

ist wohl so ziemlich die komplizerteste Form die man finden kann wie man 
das schreibt.

3A
In einer ISR sind Interrupts sowieso disabled. Es gibt also keinen Grund 
Interrupts zu disablen. Und schon gar nicht zu enablen, wenn du keinen 
wirklich guten Grund dafür hast. Darum kümmert sich schon der COmpiler.

Einen % willst du drinn nicht wirklich haben. AVR können nur schlecht 
dividieren.

3B

ISR(TIMER2_COMPA_vect)
{
  tmp_sek++;

  if( tmp_sek == 125 ) {
    tmp_sek = 0;
    Second++;
  }
}

[MWS (Gast)]

Der Wert, der errechnet wurde, muss für CTC minus 1 im OCRxA stehen, 
gibt 'ne Formel dazu im DB, aus der das hervorgeht.

[Uwe (de0508)]

Karl Heinz hat das schon sehr gut verbessert,
mir fehlt noch die Definition von tmp_sek.

Ich verwende gerne

static uint8_t tmp_sek = 0;

 innerhalb der ISR,
dann ist gansklar wo die Variable verwendet wird. Und kein anderer 
Prozess kann sie verändern.

[Bingo (Gast)]

Ist Second defined as volatile ?

mfg
Bingo Dänemark