Zähler gibt falschen wert aus

[Ma M. (monk3000)]

Hallo,

bin neu hier und habe noch nicht so viel Erfahrung mit conrollern und 
brauche ein bisschen Hilfe.

ich bin zur Zeit einen counter mit einem Atmega32 am bauen. Die Zeit 
soll über vier 7Seg Anzeigen ausgegeben werden (2 segmente minuten, 2 
segmente sekunden). Die Ausgänge des Controllers gehen auf einen 
BCD/7Segment converter(74HC453)und dann auf die einzelnen Elemente.

Mein Problem ist, dass die Anzeige die Zahlen 0,1,3,7 ausgibt und dann 
vermulich über 9 läuft. Das sieht für mich so aus, als ob die Werte 
nacheinander addiert werden anstatt die Werte immer mit eins zu 
addieren.
Wenn ich die Segmente mit festen Werten ansteuer werden die 
entsprechenden Zahlen richtig dargestellt.

seht Ihr villeicht in dem Programm einen fehler? (die zeit des Timers 
stimmt noch nicht)

----------------------------------------------------------------------

#define F_CPU 8000000UL
#include <avr/io.h>
#include <avr/interrupt.h>
volatile uint8_t counter = 0;
volatile uint8_t time_sec_ein;
volatile uint8_t time_sec_zehn;
volatile uint8_t time_min_ein;
volatile uint8_t time_min_zehn;
volatile uint8_t time_min_set;


ISR (TIMER0_OVF_vect)
{
     counter++;

}
int sec_ein()
{
     if (counter==50)
     {
         time_sec_ein++;
         counter = 0;
     }
     if (time_sec_ein == 10)
     {
         time_sec_ein = 0;
     }

}


int main(void)
{

     DDRA = 0xff;
     DDRC = 0xff;

     TIMSK |= (1 << TOIE0);
     TCCR0 |= (1<< CS00) | (1<< CS01);
     sei();

     while(1)
     {
     switch (time_sec_zehn)
     {
         case 1: PORTC |= (1<<PC4);break;
         case 2: PORTC |= (1<<PC6);break;
         case 3: PORTC |= (1<<PC4)|(1<<PC6);break;
         case 4: PORTC |= (1<<PC7);break;
         case 5: PORTC |= (1<<PC7)|(1<<PC4);break;
         case 6: PORTC |= (1<<PC7)|(1<<PC6);break;
         case 7: PORTC |= (1<<PC7)|(1<<PC6)|(1<<PC4);break;
         case 8: PORTC |= (1<<PC5);break;
         case 9: PORTC |= (1<<PC5)|(1<<PC4);break;

     }

     switch (time_sec_ein)
     {
         case 1: PORTC |= (1<<PC3);break;
         case 2: PORTC |= (1<<PC1);break;
         case 3: PORTC |= (1<<PC3)|(1<<PC1);break;
         case 4: PORTC |= (1<<PC0);break;
         case 5: PORTC |= (1<<PC0)|(1<<PC3);break;
         case 6: PORTC |= (1<<PC0)|(1<<PC1);break;
         case 7: PORTC |= (1<<PC0)|(1<<PC1)|(1<<PC3);break;
         case 8: PORTC |= (1<<PC2);break;
         case 9: PORTC |= (1<<PC2)|(1<<PC3);break;
     }


     switch (time_min_zehn)
     {
         case 1: PORTA |= (1<<PA3);break;
         case 2: PORTA |= (1<<PA1);break;
         case 3: PORTA |= (1<<PA3)|(1<<PA1);break;
         case 4: PORTA |= (1<<PA0);break;
         case 5: PORTA |= (1<<PA0)|(1<<PA3);break;
         case 6: PORTA |= (1<<PA0)|(1<<PA1);break;
         case 7: PORTA |= (1<<PA0)|(1<<PA1)|(1<<PA3);break;
         case 8: PORTA |= (1<<PA2);break;
         case 9: PORTA |= (1<<PA2)|(1<<PA3);break;
     }

     switch (time_min_ein)
     {
         case 1: PORTA |= (1<<PA4);break;
         case 2: PORTA |= (1<<PA6);break;
         case 3: PORTA |= (1<<PA4)|(1<<PA6);break;
         case 4: PORTA |= (1<<PA7);break;
         case 5: PORTA |= (1<<PA7)|(1<<PA4);break;
         case 6: PORTA |= (1<<PA7)|(1<<PA6);break;
         case 7: PORTA |= (1<<PA7)|(1<<PA6)|(1<<PA4);break;
         case 8: PORTA |= (1<<PA5);break;
         case 9: PORTA |= (1<<PA5)|(1<<PA4);break;
     }

     }
}

[4toTakoe (Gast)]

Und wo löschst du die Bits in den Port-Registern? Bei dir wird alles 
oder verknüpft...

[Peter II (Gast)]

sec_ein wird nie aufgerufen, damit können sie die Variabel nie ändern.

[dunno.. (Gast)]

Ich vermisse den Aufruf der Funktion

 int sec_ein();

Da kann doch so garnix zählen..?

Probier erstmal die Ausgabe deiner zahlen aus, zähl einfach die Zahl 
hoch und gib sie aus, damit du weißt ob deine Ansteuerung stimmt..

Außerdem ist die Lösung mit der ISR und der extra funktion nicht 
interruptsave. aber darum kannst du dich kümmern wenn die ausgabe 
richtig geht..

Benutze bitte beim Posten von Code das Code Highlighting.

Gruß