Проблема с функциями, возвращается нулевой результат Опции

[mempfis_]

Добрый день. Подскажите в чём может быть проблема: есть 2 функции вычисления crc8 одна для вычисления над данными из eeprom, а вторая - sram. Они ничем не отличаються кроме параметров функции: unsigned char __eeprom *pcBlock и unsigned char *pcBlock. Так вот вычисление crc8 над eeprom не работает всё время возвращает нулевой результат. В тоже время если перегрузить данные в оперативку и вычислить crc8 с помощью ф-ии crc8_table_sram то получается верный резльтат. Не могу понять в чём проблема crc8_result_table - общая переменная для обоих функций. И ещё вот какая проблема: когда эту переменную делаю не глобальной а static unsigned char crc8_result_table то перестаёт работать функция char crc8_table_sram Компилятор 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);
}
/***************************************************************************
***************************************************************************/

[sergeeff]

А указатель char __eeprom *pcBlock действительно указывает на правильный физический адрес?

[mempfis_]

А указатель char __eeprom *pcBlock действительно указывает на правильный физический адрес?

С eeprom разобрался - адрес вычисляется правильно, просто я брал размер всей структуры, которая хранит данные в eeprom, а надо было (размер-1) - последний байт хранит crc8.
Осталась проблема с sram. Если использовать глобальную переменную crc8_result_table всё работает, если я её делаю статической - не работает . Мистика. В чём может быть проблема?

[AHTOXA]

Осталась проблема с sram. Если использовать глобальную переменную crc8_result_table всё работает, если я её делаю статической - не работает . Мистика. В чём может быть проблема?

Если вы пишете

Код

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--)
    ...
}

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

[msalov]

Код

unsigned char crc8_table_sram(unsigned char *pcBlock, unsigned char len)
{
    unsigned char crc8_result_table = 0xFF;
    while (len--)
        crc8_result_table = Crc8Table[crc8_result_table ^ *pcBlock++];
    return(crc8_result_table);
}

А чем вам такая запись не угодила?

[mempfis_]

Код

unsigned char crc8_table_sram(unsigned char *pcBlock, unsigned char len)
{
    unsigned char crc8_result_table = 0xFF;
    while (len--)
        crc8_result_table = Crc8Table[crc8_result_table ^ *pcBlock++];
    return(crc8_result_table);
}

А чем вам такая запись не угодила?

Спасибо, такая запись тоже работает
Вобщем разобался с проблемой
Всем спасибо за советы
На будущее буду знать что статические переменные нужно инициализировать перед использованием записью вида:

Код

static unsigned char i;
i=0;

а не:

Код

static unsigned char i = 0;

[AHTOXA]

Вывод неверный:-)
Правильный вывод будет такой:
Если

статические переменные нужно инициализировать перед использованием записью вида:

Код

static unsigned char i;
i=0;

, то статическая переменная здесь скорее всего не нужна.

[mempfis_]

Вывод неверный:-)
... статическая переменная здесь скорее всего не нужна.

Объясните пожалуйста почему она сдесь не нужна?
Насколько я помню static обозначает что некоторая переменная будет использована в пределах функции и после выхода из функции про эту переменную "забудут" до следующего вызова функции.
Т.е. под неё выделяется память только когда нужно. Разве сдесь не тот случай когда нужно выделить переменную, с помощью неё чтото посчитать, возвратить результат и забыть про неё?


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

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