Atmega 328p DS1307

[Philipp P. (Gast)]

Hallo,

kann sich einer von euch mal den Code angucken und mir nen Tip geben was 
ich falsch mache? Möchte den DS1307 auslesen. Der MC soll mir die Daten 
seriell ausgeben und auf dem Display. Ich benutze die I2C lib von Peter 
Fleury und bekomms nicht hinn. Das Display zeigt 53:53:53 an :p

#include "uart.h"
#include "i2cmaster.h"
#include <util/delay.h>
#include "lcd.h"
#include "rtc2.h"
#define BAUDRATE 9600UL 



#define DS1307     0xD0         // Device Adresse des DS2307


unsigned char time_s;        
unsigned char time_m;        
unsigned char time_h;         
unsigned char time_dayofweek;   
unsigned char time_day;        
unsigned char time_month;      
unsigned char time_year;      
unsigned char buffer1[10];      // Buffer für itoa RTC 


void uart_init (void)
{
    uint16_t ubrr = (uint16_t) ((uint32_t) F_CPU/(16*BAUDRATE) - 1);
 
    UBRR0H = (uint8_t) (ubrr>>8);
    UBRR0L = (uint8_t) (ubrr);
 
    // UART Receiver und Transmitter anschalten 
    // Data mode 8N1, asynchron 
    UCSR0B = (1 << RXEN0) | (1 << TXEN0);
    //UCSR0C = (1 << URSEL) | (1 << UCSZ01) | (1 << UCSZ00);

    // Flush Receive-Buffer (entfernen evtl. vorhandener ungültiger Werte) 
    do
    {
        UDR0;
    }
    while (UCSR0A & (1 << RXC0));
}

char set_clock (unsigned char reg, unsigned char val)
{
   if (i2c_start(DS1307+I2C_WRITE) != 0) return 1;
   i2c_write(reg);
   i2c_write(val);
   i2c_stop();
   return 0;
} 

char get_time(unsigned char reg)
{
unsigned char val;

   i2c_start(DS1307+I2C_WRITE);
   i2c_write(reg);
   i2c_stop();
   i2c_start(DS1307+I2C_READ);
   val = i2c_readNak();
   i2c_stop(); 
}



void uart_puts (const char *s)
{
    do
    {
        uart_putc (*s);
    }
    while (*s++);
}



int main()
{
  lcd_init(LCD_DISP_ON); 

               
    uart_init();

    i2c_init();

      _delay_ms(300);
    
  ds1307_init;

    _delay_ms(300);
     lcd_clrscr();

    lcd_gotoxy(0,0);

     lcd_puts("Bereit");
       _delay_ms(300);
  
  set_clock(0x07, 0x00);
     set_clock(0x02, 0x00);
     set_clock(0x01, 0x00);
     set_clock(0x00, 0x00); 


  while (1){


    get_time(0x00);
      time_s = get_time;
      uart_puts(time_s);

        get_time(0x01);
      time_m = get_time;
      uart_puts(time_s);

        get_time(0x02);
      time_h = get_time;
      uart_puts(time_s);


      lcd_clrscr();
  
  sprintf(buffer1, "Zeit: %02x:%02x:%02x", time_h, time_m, time_s);
      lcd_puts(buffer1); 
             
    

    }
}

[Matthias J. (mjfruit)]

In get_time fehlt ein return des Wertes:

> char get_time(unsigned char reg)
> {
> unsigned char val;
>
>    i2c_start(DS1307+I2C_WRITE);
>    i2c_write(reg);
>    i2c_stop();
>    i2c_start(DS1307+I2C_READ);
>    val = i2c_readNak();
>    i2c_stop();
return val;
> }

Und in der main-Schleife:

>       get_time(0x00);
>       time_s = get_time;
>       uart_puts(time_s);
>
>         get_time(0x01);
>       time_m = get_time;
>       uart_puts(time_s);
>
>         get_time(0x02);
>       time_h = get_time;
>       uart_puts(time_s);

müsste normal dein Compiler meckern, weil du mit time_s = get_time; 
versuchst, time_s eine nicht existente Variable zu übergeben.
Probier mal, die Zeilen durch

 time_s = get_time(0x00);
 uart_puts(time_s);

Und so weiter zu ersetzen.

Matthias

[Philipp P. (Gast)]

Thx für die Antworten.
Ja Matthias meine Dummheit tut weh ich habs grad entdeckt mit dem return 
:) Nach Tagen funktionierts endlich. Vielen Dank und schönen Abend noch.

Gruß Philipp P.

[Philipp P. (Gast)]

Was müsste ich schreiben um am PC terminal vernüftige Formate über 
Seriell angezeigt zu bekommen?

[Matthias J. (mjfruit)]

Ich denke, das sollte genauso funktionieren wie beim Display. Also 
ebenfalls über sprintf.