Работа с DS1307

[Alex]

Потому, что нельзя передать в функцию сразу 100500 переменных. Можно конечно, но кровь из глаз потечёт от вида этой функции.

По этому и применяют указатель. Он - адрес блока данных. А по нему (указателю) уже забирают данные.

[Реклама]

Реклама: ООО ТД Промэлектроника, ИНН: 6659197470, Тел: 8 (800) 1000-321

[ARV]

Да отделаться-то можно... но в конечном итоге это может быть менее удобно.

Указатель обладает способностью "указывать" на данные любой длины, а любая переменная всегда имеет фиксированный размер. Поэтому функцию "под любой" размер с указателями всегда можно приспособить и для работы с int, например, а вот функцию для работы с int приспособить для работы с char будет сложновато...

[Реклама]

20% скидка на весь каталог электронных компонентов в ТМ Электроникс!

Акция "Лето ближе - цены ниже", успей сделать выгодные покупки!

Плюс весь апрель действует скидка 10% по промокоду APREL24 + 15% кэшбэк и бесплатная доставка!

Перейти на страницу акции

Реклама: ООО ТМ ЭЛЕКТРОНИКС, ИНН: 7806548420, info@tmelectronics.ru, +7(812)4094849

[-=FISHER=-]

2 минуты назад, Alex сказал:

Потому, что нельзя передать в функцию сразу 100500 переменных

void DS_Write(uint8_t addrr, void* pData, uint8_t size)

А вообще мы передаем в функцию addrr - адрес в памяти DS1307, void* pData - указатель на данные которые хотим передать, uint8_t size - а вот это размер чего? Этих самых данных? А как его определить?

[Реклама]

Выбираем схему BMS для корректной работы литий-железофосфатных (LiFePO4) аккумуляторов

 Обязательным условием долгой и стабильной работы Li-FePO4-аккумуляторов, в том числе и производства EVE Energy, является применение специализированных BMS-микросхем. Литий-железофосфатные АКБ отличаются такими характеристиками, как высокая многократность циклов заряда-разряда, безопасность, возможность быстрой зарядки, устойчивость к буферному режиму работы и приемлемая стоимость. Но для этих АКБ, также как и для других, очень важен контроль процесса заряда и разряда, а специализированных микросхем для этого вида аккумуляторов не так много. Инженеры КОМПЭЛ подготовили список имеющихся микросхем и возможных решений от разных производителей. Подробнее>>

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[ARV]

Только что, -=FISHER=- сказал:

А как его определить?

В общем случае при помощи sizeof, но чаще всего вы и так знаете, сколько данных собираетесь передавать.

[Alex]

1 минуту назад, -=FISHER=- сказал:

А как его определить?

Будет зависить от того, что передаёте.
Ну а вообще - sizeof()
В случае со строкой - strlen()

[-=FISHER=-]

1 минуту назад, ARV сказал:

но чаще всего вы и так знаете, сколько данных собираетесь передавать.

Ну например я хочу передать значение часов. Следовательно, вызывать функцию я буду вот с такими параметрами?

Вместо этого:

Скрытый текст

 	I2C_StartCondition(); //начало передачи
 	I2C_SendByte(0b11010000); //режим записи
 	I2C_SendByte(0x02);//Переходим на 0x02
	I2C_SendByte(RTC_ConvertFromBinDec(12)); //устанавливаем часы
 	I2C_StopCondition();

 

Будет вызов функции, предложенной @Alex , со следующими входящими параметрами

Скрытый текст

DS_Write(0x02, RTC_ConvertFromBinDec(12), ???)

 

В качестве размера мне нужно указать что? То что число 12 занимает 1 байт, то есть записать 256?... Простите если туплю

[ARV]

Только что, -=FISHER=- сказал:

1 байт, то есть записать 256?

Зачем же писать 256, если занимает 1 байт? Вы ж байтами передаете, значит 1 и писать.

[Alex]

7 минут назад, -=FISHER=- сказал:

Ну например я хочу передать значение часов.

Не нужно передавать каждое значение по-отдельности, теряется смысл создания этой функции. Её назначение - принять блок данных (массив, структуру, ...).

Если Вы со структурами не очень дружите, то для начала можете создать обычный массив, в котором будут хранится часы, минуты, секунды. И работать с ним.

PS: Кстати, эта функция, скорее всего, пойдёт и для записи данных в RAM. Просто укажите начальный адрес расположения памяти. На сколько я помню, в часах вся память линейна. Начало - часы с календарём, потом идёт RAM.

