DS 1307 Register defekt? Programmfehler ?

[Marcel B. (gigi)]

Hallo zusammen,

habe eine Uhr auf einem Atmega 8 programmiert die auch einige Zeit 
funktionierte.

Nun habe ich den Code der Übersicht wegen geändert und kann nun keine 
Zeit mehr auslesen, jedoch das Datum.

Anezige ist in etwa so :

[                ]
[ Datum: 21. 8.18]

Das Display bleibt in der ersten Zeile leer.
Komischerweise geht das Datum obwohl die Funktionen quasi identisch 
sind.
Zweites Problem:
Falls jemand weiss, wie ich die 0 auch als 0 anzeigen kann, würde ich 
mich über einen Tipp freuen.



Kann mir da jemand weiterhelfen ?

Vwerwendet werden die Libs von P. Fleury

Das Programm sieht so aus:

/*
 * LCD TWI.c
 *
 * Created: 20.08.2018 18:09:52
 * Author : Gigi
 */ 
#define F_CPU 8000000UL
#define Ds1307 0xD0  // DS1307 ID

#include <avr/io.h>
#include <stdio.h>
#include <util/delay.h>
#include "i2cmaster.h"
#include "lcd-routines.h"


void RTC_Init();
void RTC_GetDateTime();
void RTC_SetDateTime();
void clock_continue();


unsigned char sec, min, hour, weekDay, date, month,  year;
char zeit[32];
char datum[32];

int main() 
{
    lcd_init();
  RTC_Init();
  _delay_ms(5);
  
  hour = 0x20;   
    min =  0x51;
    sec =  0x00;
    date = 0x20; 
    month = 0x08;
    year = 0x18;
    weekDay = 1;
  _delay_ms(5);
  RTC_SetDateTime();
  _delay_ms(5); 
  _delay_ms(5);
  
    while(1)
    {  
        printf(zeit,"Zeit: %2x:%2x:%2x",hour,min,sec);
        lcd_setcursor(0,1);
    lcd_string(zeit); 
    sprintf(datum,"Datum: %2x.%2x.%2x",date,month,year);
    lcd_setcursor(0,2);
        lcd_string(datum);
    }
    return (0);
}

void RTC_Init(void)
{
  i2c_init();    
  i2c_start(Ds1307+I2C_WRITE);                           // Initialize the I2c module.
  
  i2c_write(0x07);            // Select the Ds1307 ControlRegister to configure Ds1307
  
  i2c_write(0x00);                        // Write 0x00 to Control register to disable SQW-Out
  
  i2c_stop();                             // Stop I2C communication after initializing DS1307
}

void RTC_SetDateTime()
{
  i2c_start(Ds1307+I2C_WRITE);                           // Start I2C communication
  
  i2c_write(0x00); // Request sec RAM address at 00H
  
  
  i2c_write(sec);                    // Write sec from RAM address 00H
  i2c_write(min);                    // Write min from RAM address 01H
  i2c_write(hour);                    // Write hour from RAM address 02H
  i2c_write(weekDay);                // Write weekDay on RAM address 03H
  i2c_write(date);                    // Write date on RAM address 04H
  i2c_write(month);                    // Write month on RAM address 05H
  i2c_write(year);                    // Write year on RAM address 06h
  
  i2c_stop();                              // Stop I2C communication after Setting the Date
}

void RTC_GetDateTime()
{
                            // Start I2C communication
  
  i2c_start(Ds1307+I2C_WRITE);        // connect to DS1307 by sending its ID on I2c Bus
  i2c_write(0x00);          // Request Sec RAM address at 00H
  i2c_stop();                         // Stop I2C communication after selecting Sec Register
  
                               
  i2c_start(Ds1307+I2C_READ);            // connect to DS1307(Read mode) by sending its ID
  
  sec = i2c_read(1);                // read second and return Positive ACK
  min = i2c_read(1);                 // read minute and return Positive ACK
  hour= i2c_read(1);               // read hour and return Negative/No ACK
  weekDay = i2c_read(1);           // read weekDay and return Positive ACK
  date= i2c_read(1);              // read Date and return Positive ACK
  month=i2c_read(1);            // read Month and return Positive ACK
  year =i2c_read(0);             // read Year and return Negative/No ACK
  i2c_stop();                              // Stop I2C communication after reading the Date
}

