Форум разработчиков электроники ELECTRONIX.ru > Ошибка при компиляции GNU (студия)

Помощь - Поиск - Пользователи - Календарь

Полная версия этой страницы: Ошибка при компиляции GNU (студия)

Форум разработчиков электроники ELECTRONIX.ru > Сайт и форум > В помощь начинающему > Программирование

skyled

Jun 30 2010, 14:41

Код

buf_rx[15]=Crc8(rx,15);

Вот на эту строчку ругается.

Код

116: warning: passing argument 1 of 'Crc8' discards qualifiers from pointer target type

Не могу разобраться что это за ошибка. Подскажите, кто знает? Спасибо.

Палыч

Jun 30 2010, 15:04

Переменная rx - это у Вас что? Указатель должен быть первым аргументом процедуры

vadimuzzz

Jul 1 2010, 02:09

Цитата(skyled @ Jun 30 2010, 21:41)

Код

buf_rx[15]=Crc8(rx,15);

Вот на эту строчку ругается.

Код

116: warning: passing argument 1 of 'Crc8' discards qualifiers from pointer target type

Не могу разобраться что это за ошибка. Подскажите, кто знает? Спасибо.

сравните объявление rx и список параметров Crc8(). предупреждение гласит, что отброшен квалификатор типа у переменной rx (например, rx объявлен как volatile char[], а параметр у Crc8 имеет тип char*)

skyled

Jul 1 2010, 07:34

Вот так объявлено:

Код

volatile unsigned char buf_rx[16];
volatile unsigned char *rx=&buf_rx[16];

Вот функция:

Код

const unsigned char Crc8Table[256] = {
    0x00, 0x31, 0x62, 0x53, 0xC4, 0xF5, 0xA6, 0x97,
    0xB9, 0x88, 0xDB, 0xEA, 0x7D, 0x4C, 0x1F, 0x2E,
    0x43, 0x72, 0x21, 0x10, 0x87, 0xB6, 0xE5, 0xD4,
    0xFA, 0xCB, 0x98, 0xA9, 0x3E, 0x0F, 0x5C, 0x6D,
    0x86, 0xB7, 0xE4, 0xD5, 0x42, 0x73, 0x20, 0x11,
    0x3F, 0x0E, 0x5D, 0x6C, 0xFB, 0xCA, 0x99, 0xA8,
    0xC5, 0xF4, 0xA7, 0x96, 0x01, 0x30, 0x63, 0x52,
    0x7C, 0x4D, 0x1E, 0x2F, 0xB8, 0x89, 0xDA, 0xEB,
    0x3D, 0x0C, 0x5F, 0x6E, 0xF9, 0xC8, 0x9B, 0xAA,
    0x84, 0xB5, 0xE6, 0xD7, 0x40, 0x71, 0x22, 0x13,
    0x7E, 0x4F, 0x1C, 0x2D, 0xBA, 0x8B, 0xD8, 0xE9,
    0xC7, 0xF6, 0xA5, 0x94, 0x03, 0x32, 0x61, 0x50,
    0xBB, 0x8A, 0xD9, 0xE8, 0x7F, 0x4E, 0x1D, 0x2C,
    0x02, 0x33, 0x60, 0x51, 0xC6, 0xF7, 0xA4, 0x95,
    0xF8, 0xC9, 0x9A, 0xAB, 0x3C, 0x0D, 0x5E, 0x6F,
    0x41, 0x70, 0x23, 0x12, 0x85, 0xB4, 0xE7, 0xD6,
    0x7A, 0x4B, 0x18, 0x29, 0xBE, 0x8F, 0xDC, 0xED,
    0xC3, 0xF2, 0xA1, 0x90, 0x07, 0x36, 0x65, 0x54,
    0x39, 0x08, 0x5B, 0x6A, 0xFD, 0xCC, 0x9F, 0xAE,
    0x80, 0xB1, 0xE2, 0xD3, 0x44, 0x75, 0x26, 0x17,
    0xFC, 0xCD, 0x9E, 0xAF, 0x38, 0x09, 0x5A, 0x6B,
    0x45, 0x74, 0x27, 0x16, 0x81, 0xB0, 0xE3, 0xD2,
    0xBF, 0x8E, 0xDD, 0xEC, 0x7B, 0x4A, 0x19, 0x28,
    0x06, 0x37, 0x64, 0x55, 0xC2, 0xF3, 0xA0, 0x91,
    0x47, 0x76, 0x25, 0x14, 0x83, 0xB2, 0xE1, 0xD0,
    0xFE, 0xCF, 0x9C, 0xAD, 0x3A, 0x0B, 0x58, 0x69,
    0x04, 0x35, 0x66, 0x57, 0xC0, 0xF1, 0xA2, 0x93,
    0xBD, 0x8C, 0xDF, 0xEE, 0x79, 0x48, 0x1B, 0x2A,
    0xC1, 0xF0, 0xA3, 0x92, 0x05, 0x34, 0x67, 0x56,
    0x78, 0x49, 0x1A, 0x2B, 0xBC, 0x8D, 0xDE, 0xEF,
    0x82, 0xB3, 0xE0, 0xD1, 0x46, 0x77, 0x24, 0x15,
    0x3B, 0x0A, 0x59, 0x68, 0xFF, 0xCE, 0x9D, 0xAC
};

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

