Помощь - Поиск - Пользователи - Календарь
Полная версия этой страницы: Ссылки в winavr - varning
Форум разработчиков электроники ELECTRONIX.ru > Микроконтроллеры (MCs) > AVR
Igor_U
Есть массив AdrDecodeReciver[] во флеше в котором описаны адреса данных в структурах (озу), все данные 16-ти битные. Задача сформировать массив Usart[] из данных которые описаны в AdrDecodeReciver[].

Вот массив:
Код
const uint16_t *AdrDecodeReciver[NumberReciver+NumberTranciver] PROGMEM=
  {
  &ResultFirst.VoltageOff,
  &ResultFirst.CurrentOff,
  ........
  };


Вот функция которая это делает:

Код
extern void CopyReciverData(void)
{
uint8_t i;
uint16_t *Adr;

for (i=0;i<(NumberTranciver+NumberReciver);i++)
  {
    Adr=pgm_read_word(&AdrDecodeReciver[i]);
    Usart[i]=*Adr;
  }
}


Компилятор выдает варнинг:warning: assignment makes pointer from integer without a cast. Хотя компилит правильный код.
Как написать правильно?
Виктория
Цитата(Igor_U @ Nov 29 2005, 12:25) *
Компилятор выдает варнинг:warning: assignment makes pointer from integer without a cast. Хотя компилит правильный код.
Как написать правильно?


Ну и введите операцию приведения типа в той строке, в которой компилятор просит.
Postoroniy_V
Цитата(Igor_U @ Nov 29 2005, 11:25) *
Есть массив AdrDecodeReciver[] во флеше в котором описаны адреса данных в структурах (озу), все данные 16-ти битные. Задача сформировать массив Usart[] из данных которые описаны в AdrDecodeReciver[].

Вот массив:
Код
const uint16_t *AdrDecodeReciver[NumberReciver+NumberTranciver] PROGMEM=
  {
  &ResultFirst.VoltageOff,
  &ResultFirst.CurrentOff,
  ........
  };


Вот функция которая это делает:

Код
extern void CopyReciverData(void)
{
uint8_t i;
uint16_t *Adr;

for (i=0;i<(NumberTranciver+NumberReciver);i++)
  {
    Adr=pgm_read_word(&AdrDecodeReciver[i]);
    Usart[i]=*Adr;
  }
}


Компилятор выдает варнинг:warning: assignment makes pointer from integer without a cast. Хотя компилит правильный код.
Как написать правильно?

компилеру "не нравится" видимо строка
Adr=pgm_read_word(&AdrDecodeReciver[i]);
сделайте так
Adr=(const uint16_t *)pgm_read_word(&AdrDecodeReciver[i]);
Igor_U
Ок. Спасибо. Было проглоченно. Сделал одним оператором: Usart[i]=*(uint16_t*)pgm_read_word(&AdrDecodeReciver[i]).
Но мне не понятно, почему приходится делать преобразование. Ведь я из массива AdrDecodeReciver беру именно указатели, при инициализации это явно указано. Почему же компилятор забывает об этом и считает, что я беру просто слово? Как-то не совсем понятно. Ожидаемо было бы наоборот - ругань в случае использования данных из AdrDecodeReciver не как указатели.
Postoroniy_V
что возвращает фуркция pgm_read_word?
задавайтесь этим вопросом каждый раз когда вам нужен именно указатель именно на тот тип какой вам нужен smile3046.gif
Igor_U
Цитата
что возвращает фуркция pgm_read_word?


ОК. Теперь понятно. Спасибо.
Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.
Invision Power Board © 2001-2025 Invision Power Services, Inc.