[-=FISHER=-]

6 минут назад, Alex сказал:

Не нужно передавать каждое значение по-отдельности, теряется смысл создания этой функции. Её назначение - принять блок данных (массив, структуру, ...).

 

8 минут назад, ARV сказал:

Вы ж байтами передаете, значит 1 и писать.

Теперь дошло!)) Спасибо большое за развернутый ответ.

[ARV]

Вам @Alex советовал, и я присоединяюсь, сделать структуру для времени, и работать уже с нею. DS-ке все равно, 1 байт вы в неё зафигачете, или сразу несколько, а для получения времени и установки со структурой будет удобнее:

Вот в соответствии с распределением памяти DS-ки вам разумно свои данные о времени хранить в структуре, полностью повторяющей первые 3 строки этой таблицы:

typedef struct{
  uint8_t	sec : 4;	// единицы секунд
  uint8_t	ten_sec : 3;	// десятки секунд
  uint8_t	ch : 1;		// ?? не помню
  uint8_t	min : 4;	// единицы минут
  uint8_t	ten_min : 3;	// десятки минут
  uint8_t	reserved : 1;	// не используется
  uint8_t	hour : 4;	// единицы часов
  uint8_t	ten_hour : 2;	// дестяки часов (см. am_pm)
  uint8_t	am_pm : 1;	// 12-часовая или 24-часовая система (влияет на интерпретацию десятков часов)
  uint8_t	reserved2 : 1;	// не используется
} ds_time_t;

Вы у себя в программе храните время в том же самом формате, как и DS-ка, например, в переменной ds_time_t my_time; 

И тогда для установки времени вам надо записать в DS-ку всю эту структуру целиком DS_Write(0, &my_time, sizeof(my_time));

А для того, чтобы узнать время, надо целиком эту структуру прочитать DS_read(0, &my_time, sizeof(my_time));

Изменено 17 августа, 2018 пользователем ARV

[Alex]

Для примера, функция чтения :

void DS_Read(uint8_t addrr, void* pData, uint8_t size){
	uint8_t * p= (uint8_t*)pData;
	I2C_StartCondition(); //начало передачи
	I2C_SendByte(0b11000000); 						// Режим чтения. Не уверен, что 0b11000000. В даташит лень лезть.
	I2C_SendByte(addrr);//Переходим на addrr
	while(size--)	*p++ = RTC_ConvertFromDecBin(I2C_ReciveByte());		// С RTC_ConvertFromDecBin не уверен. Не знаю, как у вас данные обратно выворачиваются
	I2C_StopCondition();
}

Могут быть ошибки. Проверьте всё (бит чтения, ф-ия чтения с I2C, ф-ия обратного конвертирования данных).

6 минут назад, ARV сказал:

uint8_t sec : 4; // единицы секунд

uint8_t ten_sec : 3; // десятки секунд

У него, скорее всего, функция RTC_ConvertFromBinDec сама преобразовывает из байта в тетрады. Достаточно завести в структуре байты с полными значениями секунд/минут/часов.
Хотя, не уверен. Не видя кода, тяжело сказать...

[ARV]

Только что, Alex сказал:

У него, скорее всего, функция RTC_ConvertFromBinDec сама преобразовывает из байта в тетрады. Достаточно завести в структуре байты с полными значениями секунд/минут/часов.

Для работы - достаточно, но для блочной записи-чтения уже не применимо, т.к. форматы байтов и их количество сильно разнятся.

[Alex]

Блин, Вы меня смутили Полез в старые архивы смотреть... 

[ARV]

Только что, Alex сказал:

Вы меня смутили

Да чем же?! Имхо, чем меньше преобразовний форматов данных, тем лучше.

[Alex]

Выдрал из какого-то проекта :

#ifndef _DS1307_H
#define _DS1307_H
/**************************************************************/
#include "i2c.h"
/**************************************************************/
typedef struct{
    unsigned char   sec;
    unsigned char   min;
    unsigned char   hours;
    unsigned char   day;
    unsigned char   date;
    unsigned char   month;
    unsigned char   year;
    unsigned char   control;
}t_ds1307;

/**************************************************************/
char DS1307_Init(void);
void DS1307_Read(t_ds1307* p_struct);
void DS1307_Write(t_ds1307* p_struct);

