Timer "fehler" mit tiny 2313

[Flops (Gast)]

Hallo µC Community,

gestern habe ich versucht einen Timer mit einem attiny 2313 zu 
realisieren. Über die PB3 und PB4 sind mit Transistoren LEDs 
angeschlossen, die ich durch de- und aktivieren der Pins an- und 
ausschalten kann.

Die LEDs sollen 12h brennen und dann für 12h aus sein. Wenn ich 
allerdings abends um 17:00 Uhr die Schaltung anschalte und morgens um 
7:00 Uhr schaue ob die LEDs brennen, dann sind diese immer noch an.

Wo liegt mein Fehler? Der AVR läuft mit 8Mhz und hier ist mein 
Sourcecode:

#include <avr/io.h>
#include <avr/interrupt.h>

int stunden = 12*60*60;
int count_isr = 0;
int seconds = 0;

void initTimer() {
  TCNT0 = 0;  // reset timer
  TIMSK |= (1 << TOIE0);  // enable overflow interrupt
  TCCR0B |= (1 << CS00);
}

int main(void)
{
    /* insert your hardware initialization here */
    DDRB = 0xff;  // alles als ausgang
    DDRD = 0xff;  // alles als ausgang
    sei();
    initTimer();
    PORTD |= (1 << PD3);
    //PORTB |= (1 << PB2);
  //PORTB |= (1 << PB3);

    for(;;){
        /* insert your main loop code here */
        //PORTD ^= (1 << PD3);
    }
    return 0;   /* never reached */
}

ISR(TIMER0_OVF_vect) {
  // eine sekunde ist rum, wenn die ISR 31250 mal aufgerufen wurde
  count_isr++;
  if(count_isr >= 31250) {
    count_isr = 0;
    seconds++;
  }
  if(seconds >= stunden) {
        seconds = 0;
        PORTB ^= (1 << PB2);
    PORTB ^= (1 << PB3);
  }
}

Vielen Dank

Flops

[Falk B. (falk)]

@  Flops (Gast)

>Die LEDs sollen 12h brennen

leuchten. Brennen tun nur Kerzen und manchmal der Weihnachtsbaum ;-)

>und dann für 12h aus sein. Wenn ich
>allerdings abends um 17:00 Uhr die Schaltung anschalte und morgens um
>7:00 Uhr schaue ob die LEDs brennen, dann sind diese immer noch an.

>Wo liegt mein Fehler?

Du musst erstmal in kleinerem Zeitmassstab testen. Z.B. 1 Minute.

> Der AVR läuft mit 8Mhz

Interner RC-Oszillator? Ist eher ungenau. Für genauere Sachen braucht 
man einen Quarz.

http://www.mikrocontroller.net/articles/AVR-Tutorial:_Equipment#Erg.C3.A4nzende_Hinweise_zur_Taktversorgung_.28kann_.C3.BCbersprungen_werden.29

>und hier ist mein Sourcecode:

Sieht auf den ersten Blick OK aus.

MFG
Falk

[Klaus 2. (klaus2m5)]

Zum gefühlten eintausendsten Mal:

Fuses, speziell CKDIV8

[Flops (Gast)]

Meine Fuses sind -U lfuse:w:0xe4:m -U hfuse:w:0xdf:m, CKDIV8 ist also 
nicht gesetzt.

[Karl H. (kbuchegg)]

Und zum gefühlt 500-tausendsten mal: Datentypen und deren Wertebereiche!

int stunden = 12*60*60;

geht schon schief. 12*60*60 ergibt 43200
int geht aber 'nur' von -32768 bis 32767.
Mit uint16_t wärs zumindest in dem Punkt gegangen.

[Flops (Gast)]

Oh danke, ich hab darüber nachgedacht aber letztenendes nicht 
nachgeschaut, wie groß int wirklich ist.

Es wäre also besser ein unsigned int zu verwenden, oder?

[Karl H. (kbuchegg)]

Flops schrieb:
> Oh danke, ich hab darüber nachgedacht aber letztenendes nicht
> nachgeschaut, wie groß int wirklich ist.
>
> Es wäre also besser ein unsigned int zu verwenden, oder?

1) Es ist auf einem µC fast immer besser einen unsigned Datentyp
   zu nehmen. Es sei denn, man braucht explizit und zwingend ein
   Vorzeichen.

2) Es ist auf einem µC wie den Tinys und Megas fast immer besser, die
   Bitzahl explizit vorzugeben, anstatt sie dem Compiler zu überlassen
   Also: uint16_t oder uint8_t anstelle von unsigned int

3) Anrüchige Berechnungen mit der Tendenz große Zahlen zu liefern
   IMMER mit dem Maximalwerten kontrollieren und nachrechnen.
   Overflows sind im Nachhinein nur schwer zu finden.

[Flops (Gast)]

Als ich heute morgen geschaut habe waren die LEDs aus, es lag wohl also 
tatsächlich an einem falschen Datentyp.

Nochmal vielen Dank für die Hilfe,

Flo

[Oliver (Gast)]

"Aus" ist als Funktionskontrolle schlecht geeignet, vielleicht ist ja 
auch einfach was kaputt gegangen. Du solltest schon noch wieder warten, 
ob die LEDs in zur erwarteten Zeit auch wieder an gehen.

Oliver

[Karl H. (kbuchegg)]

Die 12 Stunden werden auch nicht exakt stimmen. Noch nicht mal dann, 
wenn du einen Quarz benutzt.

Schon alleine deshalb solltest du versuchen, den exakten Zeitpunkt 
vorherzusagen und zu kontrollieren, um festzustellen, wie groß die 
Abweichung tatsächlich ist.