Перейти к содержанию

Работа с DS1307


-=FISHER=-

Рекомендуемые сообщения

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

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

Ссылка на комментарий
Поделиться на другие сайты

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

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

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

Если забанить всех, кто набрался смелости думать независимо, здорово будет на форуме - как на кладбище: тишина, птички поют...

Ссылка на комментарий
Поделиться на другие сайты

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

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

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

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

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

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

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

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

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

Если забанить всех, кто набрался смелости думать независимо, здорово будет на форуме - как на кладбище: тишина, птички поют...

Ссылка на комментарий
Поделиться на другие сайты

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

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

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

Ссылка на комментарий
Поделиться на другие сайты

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?... Простите если туплю

Мы все учились по-немногу, чему-нибудь и как-нибудь...

Ссылка на комментарий
Поделиться на другие сайты

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

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

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

Если забанить всех, кто набрался смелости думать независимо, здорово будет на форуме - как на кладбище: тишина, птички поют...

Ссылка на комментарий
Поделиться на другие сайты

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

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

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

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

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

Ссылка на комментарий
Поделиться на другие сайты

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

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

 

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

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

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

Мы все учились по-немногу, чему-нибудь и как-нибудь...

Ссылка на комментарий
Поделиться на другие сайты

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

5b767e166d4de_Screenshot2018-08-1710_49_02.png.db88a97e661f28f1bf9cee59a03db83c.png

Вот в соответствии с распределением памяти 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));

Изменено пользователем ARV

Если забанить всех, кто набрался смелости думать независимо, здорово будет на форуме - как на кладбище: тишина, птички поют...

Ссылка на комментарий
Поделиться на другие сайты

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

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 сама преобразовывает из байта в тетрады. Достаточно завести в структуре байты с полными значениями секунд/минут/часов.
Хотя, не уверен. Не видя кода, тяжело сказать...

Ссылка на комментарий
Поделиться на другие сайты

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

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

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

Если забанить всех, кто набрался смелости думать независимо, здорово будет на форуме - как на кладбище: тишина, птички поют...

Ссылка на комментарий
Поделиться на другие сайты

Только что, 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 сказал:

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

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

Ссылка на комментарий
Поделиться на другие сайты

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

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

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

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

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

Если забанить всех, кто набрался смелости думать независимо, здорово будет на форуме - как на кладбище: тишина, птички поют...

Ссылка на комментарий
Поделиться на другие сайты

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;
}

 

 

Мы все учились по-немногу, чему-нибудь и как-нибудь...

Ссылка на комментарий
Поделиться на другие сайты

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

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

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

Ссылка на комментарий
Поделиться на другие сайты

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

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

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

Если забанить всех, кто набрался смелости думать независимо, здорово будет на форуме - как на кладбище: тишина, птички поют...

Ссылка на комментарий
Поделиться на другие сайты

Присоединяйтесь к обсуждению

Вы можете написать сейчас и зарегистрироваться позже. Если у вас есть аккаунт, авторизуйтесь, чтобы опубликовать от имени своего аккаунта.
Примечание: Ваш пост будет проверен модератором, прежде чем станет видимым.

Гость
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Ответить в этой теме...

×   Вставлено с форматированием.   Восстановить форматирование

  Разрешено использовать не более 75 эмодзи.

×   Ваша ссылка была автоматически встроена.   Отображать как обычную ссылку

×   Ваш предыдущий контент был восстановлен.   Очистить редактор

×   Вы не можете вставлять изображения напрямую. Загружайте или вставляйте изображения по ссылке.

Загрузка...
  • Последние посетители   0 пользователей онлайн

    • Ни одного зарегистрированного пользователя не просматривает данную страницу
×
×
  • Создать...