/**************************************************************/
#endif  // _DS1307_H

#include "ds1307.h"

static t_ds1307     ds1307;

...........................
...........................
...........................
  
  
/****************************************************************/
static void DS1307_Read_Struct(){
unsigned char i, *p=(unsigned char*)&ds1307;
I2C_Start();
I2C_Write(0xD0); I2C_Wait_Idle();
I2C_Write(0x00); I2C_Wait_Idle();
I2C_Stop();
I2C_Start();
I2C_Write(0xD1); I2C_Wait_Idle();
for(i=0;i<8;i++){
    I2C_Write_ASK(1);
    *p++=I2C_Read();
    I2C_Wait_Idle();
}
I2C_Stop();
}
/****************************************************************/
static void DS1307_Write_Struct(){
unsigned char i, *p=(unsigned char*)&ds1307;
I2C_Start();
I2C_Write(0xD0); I2C_Wait_Idle();
I2C_Write(0x00); I2C_Wait_Idle();
for(i=0;i<8;i++){
    I2C_Write(*p++); I2C_Wait_Idle();
}
I2C_Stop();
}
/****************************************************************/

/****************************************************************/
static unsigned char Bin10_Bin(unsigned char val){
return (val&0x0F)+(unsigned char)(val>>4)*10;
}
/****************************************************************/
static unsigned char Bin_Bin10(unsigned char val){
return ((val/10)<<4)|(val%10);
}
/****************************************************************/
void DS1307_Read(t_ds1307* p_struct){
DS1307_Read_Struct();
p_struct->sec=Bin10_Bin(ds1307.sec);
p_struct->min=Bin10_Bin(ds1307.min);
p_struct->hours=Bin10_Bin(ds1307.hours);
p_struct->day=(ds1307.day&0x0F);
p_struct->date=Bin10_Bin(ds1307.date);
p_struct->month=Bin10_Bin(ds1307.month);
p_struct->year=Bin10_Bin(ds1307.year);
}
/****************************************************************/
void DS1307_Write(t_ds1307* p_struct){
ds1307.sec=Bin_Bin10(p_struct->sec);
ds1307.min=Bin_Bin10(p_struct->min);
ds1307.hours=Bin_Bin10(p_struct->hours);
ds1307.day=p_struct->day;
ds1307.date=Bin_Bin10(p_struct->date);
ds1307.month=Bin_Bin10(p_struct->month);
ds1307.year=Bin_Bin10(p_struct->year);

DS1307_Write_Struct();
}
/****************************************************************/

4 минуты назад, ARV сказал:

Да чем же?!

Тем, что я помню - в структуре были байты

5 минут назад, ARV сказал:

Имхо, чем меньше преобразовний форматов данных, тем лучше.

Согласен. Но в случае с битовыми полями, тяжело будет работать со структурой в основной программе.

[ARV]

Только что, Alex сказал:

Но в случае с битовыми полями, тяжело будет работать со структурой в основной программе

Ничуть не сложнее, чем в выбранном вами подходе  Конечно, придется значение секунд, например, брать из двух полей, но прозрачность этого действия лучше, чем Bin_Bin10, имхо.

Для индикации секунд и минут даже проще - не надо никаких преобразований, все уже преобразовано. Для редактирования через меню при настройке - тоже (если вводить поразрядно, конечно).

В общем, дело вкуса, как обычно

[Alex]

5 минут назад, ARV сказал:

В общем, дело вкуса, как обычно

Согласен...

[-=FISHER=-]

28 минут назад, Alex сказал:

У него, скорее всего, функция RTC_ConvertFromBinDec сама преобразовывает из байта в тетрады.

Вот эти функции:

Скрытый текст

#include "RTC.h"

unsigned char RTC_ConvertFromDec(unsigned char c)
{
	unsigned char ch = ((c>>4)*10+(0b00001111&c));
	return ch;
}

unsigned char RTC_ConvertFromBinDec(unsigned char c)
{
	unsigned char ch = ((c/10)<<4)|(c%10);
	return ch;
}

 

 

[Alex]

8 минут назад, -=FISHER=- сказал:

Вот эти функции:

Ну да, это оно. Двоично-десятичное конвертирование.
Один в один по функционалу как у меня.

[ARV]

Только что, Alex сказал:

Один в один по функционалу как у меня

И практически один в один то, что делает компилятор сам, при обращении к битовым полям  

[Alex]

Ну да. Только без умножения на 10