Хочет указатель и получает указатель.

volatile убрал, предупреждение осталось.

MrYuran

Jul 1 2010, 07:38

Приведите принудительно аргумент к unsigned char*, варнинг исчезнет.
Я обычно просто не обращаю внимания.
Это не ошибка, просто ахтунг, чтобы обратили внимание.

andrew_b

Jul 1 2010, 08:06

Цитата(skyled @ Jul 1 2010, 11:34)

Вот так объявлено:

Код

volatile unsigned char buf_rx[16];
volatile unsigned char *rx=&buf_rx[16];

rx у вас указывает куда-то за границу массива. Вы точно этого хотите?

MrYuran

Jul 1 2010, 08:08

Цитата(andrew_b @ Jul 1 2010, 12:06)

rx у вас указывает куда-то за границу массива.

Точно подмечено, я и не заметил...
Вот для этого, наверно, ахтунг и выставляется.

skyled

Jul 1 2010, 08:18

Цитата

rx у вас указывает куда-то за границу массива. Вы точно этого хотите?

Нет, я этого не хочу. Как правильно все сделать? Дело в том, что эта часть кода зашита в МК, а со стороны ПК точно такая же функция подсчитывает Crc8. МК формирует пакет и отправляет его, а комп принимает и проверяет целостность. При этом значения получаются разные. Так и не понятно пока почему. Монитор порта показавает, что в обе стороны все передается нормально.

Палыч

Jul 1 2010, 08:25

Зачем Вы завели переменную rx? Чем вас не устраивает

Код

buf_rx[15]= Crc8( buf_rx, 15 );

или

Код

buf_rx[15]= Crc8( &buf_rx[0], 15 );

Цитата(skyled @ Jul 1 2010, 12:18)

При этом значения получаются разные.

Это - потому, что контрольная сумма у Вас считается не пятнадцати передаваемых в buf_rx байтов, а следующих за массвиом...

skyled

Jul 1 2010, 08:27

Код

Зачем Вы завели переменную rx? Чем вас не устраивает

Да вобщем то меня устраивает все, что правильно работает. Хотела функция указатель - получила его. Может действительно переделать ее. Я про эту сторону вопроса и не подумал. Но всетаки как правильно тут с указателем разобраться?

Палыч

Jul 1 2010, 08:31

Выбросите переменную rx, если она не используется в других местах. Исправьте вызов процедуры, как Вам советовал выше.

P.S. Нужно переделать только вызов процедуры, саму процедуру переделывать не нужно!

MrYuran

