Problem mit einem 1sec takt

[Sausy (Gast)]

Hallo erstmal
Also zu meinem Problem ich hab ein C programm für eine Uhr mit meinen
attiny 2313 geschrieben die formel ist doch bei 8 bit prescala 256
(4194304/256)/256/64 = 1Hz
d.h: 64 überläufe
aber um einen ungefären sec takt zu erzeugen muss ich nicht sechs 
überläufe zählen sondern 16 aber warum

/*
 * LED_Display.c
 *
 * Created: 31.08.2012 18:47:22
 *  Author: Sausy
 */


#define XTAL 4194304
#define F_CPU XTAL

#define HIGH  1
#define LOW   0

#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include "LED_Disp.h"

volatile int sec_temp;  // seconds counter
int overflow_c = 0;

ISR(TIMER0_OVF_vect)  // 1 second clock
{  if(overflow_c < 16)     //(4194304/256)/256 = 64 mal = 1sec
       overflow_c++;       //eigentlich 64 mal 

  if(overflow_c >= 16){    //für sec++
    overflow_c = 0;
    sec_temp++;
  }
}


int main(void)
{  int  hr=9, min=0, sec=0;   //set start time
  inite_LED_D();              //inizalisiert LED display

  DDRB &= ~(1<<PB0);
  PORTB &= ~(1<<PB0);
  DDRB |= (1<<PB1);


  //----[ initialize timer1 ]---
  TIMSK |= _BV(TOIE0);   // erlaubt overflow interupt --
                        //aktiviert den overflow interrupt des timer1
  TCCR0B |= (1<<CS02);  // prescaler = 256
  TCNT0 = 0x00;
  sei();

  while(1)
  {
    number_output (hr*100+min); // setz die Variablen
                                // für meinen Led display
    {  sec=tc;
      if (sec>=60)
      {  tc=0; sec=0; min++;
        if (min>=60)
        {  min=0; hr++;
          if (hr>=12) hr=0;
        }
      }
    }
    while (sec==tc)  refresh(); //refrescht das bild
  }
  return(0);
}

[g457 (Gast)]

64/16 - das klingt verdächtig nach Faktor 4.194304MHz/1MHz. ∗Überprüf∗ 
mal ob der t2313 wirklich mit Quarz läuft oder nicht vielleicht doch mit 
den Standardeinstellungen (8Mhz interner RC mit CKDIV8).

[Hmm (Gast)]

>#define XTAL 4194304

Für diese "Behauptung" sollte man das Teeren und Federn wieder 
einführen.

Vermutlich wird entweder Dein Takgenerator die vierfache Frequenz 
erzeugen oder Deine Teilerfaktoren sind falsch berechnet oder falsch 
gesetzt.

Prüf' es doch einfach nach! Was meinst Du was wir machen würden? Beten? 
Einen Regentanz?

[Werner (Gast)]

Bist du sicher, dass du deine Quarz überhaupt benutzt und der µC nicht 
mit seinem Auslieferungstakt von 1MHz (Takt 8MHz, Prescaler 8) läuft?

[amateur (Gast)]

Auf die Schnelle fallen mir zwei Sachen auf:
1. Zeitverschwendung in der Interrupt Routine.
2. Guck mal nach wie groß ein int ist. Mach dein Compiler daraus 2 
Bytes, so musst Du beim Lesen "sec_temp" aufpassen dass Dir keine 
Unterbrechung zwischen das Lesen der zwei Bytes kommt. Ein unsigned char 
reicht für 0 ... 60 auf jeden Fall aus und beseitigt dieses Problem. 
Darüber hinaus sparst Du Speicherplatz.

[Sausy (Gast)]

thx für die schnellen antworten

Ich werde mal das quarz überprüfen und ggf. austauschen
und sec_temp auf unsigned char ändern

[Karl H. (kbuchegg)]

Sausy schrieb:
> thx für die schnellen antworten
>
> Ich werde mal das quarz überprüfen und ggf. austauschen

