ATMega644 mit 7-Segmentanzeigen

[Axel L. (ligonap)]

Hallo Zusammenen,

ich möchte eine ATMega644 mit drei 7-Segmentanzeigen und einem/zwei
Schalter betreiben.
Irgendwie ist der Wurm in der Programmierung und ich sehen den Wald
vor Bäumen nicht.

Hier der Code:

//#########################################################
// Steuerung ATMega644 mit 16 MHz + drei 7-Segmentanzeigen
//#########################################################
#include <avr/io.h>
#include <avr/iom644.h>
#include <avr/interrupt.h>


//#########################################################
// Rahmenbedingungen
//#########################################################

// Counter
uint8_t muxflag;
uint8_t e, z, h;


//#########################################################
//Segmente für die Zahlenwerte Port A
//#########################################################

uint8_t num[] =
{
  0b01101111 ,   // 0
  0b00100100 ,   // 1
  0b01110011 ,   // 2
  0b01110110 ,   // 3
  0b00111110 ,   // 4
  0b01011110 ,   // 5
  0b01011111 ,   // 6
  0b01100100 ,   // 7
  0b01111111 ,   // 8 max LED-Anzahl
  0b01111110     // 9
};



//#########################################################
// Timer0 für Displayrefresh
void counter_init(void)
//#########################################################
{
  TCCR0B |= (1<<CS02) ; // Prescaler = FCPU/256  = 62500  Hz -> 81.38  Hz pro Segment
  TIMSK0 |= (1<<TOIE0) ; //  Overflow Interrupt Enable
  TCNT0=0 ;  // Init Counter - Zähler auf 0 setzen
}



//#########################################################
// ISR für Timer0
ISR(TIMER0_OVF_vect)
//#########################################################
{
  
  // Alle Segmente aus
  PORTC &= ~(1<<PC3);
  PORTC &= ~(1<<PC4);
  PORTC &= ~(1<<PC5);

  if(muxflag==0)
  {
  PORTA = num[h];
  PORTC |= (1<<PC3);
  }

  if(muxflag==1)
  {
  PORTA = num[z];
  PORTC |= (1<<PC4);
  }

  if(muxflag==2)
  {
  PORTA = num[e];
  PORTC |= (1<<PC5);
  
  muxflag=0;
  }
   else
  {
  muxflag = muxflag + 1;
  }
    
}


//#########################################################
/* Hauptprogramm */
int main(void)
//#########################################################
{

// Als Ausgänge und Eingänge sind Definiert:
DDRA = 0b11111111; // Counter-Anzeige
DDRB = 0b11111111;
DDRC = 0b00111000; // Segmente (PC3-PC5) 
DDRD = 0b10000000; // Tasterabfrage (PD2 & PD3)  

// Internal Pull-Up aktivieren
PORTD = 0b01111111;  // Taste PD2 & PD3
 

sei();          //enable interrupts


// Startwerte für 7-Segmente und Magazin
muxflag = 0;
h = 5; // 5-Hundert -> Zeigt 0 an
z = 0; // 0-Zehner
e = 0; // 0-Einer


counter_init(); // Timer0 ein


// Variantion 1

// while ( (PIND & (1<<PIND2))  ) ; // Taster an PB2 gedrückt?

// h = 4; // -> Zeigt 0 an. Wieso??

// while ( (PIND & (1<<PIND3))  ) ; // Taster an PB3 gedrückt?

// h = 6; // -> Zeigt 6 an oder jeden anderen gewählten Wert!


// Variantion 2

while ( (PIND & (1<<PIND3))  ) ; // Taster an PB3 gedrückt?

h = 4; // -> Zeigt 0 an. Wieso??

while ( (PIND & (1<<PIND2))  ) ; // Taster an PB2 gedrückt?

h = 7; // -> Zeigt 7 an oder jeden anderen gewählten Wert!


  
while(1); //Ende

}

Egal ob ich erst die Taste an PD2 und dann die Taste an PD3 drücke oder
umgekehrt, bis zum zweiten Tastendruck wird immer 000 angezeigt.

Keine Ahung. Irgendwie stehe ich auf dem Schlauch.

[MaWin (Gast)]

Du hast wohl noch nie was von der Notwendigkeit gehört, Taster zu 
entprellen.

Auch ist ein Programm, welches nach dem zweiten prellen in einem 
while(1); für immer stoppt (RIP), etwas unbrauchbar.

[Axel L. (ligonap)]

MaWin schrieb:
> Du hast wohl noch nie was von der Notwendigkeit gehört, Taster zu
> entprellen.
>
> Auch ist ein Programm, welches nach dem zweiten prellen in einem
> while(1); für immer stoppt (RIP), etwas unbrauchbar.


Da dies ein Testprogramm ist, soll am Ende alles Stoppen.

Entprellen wird ein wenig überschätzt. Brauchte ich bisher nicht und ist 
hier auch nicht notwenig. Ob Variante 1 oder Variante 2 oder erstmal 
keine Taste, es wird immer 000 angezeigt.

[Axel L. (ligonap)]

Ich habe die Lösung endlich gefunden. Es lag an den Rahmenbedingungen.

//###################################
// Rahmenbedingungen
//###################################

// Counter
volatile  uint8_t muxflag;
volatile  uint8_t h, z, e;

Da die Vaiablen h, z und e im Hauptprogramm geändert werden und diese 
Werte global gelten sollen, muss volatile verwendet werden.