Запись в EEPROM, пишу структуру, а попадает содержимое flash'a Опции

[GoodNews]

Подскажите пожалуйста, что я делаю неправильно? Имеется структура, определенная как массив из 51 элемента для хранения в eeprom. Пытаюсь записать данные в один из элементов массива, однако в итоге получаю кусок flash'a в eeprom. Почему?

CODE

typedef struct _chInMem {
uint32_t returnValue;
} chInMem;
chInMem channelMem[] EEMEM = {
{0}, {0}, {0}, {0}, {0},
{0}, {0}, {0}, {0}, {0},
{0}, {0}, {0}, {0}, {0},
{0}, {0}, {0}, {0}, {0},
{0}, {0}, {0}, {0}, {0},
{0}, {0}, {0}, {0}, {0},
{0}, {0}, {0}, {0}, {0},
{0}, {0}, {0}, {0}, {0},
{0}, {0}, {0}, {0}, {0},
{0}, {0}, {0}, {0}, {0},
{107900000}
};
...
chInMem channelInMem;
eeprom_busy_wait();
channelInMem.returnValue = channel_freq;
eeprom_write_block(&channelInMem, &channelMem, sizeof(channelMem));
...

[GoodNews]

Я посмотрел вывод. Там как ни странно всё нормально. Т.е. в .eep вначале идёт большая структура, а потом моя переменная. То, что попало в eeprom таким образом очень странно. Тем не менее - во вложении всё необходимое. Компилятор - AVR-GCC.

Прикрепленные файлы

 mem.rar ( 19.77 килобайт ) Кол-во скачиваний: 13

 

[Сергей Борщ]

Полагаю, ошибка здесь:

Код

                    NEXT_FREE_CHANNEL = channel_number;
                    eeprom_write_block(&NEXT_FREE_CHANNEL, &channel_number, sizeof(uint8_t));

В первой строке вы пишете channel number в ОЗУ по тому же адресу, по которому в eeprom расположен NEXT_FREE_CHANNEL. Куда он попадает и что при этом рушит - не берусь судить. Больше криминала на первый взгляд не видно.
Несколько замечайнний:

Код

static uint8_t NEXT_FREE_CHANNEL EEMEM = 1; //Тут будет храниться номер следующего свободного канала

Принято заглавными буквами обозначать макроопределения и константные литералы. У вас же NEXT_FREE_CHANNEL - переменная.

Код

const uint8_t USER_CHANNELS_AVAILABLE = 50; //Всего доступно ячеек

Эта константа в общем случае будет занимать память - компилятор предполагает, что к этой константе могут быть обращения из других единиц компиляции. Поскольку она не нужна вам в других единицах компиляции - объявите ее как static const uint8_t. Это позволит компилятору подставить в код сразу ее значение. Ваша запись была бы корректной в C++. Поведение C++ в этом моменте отличается, там константы по умолчанию имеют локальное связывание. В C++ для получения доступа к константе из других файлов ее надо объявлять extern const uint8_t USER_CHANNELS_AVAILABLE = 50. Но у вас файлы имеют расширение .c, а значит, компилируются в режиме C. Добавьте static, код может стать оптимальнее.

Код

eeprom_busy_wait();

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

Код

                   eeprom_write_block(&channelInMem, &channelMem[channel_number - 1], sizeof(chInMem));
                    write_channel_eeprom(channel_number, 0, 1);
                    break;
            case 1: //Пишем номер следующего доступного канала

Здесь можно убрать

Код

                    write_channel_eeprom(channel_number, 0, 1);
                    break;

Выполнится то же самое, но без рекурсии.

Для совсем красивости можно массив ваших каналов и номер свободного канала объединить в структуру.