Selbstlaufende Uhr mit AVR Atmega8

Mike32 schrieb:
> An den Mikrokontroller habe ich einen 8 Megaherz Quarz angeschlossen.

... und hoffentlich auch aktiviert.

Gruss Klaus

>   if (neuse==altse)
>   {

Wieso "gleich"?


Der Update muss erfolgen, wenn sich was verändert hat, wenn die Werte 
also ungleich sind!

>     for(a=0;a==9;a++)      //Schieberegister auf 0 setzten


Schau bitte in ein C Buch deiner Wahl, wie die Einzelteile einer 
for-Schleife funktionieren.

Das ist

  for( initial; Schleifenbedingung; Inkrement )

Solange die Schleifenbedingung wahr ergibt (also ungleich 0) wird die 
Schleife weiter ausgeführt. Bei dir wäre das: solange a den Wert 9 hat, 
wird die Schleife ausgeführt. a wird aber nie 9 sein, denn du 
initialisierst a mit 0. 0 ist aber nicht gleich 9 und damit kommt die 
Schleife nie zur Ausführung.

Vergiss momentan mal die Zehener und Einer, Minuten und Stunden. Sieh 
zu, dass du EIN Schieberegister in Betrieb nimmst. In deinem Code 
wimmelt es nur so von lächerlichen C-Fehlern.

Volatile schüttet man nicht mit der Gießkanne aus.
Volatile sind ausschließlich die Variablen, die sowohl im Interrupt, als 
auch im Main verwendet werden.
Und Konstanten schonmal garnicht.


Peter

Mike32 schrieb:
> So, ich habe nun mal ein paar Sachen geändert.

Wo?

Ich sehe immer noch die alten Fehler.

>   if (neuse==altse)
>     for(a=0;a==b;a++)      //Schieberegister weiter "schieben"
>   if (neusz==altsz)

usw. usw.

Soll ich dir jetzt wirklich jede einzelne Zeile raussuchen, damit du die 
immer gleichen Fehler auch wirklich alle korrigierst?

Mike32 schrieb:
> Habe dort nur 9 Led's. Danach soll in den Zehner Sekunden Block eine LED
> angehen.

Wenn du meinst.
Dann zählt deine Uhr eben schon auf 10, wenn in der Realität nur 9 
Sekunden vergangen sind. Tipp: Auch 0, wenn also keine LED brennt, ist 
ein anzeigenswerte Zahl.

Aber das ist momentan noch nicht so wichtig. (Bei Minuten und Stunden 
hast du denselben Quatsch. Deine Minute zb hat nur 49 Sekunden, sogar 
weniger, weil ja pro 10 Sekunden 1 fehlt).

Erst mal kriegst du das jetzt zum Laufen. Um die korrekte Zählweise 
kannst du dich später kümmern.

Was hast du da eigentlich mit deinem
"Schieberegister auf 0 setzen"?

Braucht doch kein Mensch. Du hast doch an einem Schieberegisterstrang, 
so wie ich das im Code sehe, sowieso immer nur die Leds einer 'Spalte' 
(also Zehner oder Einer). Schieb deine Zahlen raus und gut ists.
Du musst 9 LED für die Einer setzen: Wenn die Schleifenvariable kleiner 
als die aktuelle Sekundenzahl ist, dann schiebst du ein 0 Bit raus und 
ansonsten ein 1 Bit.
Und das ganze machst du nur, wenn du musst. Sprich, wenn sich die Einer 
der Sekunden verändert haben.

Ich seh auch nicht, wo du in deinem Programm die Latch Enable bzw. 
Output Enable Leitungen der SR bedienst.


Hast du denn ein paar Vorversuche mit EINEM Schieberegister gemacht, um 
dich erst mal mit der prinzipiellen Vorgehensweise vertraut zu machen, 
ehe du dich gleich auf eine Uhr gestürzt hast?

Mike32 schrieb:

> Wenn nun 9 Leds leuchten, wollte ich dort 9x eine "0" rein schieben und
> dies schnell hintereinander damit alle neun leds wieder aus sind.

Ist doch viel zu kompliziert.
Du musst IMMER 9 Bits ins Schieberegister laden.
Die entscheidende Frage bei jedem Bit ist nur: muss es ein 0 Bit oder 
ein 1 Bit sein

1

  if (second != altse)

2

  {

3

   for( i = 0; i < 9; ++i )

4

   {

5

     if( i < second )

6

       PORTD &=~(1<<PORTD0);

7

     else

8

       PORTD |= (1<<PORTD0);

9

10

      PORTD |=(1<<PORTD1);

11

      _delay_ms(1);

12

      PORTD &=~(1<<PORTD1);

13

    }

14

15

    altse = second;

16

  }

Mike32 schrieb:
> Den Latch Enable habe ich in der Schaltung dauerhaft auf High, den
> Output Anable dauerhaft auf Low gelegt.

Das heisst man sieht, wie du die Schieberegister befüllst :-)
Keine gute Idee.
Zumindest die Output Enable Leitung willst du steuern können!
Du willst die LED gezielt abschalten, dann die Schieberegister befüllen 
und danach die LED wieder einschalten. Damit man eben nicht sieht, wie 
du die Daten reintaktest.

Man könnte jetzt höchstens aus der Not eien Tugend machen, weil man ja 
weiß dass die LED nicht in beliebiger Reihenfolge angehen, sondern immer 
eine nach der anderen. Das verkompliziert aber das Programm ein wenig.

Mike32 schrieb:

> Oder habe ich hier einen Denkfehler?

Nein. Man kann das so machen.
Aber dein Programm geht noch nicht einmal ansatzweise in diese Richtung.

Es ist zwar nicht wahnsinnig schwer mit dieser Strategie die Uhr zu 
befüllen, aber so wie ich das sehe ist das für dich noch schwer genug.

Daher: machs erst mal konventionell
Bei jeder Änderung die entsprechende Anzahl an leuchtenden LED neu 
einstellen.

Und konzentrier dich für erste nur auf die Einsterstelle/Sekunden. Alles 
andere löscht du jetzt erst mal raus.

Wenn du die korrekt laufen hast, dann baust du die Zehnerstelle ein. Und 
wenn die dann geht, baust du die Minuten ein etc. etc.

Nicht zu viel auf einmal.