Atmega8 Uhr problem

[K. Y. (Firma: privat) (avrfreak12)]

Hallo Leute,

Hier habe ich ein code für eine einfache Uhr zusammengestellt. Leider 
kriege ich nicht das was ich will am LCD. Die zahl der sekunden zahlt 
bis 59 oder 60 und springt zu 19 und aufwärts also nicht wieder von 
null.

Kann jemand mir helfen um festzustellen wo liegt ein Fehler im code.

Ich benütze ATmega8 @4 MHz, Genauigkeit ist mir jetzt nicht wichtig.
Ich lerne die TIMERx_OVF in Zahlen auf Display zu setzen.

Danke im Vorraus

\\Uhr Versuch, Atmega8. 4MHz.

#include <stdio.h>
#include <avr/delay.h>
#include <stdlib.h>
#include <avr/io.h>
#include <avr/pgmspace.h>
#include <avr/signal.h>
#include <avr/interrupt.h>
#include "lcd.h"


// define global variables 

volatile uint8_t Seconds;
volatile uint8_t Minutes;
volatile uint8_t Hours;
char buffer[4];

// ISR 
 ISR(TIMER1_OVF_vect)//
 {
    
  if (++Seconds ==60)
  {
    Seconds=0;
    
    if (++Minutes == 60)
    {
      Minutes = 0;

    
      if (++Hours == 24)
      {
        Hours = 0;

         Seconds=0;

  }
  
  }
  
  }

  }


  void init_timer(void)
  {
  
  TIMSK|=(1<<TOIE1);//enable overflow interrrupt.
  TCCR1B|=(1<<CS11);
  sei();    // enable global interrupts 
}


int main(void)
{
        
  

  init_timer();      // Timer init 
  lcd_init(LCD_DISP_ON);  // LCD-Display init 

    lcd_puts("Time");
    lcd_gotoxy(4,1);


    for (;;)        // Loop 
  
    {
    
      lcd_gotoxy(4,1);//
            lcd_puts(":");
      itoa(Hours, buffer,10);
      lcd_puts(buffer);
            

      lcd_gotoxy(7,1);
            lcd_puts(":");
        itoa(Minutes, buffer,10);
        lcd_puts(buffer);
      

      lcd_gotoxy(10,1);
            lcd_puts(":");
            itoa(Seconds,buffer,10);
          lcd_puts(buffer);
        
      }
}

[Karl H. (kbuchegg)]

wenn du am LCD den Text 59 mit dem Text 1 überschreibst, dann 
überschreibt die 1 nur die 5, die 9 bleibt weiterhin sichtbar und du 
liest dann 19.

gib nach der der 1 (oder generell nach dem text) noch ein Leerzeichen 
aus, damit auch eine eventuell Textstelle von vorher gelöscht wird.

[Torsten K. (ago)]

K. Yousef schrieb:
> if (++Hours == 24)
>       {
>         Hours = 0;
>
>          Seconds=0;

"Seconds" brauchst du an dieser Stelle nicht mehr (nochmal) auf 0 
setzen.

[Helfer (Gast)]

// Uhr Versuch, Atmega8. 4MHz.

#include <stdlib.h>
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/atomic.h>
#include "lcd.h"

// define global variables 

volatile uint8_t Seconds;
volatile uint8_t Minutes;
volatile uint8_t Hours;
char buffer[4];

ISR(TIMER1_OVF_vect)
{
  if( ++Seconds == 60 ) {
    Seconds = 0;
    if( ++Minutes == 60 ) {
      Minutes = 0;
      if( ++Hours == 24 ) {
        Hours = 0;
      }
    } 
  }
}

void init_timer(void)
{
  TIMSK |= (1<<TOIE1); //enable overflow interrrupt.
  TCCR1B |= (1<<CS11);
  sei(); // enable global interrupts 
}

int main(void)
{
  init_timer();
  lcd_init(LCD_DISP_ON);
  lcd_puts("Time");

  for(;;) {
    uint8_t h;
    uint8_t m;
    uint8_t s;

    // Siehe Artikel Interrupt
    ATOMIC_BLOCK(ATOMIC_FORCEON)
    {
      h = Hours;
      m = Minutes;
      s = Seconds;
    }

    // "Time:HH"
    // nicht itoa() verwenden, weil itoa bei einstelligen 
    // Zahlen keine führenden Nullen erzeugt!
    lcd_gotoxy(4, 1);
    buffer[0] = ':';
    buffer[1] = h / 10 + '0'; // Zehnerstelle
    buffer[2] = h % 10 + '0'; // Einerstelle
    buffer[3] = '\0';
    lcd_puts(buffer);
            
    // "Time:HH:MM"
    lcd_gotoxy(7, 1);
    buffer[0] = ':';
    buffer[1] = m / 10 + '0';
    buffer[2] = m % 10 + '0';
    buffer[3] = '\0';
    lcd_puts(buffer);

    // "Time:HH:MM:SS"
    lcd_gotoxy(10, 1);
    buffer[0] = ':';
    buffer[1] = s / 10 + '0';
    buffer[2] = s % 10 + '0';
    buffer[3] = '\0';
    lcd_puts(buffer);
  }
}

[K. Y. (Firma: privat) (avrfreak12)]

Karl Heinz Buchegger schrieb:
> wenn du am LCD den Text 59 mit dem Text 1 überschreibst, dann
> überschreibt die 1 nur die 5, die 9 bleibt weiterhin sichtbar und du
> liest dann 19.
>
> gib nach der der 1 (oder generell nach dem text) noch ein Leerzeichen
> aus, damit auch eine eventuell Textstelle von vorher gelöscht wird.

Danke für die Hilfe ich werde es probieren.

[K. Y. (Firma: privat) (avrfreak12)]

Torsten K. schrieb:
> K. Yousef schrieb:
>> if (++Hours == 24)
>>       {
>>         Hours = 0;
>>
>>          Seconds=0;
>
> "Seconds" brauchst du an dieser Stelle nicht mehr (nochmal) auf 0
> setzen.

Danke auch für den Beitrag mir zu Hilfen, ich habe dabei gedacht etwas 
bleibt hängen im Puffer und dann kann man so löschen.

[K. Y. (Firma: privat) (avrfreak12)]

Helfer schrieb:
>

> // Uhr Versuch, Atmega8. 4MHz.
> 
> 
>

Recht herzlichen Dank, alles hat super geklappt. Here 
http://www.mikrocontroller.net/articles/Pollin_Funk-AVR-Evaluationsboard
wurde darüber etwas geschrieben aber ich habe das nicht wahrgenommen.

Danke nochmal.