Jul 1 2010, 08:35

Цитата(skyled @ Jul 1 2010, 12:27)

Но всетаки как правильно тут с указателем разобраться?

Имя массива - это и есть указатель на начало массива.
Если вы указываете явно номер элемента, то вставляете в функцию адрес не начала массива, а указанного элемента.
Со всеми вытекающими.
В вашем случае вы вообще указываете на адрес за последним элементом массива.

Цитата(Палыч @ Jul 1 2010, 12:31)

Выбросите переменную rx, если она не используется в других местах.

Указатель rx имеет смысл только в том случае, если буферов несколько и надо между ними переключаться.
Если буфер один - смысла нет, только лишний расход памяти и лишние телодвижения при обращении.

vadimuzzz

Jul 1 2010, 08:45

Цитата(skyled @ Jul 1 2010, 14:34)

volatile убрал, предупреждение осталось.

нету никаких ворнингов:

Код

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

int main()
{
    unsigned char buf_rx[16];
    unsigned char *rx=&buf_rx[0];
    return Crc8(rx,16);
}

или так

Код

unsigned char buf_rx[16];
return Crc8(&buf_rx[0],16);

MrYuran

Jul 1 2010, 08:48

Цитата(vadimuzzz @ Jul 1 2010, 12:45)

нету никаких ворнингов:

Возможно, они у вас просто отключены.

vadimuzzz

Jul 1 2010, 08:55

Цитата(MrYuran @ Jul 1 2010, 15:48)

Возможно, они у вас просто отключены.

нет, если добавить volatile, они появятся. собственно вот опции: -O0 -g3 -Wall -c -fmessage-length=0. просто топикстартеру надо иметь в виду, что volatile там не просто так стоит.

skyled

Jul 1 2010, 09:05

Код

unsigned char *rx=&buf_rx[0];

Так я делал и предупреждение оставалось.

Код

unsigned char Crc8(unsigned char len) {
    unsigned char crc = 0xFF;
    while (len--)
    crc = Crc8Table[crc ^ buf_rx[len]];
    return crc;}

Вот так оно теперь с обоих сторон, но значения всеравно разные. Что делать?

vadimuzzz

Jul 1 2010, 09:10

Цитата(skyled @ Jul 1 2010, 16:05)

Что делать?

давайте содержимое какого-нибудь пакета от МК

Палыч

Jul 1 2010, 09:16

Цитата(skyled @ Jul 1 2010, 13:05)

...но значения всеравно разные. Что делать?

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

Цитата(skyled @ Jul 1 2010, 13:05)

Код

while (len--)
crc = Crc8Table[crc ^ buf_rx[len]];

Так - нельзя! Байт buf_rx[15] не будет участвовать в формировании контрольной суммы!

P.S. Зря полезли править процедуру подсчета контрольной суммы - там-то всё было верно.
P.P.S. Сколько Вы байт передаёте в сообщении? По-моему - шестнадцать (с buf_rx[0] по buf_rx[15] включительно). Почему при вызове процедуры указываете длину - пятнадцать?

skyled

Jul 1 2010, 09:18

Вот пакет:

Код

F0 55 FF 80 01 00 01 0F 01 02 03 04 05 06 07 FF

Последние FF и есть crc8 предидущих 15.

Цитата

crc = Crc8Table[crc ^ buf_rx[(len+1)]];

Вот так сделал. Последний байт стал FE. Не совпадает.

MrYuran

Jul 1 2010, 09:21

Цитата(vadimuzzz @ Jul 1 2010, 12:55)

собственно вот опции: -O0 -g3 -Wall -c -fmessage-length=0.

а у меня ещё вот:
WARNINGS += -Wshadow -Wpointer-arith -Wbad-function-cast -Wcast-align -Wsign-compare
Ну и ещё по мелочи...
-Wall, как ни странно, некоторые варнинги не выдаёт
Можно педантик прицепить

skyled

