Переброс массива из EEPROM в RAM используя указатели Опции

[SZ0]

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

Есть массив в EEPROM, при запуске проца надо перебросить его в RAM. Написал такой код:

Код

unsigned short *ptr_win, *ptr_win_ee, *ptr_eeprom, g = 0;

ptr_win = (unsigned short *)&win[0];
ptr_win_ee = (unsigned short *)&win_ee[0];

unsigned short size_w = sizeof(win), i = 0; //size_w размер массива в eeprom

while(i < size_w)
{
  EEAR = g++;
  EECR |= (1<<EERE);
  *ptr_win++ = EEDR;
  i++;
}

Не получилось вместо g использовать указатель ptr_win_ee, который указывает на массив в eeprom и присваивать его значение EEAR. И при увеличении указателя *ptr_win++ он увеличивается на два байта, вместо 1. А как его заставить увеличиваться на байт, не могу сообразить
Выручайте, а то голова ваще гудит

В таком виде из EEPROM читаются все байты из массива, но пишутся они в RAM через 1 байт. Использование g не правильно, т.к. g инициализировано на начало eeprom. Потому что я заранее знаю где начало массива. А правильно через указание EEAR = *ptr_win_ee++ не получается.

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

Начинать надо с того, что из EEPROM считывание идет побайтно. Значит ваши указатели должны быть указателями на байт. К слову, sizeof() также возвращает размер объекта в байтах. Поскольку исходный массив у вас в словах, надо воспользоваться явным приведением указателей:

Код

unsigned char *ptr_win, *ptr_win_ee, i;

ptr_win = (unsigned char *)win;    // указатель на массив тождественен указателю на его нулевой элемент,
                            // поэтому более короткая запись вместо &win[0]
ptr_win_ee = (unsigned char *)win_ee;

do
{
  EEAR = (unsigned int)(ptr_win_ee++);
  EECR |= (1<<EERE);
  *ptr_win++ = EEDR;
}
while(++i < sizeof(win));  // массив явно ненулевого размера, поэтому используем более эффективный цикл do {} while()

Если компилятор IAR, то все проще:

Код

unsigned char *ptr_win;
__eeprom unsigned char *ptr_win_ee;
unsigned char i = 0;

ptr_win = (unsigned char *)win;
ptr_win_ee = (__eeprom unsigned char *)win_ee;

do
{
  *ptr_win++ = *ptr_win_ee++;
}
while(++i < sizeof(win));

Т.к. в указателях адрес. А то что один адрес на еепром указывает, IAR не телепат

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

Применение while(i < (size_w/2)) в таком виде ведёт к лишним затратам, т.к. size_w/2 постоянно обрабатывается в цикле.

Так выкиньте лишнюю переменную и используйте в цикле sizeof(win)/2, которое вычисляется на этапе компиляции. Хотя, если включена оптимизация, компилятор должен вынести деление на 2 из цикла даже если size_w передается как параметр в функцию.

А еще эффективнее перейти к циклу do {} while (--i):

Код

ptr_win = (unsigned char *)win;
ptr_win_ee = (__eeprom unsigned char *)win_ee;
unsigned char i = sizeof(win);

do
{
  *ptr_win++ = *ptr_win_ee++;
}
while(--i);

Еще одна уловка: если будете использовать цикл с поэлементным (не побайтным) копированием, то лучше вместо sizeof(win)/2 сразу писать sizeof(win)/sizeof(win[0]) - это позволит не менять исходник этого кода если вдруг придется изменить размер элемента массива.

[SZ0]

Начинать надо с того, что из EEPROM считывание идет побайтно...

Спасибо за подробные разъяснения, всё работает. Только здесь __eeprom unsigned char пришлось поменять на unsigned char __eeprom иначе IAR выдаёт ошибку.

Сообщение отредактировал SZ0 - Aug 23 2008, 13:02