Как получить адрес элемента структуры?

[k0stya_]

Пишу программу на Си (WinAVR) для ATmega8 и вот сталкнулся с такой проблемой.

Есть структура

struct dataexchange         // Описание структуры для передачи данных
{
  unsigned char cmd;         // Код команды, которую нужно выполнить.
  unsigned char adr1;        // Адрес, используется в некоторых командах (старший байт).
  unsigned char adr2;        // Адрес, используется в некоторых командах (младший байт).
  unsigned char dat1;        // Данные, относящиеся к команде (первый байт).
  unsigned char dat2;        // Данные, относящиеся к команде (второй байт).
};
 
struct dataexchange pdata = {0, 0, 0, 0, 0};

В процессе работы нужно записывать/считывать данные в/из EEPROM. В библиотеке WinAVR указаны сигнатуры

/** \ingroup avr_eeprom
    Read one byte from EEPROM address \a __p.
 */
uint8_t eeprom_read_byte (const uint8_t *__p) __ATTR_PURE__;

/** \ingroup avr_eeprom
    Write a byte \a __value to EEPROM address \a __p.
 */
void eeprom_write_byte (uint8_t *__p, uint8_t __value);

Я делаю так

eeprom_write_byte (&pdata.adr1, pdata.dat1); // Записываем

pdata.dat1 = eeprom_read_byte (&pdata.adr1); // Считываем

и при этом данные записываются и счиваются только в одной ячейке по адресу 0x0064, т.е. адрес вычесляется неверно.
Искал в google, заглянул в пару книг по Си но везде указано только как получить адресс самой структуры и дальше с помощью оператора -› обращаться к элементам, но мне это не подходит, т.к. нужно передать адрес самого элемента.

Прошу помочь.

[kison]

&pdata.adr1 - правильный способ. Дает указатель на элемент структуры.
Вам надо с записью разобраться. Ответить себе на пару вопросов - например что такое adr1. Скорее всего ответ будет - адрес куда писать в еепром. По сути - указатель. Так наверно и надо передавать не указатель на указатель, а само значение.
eeprom_write_byte ((void*)pdata.adr1, pdata.dat1); // Записываем
Возможно я конечно не понял цель. Что хотим получить то?

[Yurkin2007]

данные записываются и счиваются только в одной ячейке по адресу 0x0064,

Ну, всё правильно. Вот, смотрите. Вы объявили переменную pdata, которая есть структура. При компилировании программы компилятор разместил эту переменную в памяти RAM процессора так, что элемент cmd этой переменной лежит по адресу 0x0063, элемент adr1 по адресу 0x0064, элемент adr2 по адресу 0x0065 и т.д. Причем эти адреса - константы и не изменяются при работе программы, то есть переменную pdata как положили в память, так она там и лежит, не двигаясь ...
Поэтому запись &pdata.adr1 = 0х0064 всегда.
Если же Вы хотите писать куда-то по адресу, который лежит в ячейках adr1,adr2, то нужно использовать значения этих элементов, а не их адреса. Как я понял, adr1 - старший байт нужного адреса записи, а adr2 - младший. Тогда полный адрес будет = (adr1 ‹‹ 8 ) + adr2. И ещё нужно сказать компилятору, что по этому адресу будет записан беззнаковый байт uint8_t.
То есть, получается так:

;
eeprom_write_byte ((uint8_t *)((pdata.adr1‹‹8)+pdata.adr2), pdata.dat1);
;

[nonamedov]

Думаю, проблема в приоритете операций, приоритет операции взятие ссылки & ниже чем операции обращения к полю ., поэтому надо писать так: (&pdata).adr1.

[k0stya_]

Фух, разобрался.
Спасибо Yurkin2007 и kison за помощь.

Все заработало в таком вот виде:

pdata.dat1 = eeprom_read_byte ((uint8_t *)pdata.adr1); // Считываем

eeprom_write_byte ((uint8_t *)pdata.adr1, pdata.dat1); // Записываем

Выходит операция взятия адреса была лишней и я все время получал указатель на указатель.

[_Артём_]

Фух, разобрался.
Спасибо Yurkin2007 и kison за помощь.

Все заработало в таком вот виде:

Код:

pdata.dat1 = eeprom_read_byte ((uint8_t *)pdata.adr1); // Считываем eeprom_write_byte ((uint8_t *)pdata.adr1, pdata.dat1); // Записываем

Выходит операция взятия адреса была лишней и я все время получал указатель на указатель.

››Выходит операция взятия адреса была лишней
Нет, не лишняя.
Не буду утверждать на все 100% (на 99 можно):