Jul 1 2010, 09:23

Код

P.S. Зря полезли править процедуру подсчета контрольной суммы - там-то всё было верно.

А выход какой?

vadimuzzz

Jul 1 2010, 09:31

Цитата(skyled @ Jul 1 2010, 16:23)

А выход какой?

у меня вот:
Crc8(&buf_rx[0],15)=0xb9
по-моему контроллер брешет.

Сергей Борщ

Jul 1 2010, 09:31

Цитата(skyled @ Jul 1 2010, 12:05)

Вот так оно теперь с обоих сторон, но значения всеравно разные. Что делать?

Теперь вы обсчитываете массив с конца в начало, что несколько неожиданно для расчета CRC.

MrYuran

Jul 1 2010, 09:31

Цитата(skyled @ Jul 1 2010, 13:23)

А выход какой?

Вернуть всё взад, только изменить аргумент при вызове функции на адрес массива (без индекса)

skyled

Jul 1 2010, 09:34

Компьютер говорит 0хЕ5. Оба брешут. Как бы "контрольные весы" изобразить?

vadimuzzz

Jul 1 2010, 09:36

Код

#include <stdio.h>

const unsigned char Crc8Table[256] = {
    0x00, 0x31, 0x62, 0x53, 0xC4, 0xF5, 0xA6, 0x97,
    0xB9, 0x88, 0xDB, 0xEA, 0x7D, 0x4C, 0x1F, 0x2E,
    0x43, 0x72, 0x21, 0x10, 0x87, 0xB6, 0xE5, 0xD4,
    0xFA, 0xCB, 0x98, 0xA9, 0x3E, 0x0F, 0x5C, 0x6D,
    0x86, 0xB7, 0xE4, 0xD5, 0x42, 0x73, 0x20, 0x11,
    0x3F, 0x0E, 0x5D, 0x6C, 0xFB, 0xCA, 0x99, 0xA8,
    0xC5, 0xF4, 0xA7, 0x96, 0x01, 0x30, 0x63, 0x52,
    0x7C, 0x4D, 0x1E, 0x2F, 0xB8, 0x89, 0xDA, 0xEB,
    0x3D, 0x0C, 0x5F, 0x6E, 0xF9, 0xC8, 0x9B, 0xAA,
    0x84, 0xB5, 0xE6, 0xD7, 0x40, 0x71, 0x22, 0x13,
    0x7E, 0x4F, 0x1C, 0x2D, 0xBA, 0x8B, 0xD8, 0xE9,
    0xC7, 0xF6, 0xA5, 0x94, 0x03, 0x32, 0x61, 0x50,
    0xBB, 0x8A, 0xD9, 0xE8, 0x7F, 0x4E, 0x1D, 0x2C,
    0x02, 0x33, 0x60, 0x51, 0xC6, 0xF7, 0xA4, 0x95,
    0xF8, 0xC9, 0x9A, 0xAB, 0x3C, 0x0D, 0x5E, 0x6F,
    0x41, 0x70, 0x23, 0x12, 0x85, 0xB4, 0xE7, 0xD6,
    0x7A, 0x4B, 0x18, 0x29, 0xBE, 0x8F, 0xDC, 0xED,
    0xC3, 0xF2, 0xA1, 0x90, 0x07, 0x36, 0x65, 0x54,
    0x39, 0x08, 0x5B, 0x6A, 0xFD, 0xCC, 0x9F, 0xAE,
    0x80, 0xB1, 0xE2, 0xD3, 0x44, 0x75, 0x26, 0x17,
    0xFC, 0xCD, 0x9E, 0xAF, 0x38, 0x09, 0x5A, 0x6B,
    0x45, 0x74, 0x27, 0x16, 0x81, 0xB0, 0xE3, 0xD2,
    0xBF, 0x8E, 0xDD, 0xEC, 0x7B, 0x4A, 0x19, 0x28,
    0x06, 0x37, 0x64, 0x55, 0xC2, 0xF3, 0xA0, 0x91,
    0x47, 0x76, 0x25, 0x14, 0x83, 0xB2, 0xE1, 0xD0,
    0xFE, 0xCF, 0x9C, 0xAD, 0x3A, 0x0B, 0x58, 0x69,
    0x04, 0x35, 0x66, 0x57, 0xC0, 0xF1, 0xA2, 0x93,
    0xBD, 0x8C, 0xDF, 0xEE, 0x79, 0x48, 0x1B, 0x2A,
    0xC1, 0xF0, 0xA3, 0x92, 0x05, 0x34, 0x67, 0x56,
    0x78, 0x49, 0x1A, 0x2B, 0xBC, 0x8D, 0xDE, 0xEF,
    0x82, 0xB3, 0xE0, 0xD1, 0x46, 0x77, 0x24, 0x15,
    0x3B, 0x0A, 0x59, 0x68, 0xFF, 0xCE, 0x9D, 0xAC
};

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

