реклама на сайте
подробности

 
 
> Проблема с функциями, возвращается нулевой результат
mempfis_
сообщение Aug 18 2008, 07:05
Сообщение #1


Профессионал
*****

Группа: Свой
Сообщений: 1 001
Регистрация: 27-06-06
Пользователь №: 18 409



Добрый день. Подскажите в чём может быть проблема: есть 2 функции вычисления crc8 одна для вычисления над данными из eeprom, а вторая - sram. Они ничем не отличаються кроме параметров функции: unsigned char __eeprom *pcBlock и unsigned char *pcBlock. Так вот вычисление crc8 над eeprom не работает sad.gif всё время возвращает нулевой результат. В тоже время если перегрузить данные в оперативку и вычислить crc8 с помощью ф-ии crc8_table_sram то получается верный резльтат. Не могу понять в чём проблема sad.gif crc8_result_table - общая переменная для обоих функций. И ещё вот какая проблема: когда эту переменную делаю не глобальной а static unsigned char crc8_result_table то перестаёт работать функция char crc8_table_sram sad.gif Компилятор IAR 5.10, контроллер ATmega8. Сам я на программиста не учился может поэтому не замечаю каких-то тонкостей в написании программ на С....

Код
unsigned char crc8_result_table;

/***************************************************************************
***************************************************************************/
//вычисление crc8 над данными в еепром табличным методом
unsigned char crc8_table(unsigned char __eeprom *pcBlock, unsigned char len)
{
    crc8_result_table = 0xFF;

    while (len--)
        crc8_result_table = Crc8Table[crc8_result_table ^ *pcBlock++];

return(crc8_result_table);
}
/***************************************************************************
***************************************************************************/

/***************************************************************************
***************************************************************************/
//вычисление crc8 над данными в sram табличным методом
unsigned char crc8_table_sram(unsigned char *pcBlock, unsigned char len)
{
    crc8_result_table = 0xFF;

    while (len--)
        crc8_result_table = Crc8Table[crc8_result_table ^ *pcBlock++];

return(crc8_result_table);
}
/***************************************************************************
***************************************************************************/
Go to the top of the page
 
+Quote Post
 
Start new topic
Ответов
sergeeff
сообщение Aug 18 2008, 07:47
Сообщение #2


Профессионал
*****

Группа: Свой
Сообщений: 1 481
Регистрация: 10-04-05
Пользователь №: 4 007



А указатель char __eeprom *pcBlock действительно указывает на правильный физический адрес?
Go to the top of the page
 
+Quote Post
mempfis_
сообщение Aug 18 2008, 09:08
Сообщение #3


Профессионал
*****

Группа: Свой
Сообщений: 1 001
Регистрация: 27-06-06
Пользователь №: 18 409



Цитата(sergeeff @ Aug 18 2008, 11:47) *
А указатель char __eeprom *pcBlock действительно указывает на правильный физический адрес?

С eeprom разобрался - адрес вычисляется правильно, просто я брал размер всей структуры, которая хранит данные в eeprom, а надо было (размер-1) - последний байт хранит crc8.
Осталась проблема с sram. Если использовать глобальную переменную crc8_result_table всё работает, если я её делаю статической - не работает 07.gif . Мистика. В чём может быть проблема?
Go to the top of the page
 
+Quote Post
AHTOXA
сообщение Aug 18 2008, 09:25
Сообщение #4


фанат дивана
******

Группа: Свой
Сообщений: 3 387
Регистрация: 9-08-07
Из: Уфа
Пользователь №: 29 684



Цитата(mempfis_ @ Aug 18 2008, 15:08) *
Осталась проблема с sram. Если использовать глобальную переменную crc8_result_table всё работает, если я её делаю статической - не работает 07.gif . Мистика. В чём может быть проблема?


Если вы пишете
Код
unsigned char crc8_table_sram(unsigned char *pcBlock, unsigned char len)
{
    crc8_result_table = 0xFF;
    while (len--)
    ...
}


, то присвоение crc8_result_table = 0xFF происходит каждый раз при входе в функцию.
А если добавить static:
Код
    static unsigned char crc8_result_table = 0xFF;

, то присвоение происходит один раз, при старте программы.

Поэтому для правильного подсчёта надо добавить строчку
Код
unsigned char crc8_table_sram(unsigned char *pcBlock, unsigned char len)
{
    static unsigned char crc8_result_table;
    crc8_result_table = 0xFF;
    while (len--)
    ...
}


Хотя мне не ясно, зачем здесь глобальная либо статическая переменная...


--------------------
Если бы я знал, что такое электричество...
Go to the top of the page
 
+Quote Post
mempfis_
сообщение Aug 18 2008, 10:31
Сообщение #5


Профессионал
*****

Группа: Свой
Сообщений: 1 001
Регистрация: 27-06-06
Пользователь №: 18 409



a14.gif AHTOXA

с записью типа:
Код
static unsigned char crc8_result_table = 0xFF;
не работает.


А вот с:
Код
static unsigned char crc8_result_table;
crc8_result_table = 0xFF;
всё верно работает.


Т.е. в первом случае компилятор при старте программы один раз проинициализирует эту переменную и при следующих вызовах функции будет использовать значения оставшиеся от предидущих вызовов функции. А во втором случае будет заново инициализировать её необходимой величиной (0xFF) ?

P.S. А как Вы предлагаете вычислять без использования глобальных или статических переменных?
Go to the top of the page
 
+Quote Post



Reply to this topicStart new topic
1 чел. читают эту тему (гостей: 1, скрытых пользователей: 0)
Пользователей: 0

 


RSS Текстовая версия Сейчас: 28th July 2025 - 09:08
Рейтинг@Mail.ru


Страница сгенерированна за 0.01409 секунд с 7
ELECTRONIX ©2004-2016