pdata.dat1 = eeprom_read_byte ((uint8_t *)pdata.adr1); // Считываем

eeprom_write_byte ((uint8_t *)pdata.adr1, pdata.dat1); // Записываем

этот код по-моему чушь полная...

pdata.dat1 = eeprom_read_byte ((uint8_t *)pdata.adr1); // Считываем

Это как я понимаю, чтение байта pdata.dat1 по адресу равному &pdata.dat1 (такое может быть если в eeprom лежит копия ОЗУ и адреса совпадают). Это еще ладно, но вот это:

pdata.dat1 = eeprom_read_byte ((uint8_t *)pdata.adr1); // Считываем

вообще нечто. Функции eeprom_read_byte передаётся как адрес значение байта pdata.adr1 и pdata.adr1 им теперь уже инициализируется(генерато� � случайных чисел может быть?). Мне неизвестен диапазон значений pdata.adr1, но если предположить что 0-255, то pdata.adr1 инициализируется байтами из eeprom с адресами 0-255.

Не знаю точно как для WinAVR. поэтому приведу код для IAR:

struct dataexchange         // Описание структуры для передачи данных
{
  unsigned char cmd;         // Код команды, которую нужно выполнить.
  unsigned char adr1;        // Адрес, используется в некоторых командах (старший байт).
  unsigned char adr2;        // Адрес, используется в некоторых командах (младший байт).
  unsigned char dat1;        // Данные, относящиеся к команде (первый байт).
  unsigned char dat2;        // Данные, относящиеся к команде (второй байт).
};
__eeprom struct dataexchange pdata_EEPROMCopy;// копия в еепром
struct dataexchange pdata;// структура в ОЗУ

unsigned char ReadByteFromEEPROM(unsigned short addr);
void WriteByteToEEPROM(unsigned short addr, unsigned char new_byte);

// запись
WriteByteToEEPROM((unsigned short)&pdata_EEPROMCopy.adr1, pdata.adr1);
// чтение 
pdata.adr1=ReadByteFromEEPROM((unsigned short)&pdata_EEPROMCopy.adr1);

Для WinARV __eeprom нужно заменить соотв. атрибутом (__attribute__ ((section (".eeprom"))) вроде).

[kison]

Не буду утверждать на все 100% (на 99 можно):

Оказывается 1% это много...

Код:

pdata.dat1 = eeprom_read_byte ((uint8_t *)pdata.adr1); // Считываем

Это как я понимаю, чтение байта pdata.dat1 по адресу равному &pdata.dat1 (такое может быть если в eeprom лежит копия ОЗУ и адреса совпадают).

Не путаем адреса с данными. Это чтение по адресу adr1 в переменную dat1. dat1 и adr1 это не одна и та же переменная.

Это еще ладно, но вот это:

Код:

pdata.dat1 = eeprom_read_byte ((uint8_t *)pdata.adr1); // Считываем

вообще нечто. Функции eeprom_read_byte передаётся как адрес значение байта pdata.adr1 и pdata.adr1 им теперь уже инициализируется(генерато� � случайных чисел может быть?).

И опять.

Мне неизвестен диапазон значений pdata.adr1, но если предположить что 0-255, то pdata.adr1 инициализируется байтами из eeprom с адресами 0-255.

Это секретный тип - uint8_t. И его диапазон тоже секретен.

Не знаю точно как для WinAVR. поэтому приведу код для IAR:

А теперь представьте, что еепром содержит один сплошной массив. Имеется структура в ОЗУ, которой Вы зачем то копию в еепром создали. В этой структуре есть поле с адресом - фактически это индекс массива в еепром. И поле с данными которые нужно писать/читать в тот самый массив в еепром. И все встанет на свои места. Правда почему то окажется что 1% куда больше 99...

[nonamedov]

Смысла в копировании в ЕЕПРОМ нет, как Вы, вообще, сможете воспользоваться адресом указанным в структуре в ЕЕПРОМ?! Вам нужна структура в ОЗУ, которая хранила (упорядочивала) бы данные в ЕЕПРОМ.

__eeprom struct dataexchange pdata_EEPROMCopy;// копия в еепром
struct dataexchange pdata;// структура в ОЗУ

[k0stya_]

Все таки я плохо искал в гугле...
Вот, может кому-то пригодиться

Tip on storing initial values in EEPROM

[CodeNameHawk]

Для структуры по идее должно подойти
void eeprom_write_block (const void *__src, void *__dst, size_t __n)
void eeprom_read_block (void *__dst, const void *__src, size_t __n)