Inkrementierung will nicht so recht

[Hubert.G]

if(~PINB & 1 << PB0)
Ich schreibe Pin-Abfragen immer so:
if(!(PINB&(1<<PB0)))

Das Variable die in einer ISR verwendet und verändert werden können, als volatile deklariert werden müssen ist schon klar.

[tirell]

Hallo Hubert, ja die Variablen sind global als volatile unsigned char deklariert.

[Bumbum]

Hallo tirell,

ist deine Abfrage in der main in einer Schleife? Dann kann es sein, dass die Minuten_Einer solange gezählt werden wie die Bedingung erfüllt ist. Da du sie in deiner Bitoperation nicht filterst zählen dadurch auch die Zehner, wenn der Wert in Minute_Einer über 15 ist.

Weiterhin ist mir noch aufgefallen, dass du PORTD "veroderst". Ich würde es wie folgt lösen:

PORTD = (Minuten_Zehner << 4) | Minuten_Einer & 0xF;

Damit hast du dann aber noch nicht die Ursache behoben. Wie gesagt aber nur, wenn die PINB Abfrage in einer Schleife liegt. In diesem Fall musst du nach der Inkrementierung warten, bis die Bedingung wieder "erloschen" ist.

Viele Grüße
Andreas

[tirell]

Alles klar, Problem gelöst. Du hast mich auf den richtigen Ansatz gebracht mit dem warten,
habe probehalber mal ein delay nach der Inkrementierung eingefügt und nun funktioniert es.

Hier ist nochmal der korrigierte Code

PORTD &= 0x00;				// output minutes ones and tens
PORTD = (s2 << 4) | (s1 & 0xF);		// output minutes ones and tens
PORTA &= 0x00; 				// output hours ones and tens
PORTA = (m2 << 4) | (m1 & 0xF);		// output hours ones and tens

while(1)
{
	if(!(PINB & (1 << PB0)))	// read input
	{
		m1 ++;
		_delay_ms(1000);
	}
	else{}
}

Wird Zeit die Tasten zu entprellen, dadurch sollte sich das Problem endgültig lösen.
Vielen Dank für die guten Ansätze!

[Bumbum]

Hallo tirell,

mit Tasten-Entprellen hat das Problem, dass du hast aber nichts zu tun. Ich denke kein Mensch kann die Taste nur einen Takt-Zyklus halten. Und auch das Delay ist nur eine Pfusch-Lösung. Du musst eine Sperre einbauen die das zählen so lange verhindert, bis PB0 wieder losgelassen wurde.
Und erst dann brauchst du natürlich noch eine Entprellung.

Viele Grüße
Andreas

[tirell]

Hallo, das stimmt natürlich. Habe mich an dem Tutorial hier bezüglich des Auslesens von Tasterbetätigungen orientiert und meinte das fälschlicherweise als Entprellung
Funktioniert jetzt alles bestens und wie erwünscht.