Austauschen kannst du dir sparen.
Aktivier ihn lieber erst mal.

Falls er aktiv ist, dann läuft der auch. Sonst würde dein µC gar nicht 
arbeiten.

Aber die Vermutung ist eher: Der Quarz hängt zwar physisch am µC, aber 
dem µC hat keiner (mittels Fuse-Bits) mitgeteilt, dass da ein Quarz 
vorhanden ist. Also benutzt er ihn auch nicht.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

amateur schrieb:
> 2. Guck mal nach wie groß ein int ist. Mach dein Compiler daraus 2
> Bytes, so musst Du beim Lesen "sec_temp" aufpassen dass Dir keine
> Unterbrechung zwischen das Lesen der zwei Bytes kommt. Ein unsigned char
> reicht für 0 ... 60 auf jeden Fall aus und beseitigt dieses Problem.

Und das gilt m.E. auch für alle anderen Variablen in diesem Programm. 
uint8_t oder char reichen für alles hier. Nur etwaige Tage des Jahres 
oder das Jahr selber bräuchten words. Damit lösen sich etwaige atomic 
access Probleme in Luft auf.

[Troll (Gast)]

Karl Heinz Buchegger schrieb:
> Aber die Vermutung ist eher: Der Quarz hängt zwar physisch am µC, aber
> dem µC hat keiner (mittels Fuse-Bits) mitgeteilt, dass da ein Quarz
> vorhanden ist. Also benutzt er ihn auch nicht.

einfachster test am steck brett, ein langsam blickende LED - wenn du den 
Quarz trennst (aka rausziehen) bleibt die LED stehen

[amateur (Gast)]

@Matthias
Das stimmt zwar, aber die Anzahl an Variablen, die in der/einer 
Interruptroutine verändert werden und von außen befummelt werden, ist 
exakt gleich eins.

Sollten aber noch ein paar, hier nicht aufgeführte, da sein, so lautet 
mein Vorschlag: Optimieren sowie optimieren als auch optimieren.
Sollte das Teil aber nur Ticken und Anzeigen, so vergiss es.

[Karl H. (kbuchegg)]

amateur schrieb:
> @Matthias
> Das stimmt zwar, aber die Anzahl an Variablen, die in der/einer
> Interruptroutine verändert werden und von außen befummelt werden, ist
> exakt gleich eins.

Es geht weniger um das Zeitproblem, sondern um das Problem des atomaren 
Zugriffs.
Gut, da seine Variablen nur Wertebereiche kleiner 255 haben, ist das 
hier im konkreten kein Problem, weil das High-Byte sowieso immer 0 ist. 
Aber generell ist man gut beraten, wenn man den kleinsten Datentyp 
nimmt, der passend ist. Wenn geht, dann uint8_t oder int8_t. Und bei ihm 
geht problemlos.

Und über die Sache mit der 1 Variable reden wir mit dem TO noch mal, 
wenn er die Sache mit dem Takt auseinanderdividiert hat. Denn in dem 
Fall ist das nicht besonders schlau. Die Minuten und Stundenerhöhung 
gehört mit in die ISR hinein. Die Uhr muss als Ganzes interruptgetrieben 
weiterlaufen. Selbst dann, wenn in der Hauptschleife 2 Millionen 
teuflisch schwierige quadratische Gleichungen zu lösen sind und 
zwischendurch 58 _delay_ms( 10000 ) gemacht werden.

[Sausy (Gast)]

Also der sec takt funktioniert jetz

@ Karl Heinz Buchegger
>Aber die Vermutung ist eher: Der Quarz hängt zwar physisch am µC, aber
>dem µC hat keiner (mittels Fuse-Bits) mitgeteilt, dass da ein Quarz
>vorhanden ist. Also benutzt er ihn auch nicht.

Du hattest recht ich hatte vergessen die fuses zu setzen

P.S.: Die größe der varialen hab ich auch gändert


thx für alle antworten