Genaue Registerdefinition eines PCF8583

[Georg T. (microschorsch)]

Hallo zusammen,
ich habe für ein Projekt einen PCF8583 eingebunden. Ich habe 
Schwierigkeiten mit der Definition der Register. Im Datenblatt steht das 
auch nicht so genau.  Ich benötige quasi eine Tabelle welches Bit, 
welche Infos bereithält. Auf diese Sache mit den Einer- und Zehnerwerten 
bin ich natürlich auch schon gekommen, aber momentan bekomme ich Minuten 
größer 60...

Zum Dekodieren habe ich folgende Funktionen entwickelt, da muss irendwo 
was nicht stimmen:

char * I2CreadDateTime (void) { 
  unsigned char i;
 
  I2CstartCondition();
  I2CsendByte (WRITE_PCF);
  I2CsendByte (0x02);

  I2CstartCondition();
  I2CsendByte(READ_PCF);
  for (i = 0; i < 5; i++)
    data[i] = I2CgetByte();
  I2CstopCondition();
  return &data;
}
 
void I2CwriteDateTime (char myData[]) { 
  unsigned char i;
  
  I2CstartCondition();
  I2CsendByte (WRITE_PCF);
  I2CsendByte (0x02);

  for (i = 0; i < 5; i++)
    I2CsendByte (myData[i]);
  I2CstopCondition();
}

char * I2CgetDaTime() {
  char out[18];
  uint8_t year;
  uint8_t month;
  uint8_t day;
  uint8_t hour;
  uint8_t min;
  uint8_t sec;
  
  I2CreadDateTime();
  sec = (data[0] & 0x0f) + 10*((data[0] >> 4) & 0x07);
  min = (data[1] & 0x0f) + 10*((data[1] >> 4) & 0x07);
  hour = (data[2] & 0x0f) + 10*((data[2] >> 4) & 0x03);
  day = (data[3] & 0x0f) + 10*((data[3] >> 4) & 0x03);
  month = (data[4] & 0x0f) + 10*((data[4] >> 4) & 0x03);
  year = ((data[3] >> 6) & 0x03);
  
  sprintf(out,"%02d:%02d:%02d",hour,min,sec);
  return &out;
  
  
}

void I2CsetTime(uint8_t hour, uint8_t min, uint8_t sec) {
  data[0] = (sec % 10) | (((sec/10)*10)<<4);
  data[1] = (min % 10) | (((min/10)*10)<<4);
  data[2] = (hour % 10) | (((hour/10)*10)<<4);
  
  I2CwriteDateTime(&data);
  
}

hab ich irgendwelche Bits verschoben?

Danke
Schorsch

[(prx) A. K. (prx)]

> Im Datenblatt steht das auch nicht so genau.

Hab hier eines mit 28 Seiten, da steht alles genau genug drin.

[(prx) A. K. (prx)]

Georg T. schrieb:
> char * I2CgetDaTime() {
>   char out[18];
...
>   return &out;
> }

Lokale Variablen verlieren ihr Gültigkeit mit dem Ende der Funktion, 
sofern nicht "static". Ausserdem ist das "&" falsch.

[Georg T. (microschorsch)]

das stimmt, danke


...beantwortet aber nicht die eigentliche Frage....

Schorsch

[(prx) A. K. (prx)]

Georg T. schrieb:
> ...beantwortet aber nicht die eigentliche Frage....

Ich kann nur den Code beurteilen, den ich sehe. Und der ist nicht 
geeignet, eine Aussage über den Inhalt der Register des PCF zu treffen, 
wenn out[] die Grundlage dieser Aussage sein sollte.

Läuft der PCF überhaupt?

[(prx) A. K. (prx)]

Georg T. schrieb:
> ...beantwortet aber nicht die eigentliche Frage....

Also gut: nein.

[Jobst Q. (joquis)]

Siehe

Beitrag "RTC PCF8583 in C"

Im Zip findest du PCF8583.h, da steht alles drin.

[Mw E. (Firma: fritzler-avr.de) (fritzler)]

Im Datenblatt steht das genaustens, weis jetzt nich wie man das 
übersehen kann?

Ansonsten eben ne fertige Lib nutzen:
http://www.fritzler-avr.de/HP/Librarys/PCF8583_his.php

Außerdem hat A.K. dir schon den schlimmsten Fehler gezeigt, der auch 
alles kaputt macht.

[(prx) A. K. (prx)]

Georg T. schrieb:
>   data[0] = (sec % 10) | (((sec/10)*10)<<4);
>   data[1] = (min % 10) | (((min/10)*10)<<4);
>   data[2] = (hour % 10) | (((hour/10)*10)<<4);

Da solltest du nochmal drüber nachdenken. Vorher Mist reinschreiben kann 
schon dazu führen, dass hinterher Mist rauskommt.

[Georg T. (microschorsch)]

Hallo zusammen,

danke für Eure Hilfe.
Hier war in der Tat der Fehler, habe jetzt:

void I2CsetTime(uint8_t hour, uint8_t min, uint8_t sec) {
  data[0] = (sec % 10) | ((sec/10)<<4);
  data[1] = (min % 10) | ((min/10)<<4);
  data[2] = (hour % 10) | ((hour/10)<<4);
  
  I2CwriteDateTime(&data);
  
}

hier war ein "mal 10" zuviel drin. Hatte ich einen Denkfehler.

scheint jetzt zu laufen

Schorsch