ATNEL tech-forum

Re: mK_multi_uart i Nextion

2018-12-13T10:12:07+01:00

Dzięki. Też mnie tak oświeciło jak pisałem zapytanie.
I oczywiście poprawię się z tymi callbackami.

Edit.
Hmm problem jest taki że to nie końca ascii i "13" może wystąpić.
Czy gdzieś jeszcze jest sprawdzany znak końca linii ?

Statystyki: Napisane przez jaca_76 — 13 gru 2018, o 10:12

Re: mK_multi_uart i Nextion

2018-12-13T10:02:35+01:00

Wywołujesz callbacki z przerwania ?!?!?!?! .... toż to istna masakra - taka porada - NIGDY tak nie rób. Tylko proszę nie pisz mi tutaj:

"a mi to działa..."

Bo odpowiedź brzmi - pewnie, że działa - to tak samo jak program z _delay_ms(1000) w przerwaniu też może działać ....

Nie piszę tego broń Boże ze złością czy coś - tylko tak jak mówię ze zdziwieniem no i chcę ci poradzić dobrze. Skoro mówiłeś, że bibliotekę jako tako opanowałeś to sprawdź, że w przerwaniu jest tylko ustawiana flaga dzięki której później w zdarzeniu EVENT wywoływane są callbacki i ty dokładnie tak samo możesz zrobić.

ok to nie na temat twojego problemu - no ale jak widzę taką zgrozę w przerwaniu to uważam, że warto naprowadzić że jest źle a nawet bardzo źle

----------------------

A teraz na temat - w przerwaniu badaj wystąpienie zamiast znaku CR to wystąpienia trzech twoich znaków końca linii i to tyle - wystarczy wtedy jeden z nich zamienić na CR a dwa zignorować tak jak znaki LF i nawet nie musisz zmieniać w żadnej dalszej części biblioteki

Statystyki: Napisane przez mirekk36 — 13 gru 2018, o 10:02

mK_multi_uart i Nextion

2018-12-13T09:44:16+01:00

Zakupiłem tą bibliotekę z myślą o wykorzystaniu w projekcie gdzie mam atnelowskie wifi i wyświetlacz Nextiona.
Nextion nie korzysta ze znaku końca linii a z trzech 0xff, wcześniej przerobiłem bibliotekę z książek mirka i w sumie to działało.
Mój problem jest taki że mk_multi_uart jeszcze do nie do końca ogarniam i nie mam pomysłu na modyfikację.
Jeśli któryś z kolegów ma jakiś pomysł byłbym wdzięczny za podzielenie się wskazówkami.
Poniżej zmodyfikowany kod którego obecnie używam,:
[syntax=c]void UART2_RX_STR_EVENT(char * rbuf) {
if( hex_line ) {
uart_get_cmd( rbuf );
int i=*(rbuf);
if (i==0x65) {
if( uart2_rx_touch_event_callback ) {
(*uart2_rx_touch_event_callback)( rbuf );
} else {
UART2_RxHead = UART2_RxTail;
}
} else if(i==0x70){
if( uart2_rx_str_event_callback ) {
(*uart2_rx_str_event_callback)( rbuf );
} else {
UART2_RxHead = UART2_RxTail;
}
} else {
UART2_RxHead = UART2_RxTail;
}
}
}

// pobieranie komend
char * uart_get_cmd(char * buf) {
int c;
uint8_t cnt_ff=0;
char * wsk = buf;
if( hex_line ) {
while(1) {
c = uart2_getc();
*buf++ = c;
if (c==0xff){
cnt_ff++;
if(cnt_ff==3) break;
}
else {
cnt_ff=0;
}
}
*buf=0; // czy to napewno jest potrzebne - dodoawanie 0 na koncu?
hex_line--;
}
return wsk;
}[/syntax]

Czyżby wystarczyło lekko zmodyfikować ISR (USART0) ?

Statystyki: Napisane przez jaca_76 — 13 gru 2018, o 09:44