unsigned char buf_rx[16]={0xF0, 0x55, 0xFF, 0x80, 0x01, 0x00, 0x01, 0x0F, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0xFF};

int main()
{
    printf("Crc8(&buf_rx[0],15)=0x%x\n",Crc8(&buf_rx[0],15));
    return Crc8(&buf_rx[0],16);
}

что выдает?

demiurg_spb

Jul 1 2010, 09:42

Цитата(Сергей Борщ @ Jul 1 2010, 13:31)

Теперь вы обсчитываете массив с конца в начало, что несколько неожиданно для расчета CRC.

Действительно, это не дело!
CRC, посчитанное для пакета данных с учётом добавленного в конце CRC должно дать 0.
Если это не так - вы изобретаете велосипед...

Палыч

Jul 1 2010, 09:47

Цитата(skyled @ Jul 1 2010, 13:18)

Вот так сделал.

Извините, по своей невнимательности я Вас сбил с правильного пути... Отчего-то решил, что в сообщении 16 байт данных + контрольная сумма. Было правильно. Ещё раз извините.

Ink

Jul 1 2010, 09:58

Я не понял одного: на компе и в девайсе один и тот же известный алгоритм? Тогда какие проблемы-то?

Код

volatile unsigned char buf_rx[16]={0xF0,0x55,0xFF,0x80,0x01,0x00,0x01,0x0F,0x01,0x02,0x03,0x04,0x05,0x06,0x07
,0xFF};

const unsigned char Crc8Table[256] = {
...
};

Код

unsigned char Crc8(unsigned char *buf, unsigned char len)
{
    unsigned char crc = 0xFF;
    while (len--)
      crc = Crc8Table[crc ^ *buf++];
    return crc;
}

int main(void)
{
  unsigned char crc;

  printf("hello\n");

  crc=Crc8((unsigned char*)buf_rx,15);
  printf("CRC=%x\n",(int)crc);
  return 0;
}

Вот так будет считаться везде одинаково и без варнингов (хотя на счет volatile - сами смотрите по месту). Но будет там, естественно, не FF, как у вас в пакете.

skyled

Jul 1 2010, 10:52

Код

unsigned char Crc8(unsigned char len) {
    unsigned char crc = 0xFF;
    unsigned char q=0;
    while (len--)
    {crc = Crc8Table[crc ^ buf_rx[q]]; q++;}
    return crc;}

Компьютерная часть. =185

Код

unsigned char Crc8(unsigned char len)
{unsigned char crc = 0xFF;
unsigned char q=0;
while (len--)
{crc = Crc8Table[crc ^ buf_rx[q]]; q++;}
return crc;}

Котроллерная часть. =255
Откуда разница не понимаю.

Цитата

А вот и не будет. Переделал в точности с Вашими рекомендациями. Результат тот же. 185 и 255

Код

