ATMEGA168 - Uhr

Henry schrieb:
> Hat jemand einen Vorschlag?

Ja. Schaltplan und Code posten.
Glaskugel ist leider gerade in der Abkühlphase.

Henry schrieb:
> Zählt der Timer auch dann hoch, wenn gerade eine
> ISR (Timer Overflow) abläuft, oder bleibt er währenddessen konst.?

Ja.

Wenn du deinen Code zeigst bekommst du mehr Hilfe.

Interner Takt ist eh zu Ungenau.

Der interne RC-Oszillator ist eigentlich nicht genau genug für eine Uhr. 
Nimm lieber einen Quartz.

Erstmal herzlichen Dank für die Antworten. Das die Uhr nicht genau geht 
ist bei den Voraussetzungen ja klar, aber wie schon erwidert können alle 
Abweichungen nicht ein derartigen Fehler erzeugen.

Den Schaltplan müsste ich erst noch "zeichnen". Daher poste ich zunächst 
nur den Code:

Ich gebe zu bedenken, dass ich harter Anfänger bin - der Programmierstil 
ist mit Sicherheit verbesserungswürdig:


#include <avr/io.h>
#include <stdint.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#ifndef F_CPU
#define F_CPU 8000000UL
#endif

volatile uint8_t min=0,hrs=0, i;
uint16_t counter=0;
uint8_t segment_array[4] ={0,0,0,0};


uint8_t debounce_h (volatile uint8_t *reg, uint8_t pin); //PB6
uint8_t debounce_m (volatile uint8_t *reg, uint8_t pin); //PB7
void  integer_split (volatile uint8_t stunden, uint8_t minuten);

int main (void)
{


DDRB= 0xFF;
DDRD= 0xFF; //Transistoren
DDRC &= ~(1<<PC5) | (1<<PC4); // Taster-Eingänge

TCCR0B |= ((1<<CS02)|(1<<CS00));
TIMSK0 |= (1<<TOIE0);
PCICR |= (1<<PCIE1);
PCMSK1 |= (1<<PCINT13) | (1<<PCINT12); // PCINT 5 und 4 aktiviert

sei();

while (1)
{

for (i=0;i<=3; i++)
  { PORTD = 0x00;
    PORTB = 0x00;

   switch (i)
     { case 0:
      PORTD |= (1<<PD0);
      break;

      case 1:
        PORTD |= (1<<PD1);
                  break;

      case 2:
        PORTD |= (1<<PD2);
      break;

      case 3:
        PORTD |= (1<<PD3);
      break;
    }

    switch (segment_array[i])
    { case 0:
      PORTB = 0b11111100;
      break;

      case 1:
            PORTB = 0b01100000;
      break;

      case 2:
      PORTB =  0b11011010;
      break;

      case 3:
      PORTB =  0b11110010;
      break;

      case 4:
      PORTB =  0b01100110;
      break;

      case 5:
      PORTB =  0b10110110;
      break;

      case 6:
      PORTB =  0b10111110;
      break;

      case 7:
      PORTB =  0b11100000;
      break;

      case 8:
      PORTB =  0b11111110;
      break;

      case 9:
      PORTB =  0b11110110;
      break;
       }

  }

}
return 0;
}


uint8_t debounce_h (volatile uint8_t *reg, uint8_t pin)
{cli();
 if ((*reg & (1<<pin))!=0)
  {  _delay_ms(30);
     _delay_ms(30);

  if ((*reg & (1<<pin))!=0)
  { _delay_ms(30);
    _delay_ms(30);
    return 1;
  }

  }
  return 0;
  sei();
}

uint8_t debounce_m (volatile uint8_t *reg, uint8_t pin)
{cli();
 if ((*reg & (1<<pin))!=0)
  {  _delay_ms(30);
     _delay_ms(30);

  if ((*reg & (1<<pin))!=0)
  { _delay_ms(30);
    _delay_ms(30);
    return 1;
  }
  }
  return 0;
  sei();
}

void integer_split (volatile uint8_t stunden, uint8_t minuten)
{  uint8_t k;
   k = stunden - (stunden%10);
   segment_array[0]= k/10;
   segment_array[1]= stunden%10;
   k = minuten - (minuten%10);
   segment_array[2]= k/10;
   segment_array[3]=minuten%10;
}


ISR(TIMER0_OVF_vect)
{counter++;

if (counter > 30)
{  if (min<59)
      { min++;
    }
   else
      { min = 0;
        hrs++;
      }

   if (hrs>23)
     {hrs=0;
     }
   counter=0;
integer_split(hrs, min);
}
}


ISR (PCINT1_vect)
{
   if (debounce_h(&PINC, PC5)==1)
      { hrs++;}
   if (debounce_m(&PINC, PC4)==1)
      { min++;}
   integer_split(hrs, min);
}


Im Code ist auch noch ein Fehler, die Variable "min" soll später mal die 
Minuten darstellen (habe nur vier Anzeigen HH:MM)

Den Vorschlag mit dem Delay werde ich schnellstmöglich mal testen.

Du darfst natürlich nur alle 60s die Minute inkrementieren.
Du machst es aber alle 30s.

Nimmt man an, das Du die CKDIV8 Fuse nicht zurückgesetzt hat, läuft der 
uC mit 1MHz. Das würde etwa den Faktor 3-4 (im Zusammenhang mit dem 
nicht korrekten 30s wg. 8MHz / 1024) erklären.

Oops. Da habe ich mich verguckt. Du inkrementierst die Minute jedenfalls 
denke ich alle Sekunde da Du 30 Überläufe pro Sekunde hast.
CKDIV8 würde ich auf jeden Fall angucken.

Hi!

Danke zunächst für die Antworten. Werde das Bit und den Takt nochmal 
überprüfen (mittels LED etc.).

Bezüglich der ISR habe ich noch eine Frage:

Ursprünglich wurde die Funktion "integer_split" in der main-schleife 
ausgeführt. Da hatte ich allerdings das Problem, dass die 
Segmentanzeigen nicht richtig funktionierten (Multiplex). Heißt, während 
der Ausführung der integer_split blieb die letzte Anzeige an. Die letzte 
Anzeige ist länger an als die anderen, so dass es fürs Auge aussieht als 
ginge nur die letzte.
Vielleicht sollte ich während der Abarbeitung des integer_split kein 
Signal auf die Anzeigen schicken....

Die überfüllte ISR braucht also u.U. derart lange, dass ich damit den 
eigentlich folgenden Overflow Interrupt verpasse?