Uhrzeit - Denkfehler oder Datentyp?

Ungefähr 17:00? Nicht vielleicht 18:12:16? (18*3600+12*60+16 == 65536) 
-- riecht nach einem Überlauf.
Außerdem wird dir der Modulo-Operator (%) sehr weiterhelfen.
Ich würde deine Berechnung ein bisschen umstellen:

1

/* Prototyp */

2

uint8_t empfangen(void);    //Besser einen sprechnederen Namen nehmen

3

                            // sowas wie get_rtc_byte oder so

4

uint32_t sekunden;

5

uint_8 t hh, mm, sec;

6

7

/* WENN empfangen() genau ein Byte liefert: */

8

sekunden = empfangen();                //LS Byte empfangen

9

sekunden |= (uint32_t)empfangen()<<8;  //..

10

sekunden |= (uint32_t)empfangen()<<16; //..

11

sekunden |= (uint32_t)empfangen()<<24; //MS Byte empfangen

12

13

hh = sekunden/3600;

14

sekunden %= 3600 ;      // Rest

15

mm = (sekunden)/60;

16

sec = sekunden % 60;

hth. Jörg

Ich Depp... Natürlich "|" statt "+"... Okay, das ist besser, 
funktioniert aber immer noch nicht wirklich. Scheint wohl etwas mit der 
Bereichsüberschreitung zu tun zu haben. Muss irgendwie rauskriegen, wie 
ich aus den 4 einzelnen Bytes, eine 4-Byte-var machte, die der AVR 
arithmetisch beackern kann. Wenn ich Deinen Code mal übertrage, bekomme 
ich jensets von 00:04:15 (das sind 255 (!) sekunden) sozusagen einen 
"Neustart" der Uhr. Aber besten Dank schonmal für den oder-hint!

Ein kleiner Tipp: Sowas mach ich immer im Simulator in dem ich mir ein 
entsprechendes Code-snippet zusammenbastle.

Gruß
Roland

man könnte hier auch gut mit einer union-struct arbeiten:

1

union

2

{

3

  uint8_t b[4];  // 4  Bytes

4

  uint32_t i;     // belegen den sellben Speicher wie i

5

} u;

6

7

unsignet cnt;

8

9

for (cnt = 0; cnt < 4; cnt++) u.b[cnt] = empfangen(); // 4x empfangen

10

11

hh = u.i / 3600;

12

...

(untested)

Gruß
Roland

Okiedo... Hier noch mal der aktuelle Code, der bei 255 sekunden "schlapp 
macht". Danach sagt die Uhr wieder 0:00:00 und fängt von vorne an.

1

uint8_t ow_rtc_1st=~(_1w_receivebyte());

2

uint8_t ow_rtc_2nd=~(_1w_receivebyte());

3

uint8_t ow_rtc_3rd=~(_1w_receivebyte());

4

uint8_t ow_rtc_4th=~(_1w_receivebyte());

5

6

7

  uint32_t seconds=ow_rtc_4th;      //MSB

8

  seconds=(seconds>>8)|ow_rtc_3rd;

9

  seconds=(seconds>>8)|ow_rtc_2nd;

10

  seconds=(seconds>>8)|ow_rtc_1st;    //LSB

11

12

13

  uint8_t hh = seconds/3600;

14

  seconds%=3600 ;      // Rest

15

  uint8_t mm = seconds/60;

16

  uint8_t ss = seconds%60;

du solltest lieber nach links schieben

Eigentlich schieb ich auch nach links, hab nur in meiner Verzweiflung 
das mal mit rechts durchgetestet und vergessen, es wieder zu ändern. In 
Echtigkeit schieb ich nach links.

1

uint8_t ow_rtc_1st=~(_1w_receivebyte());

2

  uint8_t ow_rtc_2nd=~(_1w_receivebyte());

3

  uint8_t ow_rtc_3rd=~(_1w_receivebyte());

4

  uint8_t ow_rtc_4th=~(_1w_receivebyte());

5

  _1w_detectpresence();

6

7

  uint32_t seconds=ow_rtc_4th;      //MSB

8

  seconds=(seconds<<8)|ow_rtc_3rd;

9

  seconds=(seconds<<8)|ow_rtc_2nd;

10

  seconds=(seconds<<8)|ow_rtc_1st;    //LSB

11

12

13

  uint8_t hh = seconds/3600;

14

  seconds%=3600 ;      // Rest

15

  uint8_t mm = seconds/60;

16

  uint8_t ss = seconds%60;

Kann es sein das du hier alle bits invertierts (~) ?

uint8_t ow_rtc_1st=~(_1w_receivebyte());
  uint8_t ow_rtc_2nd=~(_1w_receivebyte());
  uint8_t ow_rtc_3rd=~(_1w_receivebyte());
  uint8_t ow_rtc_4th=~(_1w_receivebyte());

versuch es doch mal so

  uint8_t ow_rtc_1st=(_1w_receivebyte());
  uint8_t ow_rtc_2nd=(_1w_receivebyte());
  uint8_t ow_rtc_3rd=(_1w_receivebyte());
  uint8_t ow_rtc_4th=(_1w_receivebyte());


Gruss Helmi

Ja, ich invertiere alle Bits, aber das ist Absicht. Wie gesagt, an der 
Kommunikation hängts nicht, sondern eher an der "Zusammenführung" oder 
Berechnung...

die Berechnung ist meiner Meinung nach richtig, also

entweder es kommen andere Daten als du erwartest,
oder die Ausgabe der Ergebnisse ist falsch!

Wie ist denn das genaue Verhalten der Zeitanzeige?

Ist das Bei-255-Sekunden-Schlapp-Mach-Problem immer noch da, oder lag
das nur an der verkehrten Schieberichtung? In meinen Augen scheint die
Umrechnung der Sekunden in hh, mm und ss korrekt zu sein.

Ein möglicher Fehler liegt aber im Einlesen der 4 Bytes: Hast du
sichergestellt, dass sich die Werte während des Einlesens nicht
ändern? Bspw. dadurch, dass du die 4 Bytes auf Hardwareseite latchst?
Sonst kann (und wird) es hin und wieder passieren, dass der
Sekundenwechsel genau während des Einlesens der Bytes stattfindet.
Die gelesenen Werte sind dann falsch.

Kontrollieren kannst du dies, indem du die 4 Bytes ein zweites Mal
einliest. Haben sie den gleichen Wert wie beim ersten Mal, sind die
Werte ok. Wenn nicht, hast du einen Sekundensprung erwischt. Die Werte
müssen dann korrigiert oder - und das ist die sicherere Methode - ein
drittes Mal eingelesen werden.

Dann gibt die Daten doch einmal einzeln aus also ow_rtc_1st .. 
ow_rtc_4st und dann die zusammenfassung seconds

wenn in z.B. in ow_rtc_1st = 0x12
                ow_rtc_2st = 0x34
                ow_rtc_3st = 0x56
                ow_rtc_4st = 0x78

dann muss in Seconds = 0x78563412 sein

wenn das dann gleich ist müsste die zusammenfassung ok sein

Gruss helmi