| |
|
  |
 Ошибка при компиляции GNU (студия), Нужна подсказка |
|
|
|
|
 Jul 1 2010, 09:05
|
Местный
Группа: Участник
Сообщений: 217
Регистрация: 11-06-10
Пользователь №: 57 868
|
Код 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;} Вот так оно теперь с обоих сторон, но значения всеравно разные. Что делать?
|
|
|
|
|
|
|
|
Jul 1 2010, 09:16
|
Гуру
Группа: Свой
Сообщений: 2 399
Регистрация: 10-05-06
Из: г. Новочеркасск
Пользователь №: 16 954
|
Цитата(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] включительно). Почему при вызове процедуры указываете длину - пятнадцать?
|
|
|
|
|
|
|
|
Jul 1 2010, 09:18
|
Местный
Группа: Участник
Сообщений: 217
Регистрация: 11-06-10
Пользователь №: 57 868
|
Вот пакет: Код 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. Не совпадает.
Сообщение отредактировал skyled - Jul 1 2010, 09:21
|
|
|
|
|
|
|
|
Jul 1 2010, 09:21
|
Беспросветный оптимист
Группа: Свой
Сообщений: 4 640
Регистрация: 26-12-07
Из: Н.Новгород
Пользователь №: 33 646
|
Цитата(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, как ни странно, некоторые варнинги не выдаёт Можно педантик прицепить
--------------------
Программирование делится на системное и бессистемное. ©Моё :)
— а для кого-то БГ — это Bill Gilbert =)
|
|
|
|
|
|
|
|
Jul 1 2010, 09:23
|
Местный
Группа: Участник
Сообщений: 217
Регистрация: 11-06-10
Пользователь №: 57 868
|
Код P.S. Зря полезли править процедуру подсчета контрольной суммы - там-то всё было верно. А выход какой?
|
|
|
|
|
|
|
|
Jul 1 2010, 09:36
|
Гуру
Группа: Свой
Сообщений: 2 291
Регистрация: 21-07-05
Пользователь №: 6 988
|
Код #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);
} что выдает?
|
|
|
|
|
|
|
|
Jul 1 2010, 09:42
|
неотягощённый злом
Группа: Свой
Сообщений: 2 746
Регистрация: 31-01-08
Из: Санкт-Петербург
Пользователь №: 34 643
|
Цитата(Сергей Борщ @ Jul 1 2010, 13:31) Теперь вы обсчитываете массив с конца в начало, что несколько неожиданно для расчета CRC. Действительно, это не дело! CRC, посчитанное для пакета данных с учётом добавленного в конце CRC должно дать 0. Если это не так - вы изобретаете велосипед...
--------------------
“Будьте внимательны к своим мыслям - они начало поступков” (Лао-Цзы)
|
|
|
|
|
|
|
|
Jul 1 2010, 09:58
|
Участник
Группа: Участник
Сообщений: 40
Регистрация: 14-08-07
Пользователь №: 29 776
|
Я не понял одного: на компе и в девайсе один и тот же известный алгоритм? Тогда какие проблемы-то? Код 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, как у вас в пакете.
|
|
|
|
|
|
|
|
Jul 1 2010, 10:52
|
Местный
Группа: Участник
Сообщений: 217
Регистрация: 11-06-10
Пользователь №: 57 868
|
Код 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 Откуда разница не понимаю. Цитата Вот так будет считаться везде одинаково и без варнингов (хотя на счет volatile - сами смотрите по месту). Но будет там, естественно, не FF, как у вас в пакете. А вот и не будет. Переделал в точности с Вашими рекомендациями. Результат тот же. 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. Хотел отключить оптимизацию, но тогда программа в МК не влазит. Какие могут быть причины неправильной работы? Просто нет уже ни одной идеи. Спасибо.
Сообщение отредактировал skyled - Jul 1 2010, 10:33
|
|
|
|
|
|
|
|
2 чел. читают эту тему (гостей: 2, скрытых пользователей: 0)
Пользователей: 0
|
|
|