#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/pgmspace.h>
#define F_CPU 14745600UL // 14.7456 MHz
#include <util/delay.h>
#include <stdio.h>
#define _rx (PORTC &= ~(1<<5))
#define _tx (PORTC |= (1<<5))
#define l4_on (PORTD &= ~(1<<4))
#define l4_off (PORTD |= (1<<4))
#define l3_on (PORTD &= ~(1<<5))
#define l3_off (PORTD |= (1<<5))
#define l2_on (PORTD &= ~(1<<6))
#define l2_off (PORTD |= (1<<6))
#define l1_on (PORTD &= ~(1<<7))
#define l1_off (PORTD |= (1<<7))
#define SETBIT(ADDRESS,BIT) (ADDRESS |= (1<<BIT))
#define CLEARBIT(ADDRESS,BIT) (ADDRESS &= ~(1<<BIT))
#define CHECKBIT(ADDRESS,BIT) (ADDRESS & (1<<BIT))

FUSES =
    {
        .low = 0xFF,
        .high = 0xDD,
        .extended = 0xFF,
    };

volatile unsigned char buf_rx[16];
volatile unsigned char buf_tx[16];
//unsigned char *rx=&buf_rx[0];
volatile unsigned char n;
volatile unsigned char m=16;
volatile unsigned char data;
volatile unsigned char bit;
volatile unsigned char vol;
volatile unsigned char __crc;

/*
  Name  : CRC-8
  Poly  : 0x31    x^8 + x^5 + x^4 + 1
  Init  : 0xFF
  Revert: false
  XorOut: 0x00
  Check : 0xF7 ("123456789")
  MaxLen: 15 байт (127 бит) - обнаружение
    одинарных, двойных, тройных и всех нечетных ошибок
*/
const unsigned char Crc8Table[256] = {
    0x00, 0x31, 0x62, 0x53, 0xC4, 0xF5, 0xA6, 0x97,
    0xB9, 0x88, 0xDB, 0xEA, 0x7D, 0x4C, 0x1F, 0x2E,
    0x43, 0x72, 0x21, 0x10, 0x87, 0xB6, 0xE5, 0xD4,
    0xFA, 0xCB, 0x98, 0xA9, 0x3E, 0x0F, 0x5C, 0x6D,
    0x86, 0xB7, 0xE4, 0xD5, 0x42, 0x73, 0x20, 0x11,
    0x3F, 0x0E, 0x5D, 0x6C, 0xFB, 0xCA, 0x99, 0xA8,
    0xC5, 0xF4, 0xA7, 0x96, 0x01, 0x30, 0x63, 0x52,
    0x7C, 0x4D, 0x1E, 0x2F, 0xB8, 0x89, 0xDA, 0xEB,
    0x3D, 0x0C, 0x5F, 0x6E, 0xF9, 0xC8, 0x9B, 0xAA,
    0x84, 0xB5, 0xE6, 0xD7, 0x40, 0x71, 0x22, 0x13,
    0x7E, 0x4F, 0x1C, 0x2D, 0xBA, 0x8B, 0xD8, 0xE9,
    0xC7, 0xF6, 0xA5, 0x94, 0x03, 0x32, 0x61, 0x50,
    0xBB, 0x8A, 0xD9, 0xE8, 0x7F, 0x4E, 0x1D, 0x2C,
    0x02, 0x33, 0x60, 0x51, 0xC6, 0xF7, 0xA4, 0x95,
    0xF8, 0xC9, 0x9A, 0xAB, 0x3C, 0x0D, 0x5E, 0x6F,
    0x41, 0x70, 0x23, 0x12, 0x85, 0xB4, 0xE7, 0xD6,
    0x7A, 0x4B, 0x18, 0x29, 0xBE, 0x8F, 0xDC, 0xED,
    0xC3, 0xF2, 0xA1, 0x90, 0x07, 0x36, 0x65, 0x54,
    0x39, 0x08, 0x5B, 0x6A, 0xFD, 0xCC, 0x9F, 0xAE,
    0x80, 0xB1, 0xE2, 0xD3, 0x44, 0x75, 0x26, 0x17,
    0xFC, 0xCD, 0x9E, 0xAF, 0x38, 0x09, 0x5A, 0x6B,
    0x45, 0x74, 0x27, 0x16, 0x81, 0xB0, 0xE3, 0xD2,
    0xBF, 0x8E, 0xDD, 0xEC, 0x7B, 0x4A, 0x19, 0x28,
    0x06, 0x37, 0x64, 0x55, 0xC2, 0xF3, 0xA0, 0x91,
    0x47, 0x76, 0x25, 0x14, 0x83, 0xB2, 0xE1, 0xD0,
    0xFE, 0xCF, 0x9C, 0xAD, 0x3A, 0x0B, 0x58, 0x69,
    0x04, 0x35, 0x66, 0x57, 0xC0, 0xF1, 0xA2, 0x93,
    0xBD, 0x8C, 0xDF, 0xEE, 0x79, 0x48, 0x1B, 0x2A,
    0xC1, 0xF0, 0xA3, 0x92, 0x05, 0x34, 0x67, 0x56,
    0x78, 0x49, 0x1A, 0x2B, 0xBC, 0x8D, 0xDE, 0xEF,
    0x82, 0xB3, 0xE0, 0xD1, 0x46, 0x77, 0x24, 0x15,
    0x3B, 0x0A, 0x59, 0x68, 0xFF, 0xCE, 0x9D, 0xAC};