void clock_continue()
{
  _delay_ms(1000);
  sec ++;                
  if (sec == 0x3B) {_delay_ms(1000); min ++; sec = 0x00;}
  if (min == 0x3B && sec == 0x3B) {_delay_ms(1000);  RTC_GetDateTime();}  
}

[Heinz R. (Gast)]

 while(1)
    {  
        printf(zeit,"Zeit: %2x:%2x:%2x",hour,min,sec);

versuch mal sprintf(), wie die Zeile drunter beim Datum

[Marcel B. (gigi)]

Oh Mann!

Vielen lieben Dank!
Das "s" muss mir abhanden gekommen sein!

LG,
Marcel

[Marcel B. (gigi)]

Gut, das Porgramm läuft jetzt.

Allerdings bekomme ich die Zahlen nicht vernünftig ausgegeben, bei jedem 
Überlauf der Sekunden zählt er in Buchstaben weiter.

Wie löst man dieses Problem am besten ?

[guest (Gast)]

Marcel B. schrieb:
> Allerdings bekomme ich die Zahlen nicht vernünftig ausgegeben, bei jedem
> Überlauf der Sekunden zählt er in Buchstaben weiter.
>
> Wie löst man dieses Problem am besten ?

Was erwartest Du?
Du gibst sämtliche Werte hexadezimal aus und wunderst Dich dann über 
Buchstaben in der Ausgabe?

[hohesC (Gast)]

Marcel B. schrieb:
> Gut, das Porgramm läuft jetzt.
>
> Allerdings bekomme ich die Zahlen nicht vernünftig ausgegeben, bei jedem
> Überlauf der Sekunden zählt er in Buchstaben weiter.
>
> Wie löst man dieses Problem am besten ?

d, i   int as a signed decimal number. %d and %i are synonymous for 
output, but are different when used with scanf() for input (where using 
%i will interpret a number as hexadecimal if it's preceded by 0x, and 
octal if it's preceded by 0.)

[Marc V. (Firma: Vescomp) (logarithmus)]

Marcel B. schrieb:
> Wie löst man dieses Problem am besten ?

 if (sec == 0x3B) {_delay_ms(1000); min ++; sec = 0x00;}

//*** Wie soll das unten jemals wahr werden ?
 if (min == 0x3B && sec == 0x3B) {_delay_ms(1000);  RTC_GetDateTime();}

[Marcel B. (gigi)]

Nicht wirklich, die Werte werden ja trotz Hex-Ausgabe (%2x) vernünftig 
als Zahl ausgegeben, nach dem lesen der RTC.

Und das Obwohl sie im BCD Format sind.
Da ist gerade mein Verständnisproblem.

Gebe ich die zahlen als INT aus, kommen von anfang an nur falsche Daten 
heraus.

Nach dem auslesen stimmt alles, aber wie zähle ich das manuell hoch ?

[guest (Gast)]

Marcel B. schrieb:
> Falls jemand weiss, wie ich die 0 auch als 0 anzeigen kann, würde ich

Was meinst Du damit? Willst Du die Zahlen zweistellig mit gegebenenfalls 
führender Null ausgeben? Falls ja, dann schreib das doch einfach mit in 
Deine Formatanweisungen rein. Wie steht in der entsprechenden 
Dokumentation, z.B. der hier:
http://www.cplusplus.com/reference/cstdio/printf/

[Marcel B. (gigi)]

Marc V. schrieb:
> Marcel B. schrieb:
>> Wie löst man dieses Problem am besten ?
>
>

>  if (sec == 0x3B) {_delay_ms(1000); min ++; sec = 0x00;}
> 
> //*** Wie soll das unten jemals wahr werden ?
>  if (min == 0x3B && sec == 0x3B) {_delay_ms(1000);  RTC_GetDateTime();}
>

0x3B = 59. Die Zahlen sind eh Binär gespeichert, also ist die Art, wie 
ich vergleiche egal.

[Marc V. (Firma: Vescomp) (logarithmus)]

Marcel B. schrieb:
> 0x3B = 59. Die Zahlen sind eh Binär gespeichert, also ist die Art, wie
> ich vergleiche egal.

 Uninteressant.
 In der Zeile davor hast du sec auf 0 gestellt, falls sec == 0x3B.
 Wie kann sec dann in der Zeile darunter wieder 0x3B sein ?

[guest (Gast)]

Marcel B. schrieb:
> 0x3B = 59.

Und warum schreibst Du dann nicht einfach gleich 59 hin?
Allerdings hast Du gerade selbst geschrieben, daß die Zahlen im 
BCD-Format sind! Und damit sind Deine Vergleiche dann sowieso ziemlich 
daneben.

Außerdem ging es Marc nicht um die Art des Vergleichs, sondern um die 
Sinnlosigkeit des zweiten 'if', da dieses niemals wahr wird.

Btw. Daß Dein obiges Programm überhaupt irgenwie zählen soll kann ich 
auch nicht so recht glauben. Dein 'clock_continue' wird ja nirgends 
aufgerufen.

[Marcel B. (gigi)]

Das habe ich bereits aufgerufen,
habe es zum Test des ersten Problems im Startbeitrag rausgenommen.

Was hier falsch ist, sehe ich auch, aber eine Lösung habe ich nicht, 
darum würde ich mir sehr über Tipps diesbezüglich freuen.

guest schrieb:
> Außerdem ging es Marc nicht um die Art des Vergleichs, sondern um die
> Sinnlosigkeit des zweiten 'if', da dieses niemals wahr wird.

Danke, ein Problem weniger.

Wie inkrementiere ich denn BCD manuell ?

[Frank _. (fbi)]

Marcel B. schrieb:
> Wie inkrementiere ich denn BCD manuell ?

Addieren und auf Überlauf prüfen, halt genau wie mit Deinen Sekunden und 
Minuten, nur für die einzelnen Dezimalstellen.

Deutlich einfacher wäre es allerdings die Werte in 'RTC_GetDateTime' 
gleich von BCD ins normale Format zu wandeln. Das würde Dir die ganzen 
Klimmzüge mit dem BCD ersparen (in 'RTC_SetDateTime' dann natürlich 
wieder zurück nach BCD).

Z.B. so:

uint8_t tmp = i2c_read(1);
sec = ( ( tmp >> 4 ) * 10 ) + ( tmp & 0x0f );

bzw.

uint8_t tmp = ( ( sec / 10 ) << 4 ) | ( sec % 10 );
i2c_write(tmp);

[guest (Gast)]

Marcel B. schrieb:
> Das habe ich bereits aufgerufen,
> habe es zum Test des ersten Problems im Startbeitrag rausgenommen.

Wo denn normalerweise? Innerhalb von 'while(1)' in 'main'? Dann hast Du 
durch die unterschiedlichen Laufzeiten von 'clock_continue' ein 
Anzeigeproblem. Du wirst z.B. nie einen Sekundenwert von 59 zu Gesicht 
bekommen, Deine Anzeige springt von 58 auf 0. Besser mit 60 vergleichen 
und die zusätzlichen '_delay_ms' wegwerfen. Also so in etwa:

void clock_continue()
{
  _delay_ms(1000);
  sec++;                
  if(sec == 60)
  {
    sec = 0;
    min++;
  }
  if(min == 60)
  {
    min = 0;
    RTC_GetDateTime();
  }
}

[Marcel B. (gigi)]

Danke Euch beiden,
das werde ich direkt mal ausprobieren. :)