Atmega 32 Uhr läuft zu schnell

[Natureengeneer]

HI ich habe mal ein bisschen herumprobiert und verwende nun den Comparemode.

       #ifndef OCR1A
	#define OCR1A OCR1  // 2313 support
	#endif
	
	#ifndef WGM12
	#define WGM12 CTC1  // 2313 support
	#endif
	#define XTAL      8000000L   // nominal value
	#define DEBOUNCE    256L        // debounce clock (256Hz = 4msec)
	#define uchar unsigned char
	#define uint unsigned int
	
	uchar prescaler;

        TCCR1B = (1<<WGM12) | (1<<CS10);	// divide by 1
	// clear on compare
	TCNT1 = 0;                          // Timmer startet mit 0
	prescaler = (uchar)DEBOUNCE;        //software teiler
	TIMSK = 1<<OCIE1A;  

       SIGNAL (SIG_OUTPUT_COMPARE1A) {
	/************************************************************************/
	/*      Insert Key Debouncing Here      */
	/************************************************************************/
	
	if( --prescaler == 0 ){
		prescaler = (uchar) DEBOUNCE;
		ss++;//Addiere +1 zu Sekunden
		LED_burned++;
		ANALOG_ss++;
		if (ss == 60)
		{
			ss = 0;
			mm++;//Addiere +1 zu Minuten
			if (mm == 60)
			{
				mm = 0;
				hh++;//Addiere +1 zu Stunden
				if (hh == 24)
				{
					hh = 0;
				}
			}
		}
	}
	#if XTAL % DEBOUNCE
	if (prescaler <= XTAL % DEBOUNCE)
	OCR1A += XTAL / DEBOUNCE +1;   /* um 1 Takt längere Periode um den Rest abzutragen */
	else
	#endif
	OCR1A += XTAL / DEBOUNCE;   /* kurze Periode */
}

Das Erfreuliche: Die Uhr Läuft nicht mehr zu schnell, dafür nun aber leider zu langsam. (Abweichung nach 5h laufzeit ca. 30 minuten. [Uhr geht zurück])

Habe nun auch mal meine Fusesettings mitgepostet falls sich der Fehler vielleicht doch dort verbirgt.

Grundsetzlich ist es aber schon möglich den Timer1 comparemode für eine SoftwareClock (ohne große Abweichungen) zu verwenden und gleichzeitig eine lauffähiges UART Programm zur Kommunikation mit dem PC zu haben?
Weil großteils wird für den UART ja der Timer1 verwendet so wie ich das beurteilen kann...

[zschunky]

Was willst Du mit OCR1A += XTAL / DEBOUNCE +1; bzw OCR1A += XTAL / DEBOUNCE; bezwecken?

OCR1A sollte nur einmal beschrieben werden (also nicht im Interrupt) und nicht jedes mal neu. Im CTC zählt der Timer bis zu dem Wert, der im OCR1A steht und wenn er diesen Wert erreicht wird der counter automatisch auf 0 gesetzt und er fängt wieder von vorn an zu zählen (man muss also nichts von Hand wieder neu aufsetzen).

Setze mal den Prescaler auf 256 (8Mhz / 256 = 31.25KHz). Das heißt Du benötigst 31250 Takte damit eine Sekunde vorbei ist. Also Schreibt man einmalig 31250 in das OCR1A. Jetzt sollte der Interrupt jede Sekunde aufgerufen werden.

Die Timer sind komplett unabhängig von der UART und sollten sich nicht gegenseitig beeinflussen.

Bei den Fuses sehe ich keine Probleme.

[Natureengeneer]

OK ich hab den Debouncingpart weggemacht und Teste dann nun das Programm mit einem OCR1A Wert von 31250

Bericht folgt.

Vielen dank schonmal für die Hilfe.

[Natureengeneer]

Leider rennt die Uhr nun übertrieben schnell hoch.

	#ifndef OCR1A
	#define OCR1A OCR1  // 2313 support
	#endif
	
	#ifndef WGM12
	#define WGM12 CTC1  // 2313 support
	#endif
	#define XTAL      8000000L   // nominal value

	TCCR1B = (1<<WGM12)| (1<<CS10);		// divide by 1						
										// clear on compare
	OCR1A = 31250;						// Output Compare Register
	TCNT1 = 0;                          // Timmer startet mit 0
	TIMSK = 1<<OCIE1A;                  // beim Vergleichswertes Compare Match
	sei();

SIGNAL (SIG_OUTPUT_COMPARE1A) {
		ss++;//Addiere +1 zu Sekunden
		LED_burned++;
		ANALOG_ss++;
		if (ss == 60)
		{
			ss = 0;
			mm++;//Addiere +1 zu Minuten
			if (mm == 60)
			{
				mm = 0;
				hh++;//Addiere +1 zu Stunden
				if (hh == 24)
				{
					hh = 0;
				}
			}
		}
	

}

Was mach ich denn nur falsch

[Searcher]

Setze mal den Prescaler auf 256 (8Mhz / 256 = 31.25KHz).

zschunky hat den Prescaler schon vorgegeben, bei Dir steht was anderes.

    TCCR1B = (1<<WGM12)| (1<<CS10);        // divide by 1                        
                                        // clear on compare
    OCR1A = 31250;                        // Output Compare Register

Der genauere Wert für das OCR1A wäre 31249. (Formel im Datenblatt bei Timer1, Modes of Operation, CTC )

Gruß
Searcher

[Natureengeneer]

Also

TCCR1B = (1<<WGM12)| (1<<CS12);

THX

[Hubert.G]

Wie wäre es mit

TCCR1B = (1<<WGM12)| (1<<CS12);

Dann wäre der Prescaler auf 256 gestellt.

[Natureengeneer]

JO dankeschön.

Bin noch ein Noob was uC angeht nimmt Rücksicht.