unsigned char Crc8(unsigned char *buf, unsigned char len)
{
    unsigned char crc = 0xFF;
    while (len--)
      crc = Crc8Table[crc ^ *buf++];
    return crc;
}

void USART_Transmit( unsigned char data ){
while ( !( UCSR0A & (1<<UDRE0)) );/* Wait for empty transmit buffer */
UDR0 = data;}/* Put data into buffer, sends the data */

ISR(USART_RX_vect){
data=UDR0;
buf_rx[n]=data;
PORTD ^= (1<<7);
if(data==0xF0){n=0; buf_rx[n]=data;}
n++;
if(n==8){TCCR0B=0b00000101;n=0;}
}

ISR(TIMER0_OVF_vect){
        TCCR0B=0;
        TCNT0=0;
//        asm("cli");
        _tx;
        __crc=Crc8((unsigned char*)buf_rx,15);
        buf_rx[8]=1;
        buf_rx[9]=2;
        buf_rx[10]=3;
        buf_rx[11]=4;
        buf_rx[12]=5;
        buf_rx[13]=6;
        buf_rx[14]=7;
        buf_rx[15]=__crc;
        _delay_us(250);
        for(unsigned char i=0;i<m;i++)
        {USART_Transmit(buf_rx[i]);}
        _delay_us(100);
        _rx;
        for(unsigned char i=0;i<m;i++)
        {buf_rx[i]=0;}
//        asm("sei");
        }

int main(void)
{
DDRB=0b00111111;
DDRC=0b11111111;
DDRD=0b11110000;
PORTB=0b00000000;
PORTC=0b00000000;
PORTD=0b11111100;
UCSR0B=0b10011000;//115200
UCSR0C=0b00000110;//8 бит; без четности
UBRR0H=0;         //1 стоп-бит
UBRR0L=7;         //
//EICRA=0b00000101;
//EIMSK=0b00000011;
TCNT0=0;
TIMSK0=0b00000001;

asm("sei");
while(1)
{

asm("nop");

}//while(1)
return 0;
}//main

Вот весь код с микроконтроллерной стороны. Может кто-то что-то увидит. Я совершенно ничего не могу понять.

Убивает то, что от изменения значений в пакете CRC не меняется.

Проверка на тестовом примере дала результат 8C вместо F7. Хотел отключить оптимизацию, но тогда программа в МК не влазит. Какие могут быть причины неправильной работы? Просто нет уже ни одной идеи. Спасибо.

Палыч

