W programie istnieje zmienna globalna wejscia1, chciałbym tą zmienną "ładnie ubrać" tak aby mogła zostać wysłana przez UART, oraz na końcu dodać sumę kontrolną ModbusCRC.
Sumę kontrolną obliczam za pomocą takiej funkcji:
[syntax=c]// ==== OBLICZANIE CRC
uint16_t ModbusCRC(char * buf, uint8_t size){
uint16_t crc = 0xffff;
int i;
while(size--){
crc ^= *buf;
buf++;
for(i=0; i<8; i++){
if(crc & 1){
crc >>= 1;
crc ^= 0xA001;
}
else{
crc >>= 1;
}
}
}
return crc;
}[/syntax]
Natomiast funkcja przygotowująca ramkę wygląda tak:
Adres jest to również zmienna typu char przechowująca dwu znakowy adres modułu.
[syntax=c]//funkcja zwraca status wszystkich wejsc
void get_in_all(void){
char status[20]="+";
char bufor[2];
const char *znak="^";
strcat(status,adres);
strcat(status,"^3^");
itoa(wejscia1,bufor,2);
strcat(status,bufor);
strcat(status,znak);
// == CRC
uint16_t crc;
char *b;
//obliczanie sumy kontrolnej, dodawanie na koniec ramki
crc = ModbusCRC(status, sizeof(status));
b = itoa(crc,bufor,2);
strcat(status,b);
crc >>= 8;
b = itoa(crc,bufor,2);
strcat(status,b);
//wyslanie ramki danych
uart_puts(status);
}[/syntax]
Docelowo wysłany łańcuch ma wyglądać tak:
+ADRES^3^TUTAJ ZMIENNA WYJSCIA1^CRC
Domyślam się, że robię coś źle przy próbie zamiany zwracanej sumy kontrolnej jako uint16_t do dwóch char. Może ktoś ma pomysł jak powinno to wyglądać prawidłowo, bardziej zoptymalizowane (bo pewnie coś da się uprościć)?
Pozdrawiam.
EDIT:
Chyba już sobie poradziłem - troszkę źle podchodziłem do tego - za bardzo zainspirowałem się funkcją porównującą CRC. Poniżej podaję kod funkcji zwracającej wersję firmware sterownika gdzie obliczanie CRC i przygotowanie ramki działa poprawnie:
[syntax=c]//wersja firmware
void get_fw(void){
char status[100]="A";
strcat(status,adres);
strcat(status,"^5^IOC_REV1_FW0011_20150302^");
uint16_t crc;
char *b;
char bufor[2];
crc = ModbusCRC(status, strlen(status));
b = itoa(crc,bufor,16);
strcat(status,b);
uart_puts(status);
}[/syntax]Statystyki: Napisane przez n326 — 3 mar 2015, o 09:58
]]>