Atmega 328P genaue Sekunde Peter Danneger

[Werner P. (Gast)]

Hallo zusammen,

da ich gerade einen genauen Sekunden Takt (Uhr) brauche habe ich mal den 
Code von Peter Danneger verwendet. Dank an Peter.

Allerdings geht meine Uhr nach ca. 10 Minuten um 2 Sekunden nach. Kann 
es sein dass der externe Quarz so ungenau ist?

Hier der Code. Und es läuft nur dieser Code um Nebeneffekte 
auszuschliessen. Kein weiterer Timer etc. Fuses usw. passen auch.

#define XTAL    8000000L
#define DEBOUNCE  256L
#define uchar unsigned char
#define uint unsigned int
uchar prescaler;
uint volatile second;

ISR(TIMER1_COMPA_vect) {
#if XTAL % DEBOUNCE
  OCR1A = XTAL / DEBOUNCE - 1;
#endif
  if( --prescaler == 0 ) {
    prescaler = (uchar)DEBOUNCE;
    second++;
#if XTAL % DEBOUNCE
    OCR1A = XTAL / DEBOUNCE + XTAL % DEBOUNCE - 1;
#endif
  }

}

// Timer Init

TCCR1B = (1<<WGM12) | (1<<CS10);
OCR1A = XTAL / DEBOUNCE - 1;
TCNT1 = 0;
prescaler = (uchar)DEBOUNCE;
second = 0;

TIMSK1 = (1<<OCIE1A);

sei();

In der main setze ich second auf 0 wenn > 0 und mache eine UART Ausgabe

Danke und Grüße

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Werner P. schrieb:
> Hier der Code. Und es läuft nur dieser Code
Mein Compiler meldet einen Fehler mit diesem Code...

> In der main setze ich second auf 0 wenn > 0
Dann lass doch die main auch noch sehen.
Vielleicht solltest du second auf 0 setzen, sondern nur 1 abziehen?
Was, wenn da second schon auf 2 war?

[Werner P. (Gast)]

Dann mal der komplette Source.

@Lothar

Wie soll second auf 2 kommen. Die Interrupt Routine wird nur 1x pro 
Sekunde aufgerufen. So lange kann der Code in der main loop gar nicht 
dauern.

[Marwin (Gast)]

Sorry, aber ich finde es total daneben, voellig undokumentierten Code zu 
posten und von anderen zu erwarten, dass sie sich da einlesen um Fehler 
zu finden. Auch wenn der Code so kurz ist wie hier.

Wenn du eine genaue Sekunde brauchst, dann setze einen Timer auf, der im 
Sekundentakt laeuft. Fertig. Der Rest ist, ohne weitere Erklaerung, 
voellig ueberfluessig.

[Peter II (Gast)]

Marwin schrieb:
> Sorry, aber ich finde es total daneben, voellig undokumentierten Code zu
> posten und von anderen zu erwarten, dass sie sich da einlesen um Fehler
> zu finden. Auch wenn der Code so kurz ist wie hier.

was soll hier noch dokumentiert werden?

> sei();  //IRQ einschaltern


@Werner P.
warum lässt du den code nicht einfach mal im Simulator laufen, dann 
diese du wie lange deine sekunde ist. Wenn es dort stimmt, dann wird es 
wohl an der Hardware liegen.

[Jürgen S. (starblue)]

Läuft eventuell der RC-Oszillator?

2 Sekunden auf 10 Minuten = 600 Sekunden sind 1/300 ~ 3000ppm,
so ungenau ist normalerweise kein Quarz.

[Werner P. (Gast)]

@marwin
sei mir nicht böse, aber was muss an dem code dokumentiert werden?

im prinzip geht es darum ob der timer richtig initialisiert wird. und 
ich denke das wird er.

ich habe den code von peter verwendet um zu testen ob meine 
initialisierung passt. beide varianten weichen in 10 minuten um ca. 2 
sekunden ab.

Jetzt mal meine Variante. F_CPU = 8000000UL

// Timer1 Init in main
// Alle 1/10 Sekunde
TCCR1B |= (1<<WGM12); // CTC Modus
TCCR1B |= (1<<CS10) | (1<<CS11); // Prescaler 64
OCR1A = 12500; // Top Wert
TIMSK1 = (1<<OCIE1A); // Interrupt ein


// Timer1 Interrupt
ISR(TIMER1_COMPA_vect){
  // Alle 1/10 Sekunde
  // second ist als globale volatile uint8_t variable definiert
  static uint8_t timer_tick;

  // @mawin. Muss ich das auch dokumentieren? ;-)
  timer_tick++;
  if (timer_tick == 10) {
    timer_tick = 0;
    second++;
  }
}

Setze ich OCR1A auf 12500 - 22 dann ist die Abweichung geringer. Also 
muss doch der Quarz das Problem sein.

[Werner P. (Gast)]

@Jürgen S.

es läuft definitiv der extern Quarz. Habe die Fuses bestimmt 10 mal 
gecheckt. Siehe Anhang.

[eli (Gast)]

Da wir hier im Markt sind, könnte ich dir 2 sekunden verkaufen.
Dann sollte deine Uhr wieder richtig gehen.

:-)

[Marcus O. (marcus6100)]

Was auch immer der Müll eigentlich machen soll, DEBOUNCE = 256L an einen 
unsigned char zuzuweisen ist schon grober Unfug.