Jul 1 2010, 11:02

Цитата(skyled @ Jul 1 2010, 14:52)

Код

        __crc=Crc8((unsigned char*)buf_rx,15);
        buf_rx[8]=1;
        buf_rx[9]=2;
        buf_rx[10]=3;
        buf_rx[11]=4;
        buf_rx[12]=5;
        buf_rx[13]=6;
        buf_rx[14]=7;
        buf_rx[15]=__crc;

Какие могут быть причины неправильной работы?

Чего-то я не понимаю... Вы 1) принимаете восемь байт в буфер; 2) считаете контрольную сумму пятнадцати байт буфера; 3) дописываете в буфер семь бай и контрольную сумму; 4) передаёте буфер назад. Передвинуть вычисление контрольной суммы после заполнения буфера!

skyled

Jul 1 2010, 11:18

Я
1) принимаю 8 байт;
2) дописываю к ним еще 7;
3) вычисляю crc;
4) заношу в буфер 16-й элемент;
5) отсылаю буфер.

Спасибо за замечание, но это опечатка. Просто копировал по ходу правки. Конечно же считаем после заполнения буфера.

Код

buf_rx[8]=__crc_tst;
buf_rx[9]=2;
buf_rx[10]=3;
buf_rx[11]=4;
buf_rx[12]=5;
buf_rx[13]=6;
buf_rx[14]=7;
__crc=Crc8((unsigned char*)buf_rx,15);
buf_rx[15]=__crc;

vadimuzzz

Jul 1 2010, 11:31

сначала идет "отче наш", а потом "иже еси". т.е. я хотел сказать что контрольная сумма сичтается после заполнения пакета, а не до, как у вас:

Код

skyled

Jul 1 2010, 11:33

Причина локализована, можно сказать. Что-то не то я считаю вместо буфера. Отдельно взятая функция правильно считает тестовый буфер (там действительно 8С) и передаваемый буфер (там 185). Тестовый буфер в том коде, что опубликован, считается верно. Значит проблема с подстановкой значения буфера. В чем тут может быть причина?

Цитата

сначала идет "отче наш", а потом "иже еси"

Правильно. Сначала считается с 0 по 14 (15 байт), и дописывается в 15 ячейку (16-элемент).

vadimuzzz

Jul 1 2010, 11:41

забейте в контроллер массив констант, как в примере, посчитайте КС и отправьте на комп

skyled

Jul 1 2010, 12:09

Проблема оказалась в нижней строчке:

Код

for(unsigned char i=0;i<16;i++) {USART_Transmit(buf_rx[i]);}
_delay_us(200);

Там было 100. Изменил на 200 и последний байт стал уходить верным. При скорости 115 200 время на отправку байта 79 мкс. Зачем 200 не понимаю.

Код

забейте в контроллер массив констант, как в примере, посчитайте КС и отправьте на комп

Именно так и нашел. Спасибо за дельный совет.

Есть еще одна тонкость. Иногда (примерно 20-30%) случаев контроллер ничего не отсылает на запрос. Теперь думаю откуда эти сбои.

Ink

Jul 1 2010, 12:09

От _delay_us() надо избавляться.
И хорошо было бы не делать пауз в прерывании.

skyled

Jul 1 2010, 12:21

Код

И хорошо было бы не делать пауз в прерывании.

Это я понимаю. Думаю как.

vadimuzzz

Jul 1 2010, 12:46

Цитата(skyled @ Jul 1 2010, 19:09)

Есть еще одна тонкость. Иногда (примерно 20-30%) случаев контроллер ничего не отсылает на запрос. Теперь думаю откуда эти сбои.

у вас не совсем правильно определяется начало пакета: не учитывается, что внутри пакета может оказаться 0xF0 не только в заголовке, м.б. проблема в этом.

skyled

Jul 1 2010, 13:19

Код

Это пока. Будет маскирующий байт. Я сознательно не писал в пакет другие F0.

Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.