ATNEL tech-forum 2014-05-05T20:24:26+01:00 https://forum.atnel.pl/feed.php?f=8&t=6929&mode 2014-05-05T20:24:26+01:00 2014-05-05T20:24:26+01:00 https://forum.atnel.pl/viewtopic.php?t=6929&p=80226#p80226 <![CDATA[Re: L21_UART_ASCII_AT problemy]]>
Główna zmiana to dodanie kodu
[syntax="c"]len_cmd = strlen_P(polecenia_at[i].polecenie_at);
if ( len_buf == len_cmd ){[/syntax]
którego zadanie jest porównanie wprowadzonej komendy z wzorcem co do długości.
Nawet jeśli komendy są bardzo podobne
mirekk36 napisał(a):

i dobrze ale też może być kłopot jeśli będą np dwa takie polecenia

AT+LED
AT+LEM

to i tak wybierze właściwą to którą wprowadzimy.

Tutaj kod całej procedury z moimi modyfikacjami,
[syntax="c"]void parse_uart_data( char * pBuf ) {

int8_t (*_at_srv)(uint8_t inout, char * data);

char * cmd_wsk;
char * reszta;
uint8_t i=0, len_buf, len_cmd;

if ( strpbrk(pBuf, "=?")){
// obsługa poleceń AT we/wy + parametry

if ( strpbrk(pBuf, "?")){
// zapytania do układu w postaci: AT+CMD?

cmd_wsk = strtok_r(pBuf, "?", &reszta);

// przypisanie dłogości
len_buf = strlen(cmd_wsk);
for(i=0;i<AT_CNT;i++) {
len_cmd = strlen_P(polecenia_at[i].polecenie_at);

// porównanie długości buforu z komędą
if ( len_buf == len_cmd ){
if ( len_buf && 0 == strncasecmp_P(cmd_wsk, polecenia_at[i].polecenie_at, len_cmd) ) {
if( pgm_read_word(polecenia_at[i].polecenie_at) ) { // <--- UWAGA! w tekście książki zabrakło pgm_read_word()
_at_srv = (void *)pgm_read_word( &polecenia_at[i].at_service );
if( _at_srv) {
if( _at_srv( 0, reszta ) < 0 ) {

// wyświetla błąd
parse_error ( pBuf );
}
}
}
uart_puts("\r\n");
break;
}
}
}

} else {
// ustawienia układu w postaci: AT+CMD=parametry

cmd_wsk = strtok_r(pBuf, "=", &reszta);
len_buf = strlen(cmd_wsk);
for(i=0;i<AT_CNT;i++) {
len_cmd = strlen_P(polecenia_at[i].polecenie_at);
if ( len_buf == len_cmd ){
if ( len_buf && 0 == strncasecmp_P(cmd_wsk, polecenia_at[i].polecenie_at, len_cmd) ) {
if( pgm_read_word(polecenia_at[i].polecenie_at) ) { // <--- UWAGA! w tekście książki zabrakło pgm_read_word()
_at_srv = (void *)pgm_read_word( &polecenia_at[i].at_service );
if( _at_srv && ! _at_srv( 1, reszta ) ) uart_puts("OK\r\n");
else {
parse_error ( pBuf );
}
}
uart_puts("\r\n");
break;
}
}
}
}

} else {
// obsługa poleceń AT bez parametrów

if( 0 == pBuf[0] ) uart_puts("\r\n");// reakcja na znak CR lub CRLF z terminala
else {
len_buf = strlen(pBuf);
for(i=0;i<AT_CNT;i++) {
len_cmd = strlen_P(polecenia_at[i].polecenie_at);
if ( len_buf == len_cmd ){
if ( 0 == strncasecmp_P(pBuf, polecenia_at[i].polecenie_at, len_cmd) ) {
if( pgm_read_word(polecenia_at[i].polecenie_at) ) { // <--- UWAGA! w tekście książki zabrakło pgm_read_word()
_at_srv = (void *)pgm_read_word( &polecenia_at[i].at_service );
if( _at_srv) _at_srv(2,0);
}
break;
}
}
}
}
}

if( AT_CNT == i ){
parse_error( pBuf );
uart_puts("\r\n");
}
}[/syntax]

Z drugiej strony jeśli chodzi o optymalizację w przypadku różne długości komendy to procedura nie będzie spowalniać, lecz nawet będzie szybciej wykonywana. Gdyż na początku sprawdzamy długość komendy i wzorca.

Statystyki: Napisane przez EmCom5 — 5 maja 2014, o 20:24

]]> 2014-04-30T08:31:57+01:00 2014-04-30T08:31:57+01:00 https://forum.atnel.pl/viewtopic.php?t=6929&p=79683#p79683 <![CDATA[Re: L21_UART_ASCII_AT problemy]]>
Chyba troszeczkę już przysypiałem.

Jeszcze nie testowałem w przypadku komend z parametrami, ale na to przyjdzie czas po "krótkim" weekendzie majowym.

Tak po za tym życzę słonecznego weekendy wszystkim forumowiczom. :)

Statystyki: Napisane przez EmCom5 — 30 kwi 2014, o 08:31

]]> 2014-04-29T21:32:58+01:00 2014-04-29T21:32:58+01:00 https://forum.atnel.pl/viewtopic.php?t=6929&p=79669#p79669 <![CDATA[Re: L21_UART_ASCII_AT problemy]]>

ty na szybko wymyśliłeś to:

[syntax=c]if ( strlen(pBuf) == strlen(polecenia_at[i].polecenie_at) )[/syntax]

i dobrze ale też może być kłopot jeśli będą np dwa takie polecenia

AT+LED
AT+LEM

prawda ? ;) ... bo w obydwu przypadkach zareaguje na AT+L ale tylko jedno z nich się wykona - to pierwsze .... trzeba nieco inaczej rozwiązać samo porównanie stringów i będzie po zawodach - a super optymalnością się nie przejmować bo to nie jakaś czasówka gdzie każda nanosekunda się liczy ;) ważne aby działało

Statystyki: Napisane przez mirekk36 — 29 kwi 2014, o 21:32

]]> 2014-04-29T21:26:50+01:00 2014-04-29T21:26:50+01:00 https://forum.atnel.pl/viewtopic.php?t=6929&p=79668#p79668 <![CDATA[L21_UART_ASCII_AT problemy]]>
Zająłem się obsługą uart-a z lekcji biblioteki od Mirka z zielonego Booka.

Mam mały problem nie chodzi o komunikacje po RS nie ma żadnego problemu.

Problem który minie nurtuje to to że gdy wysyłam komendę AT do układu:
wysłany ciąg znaków >> otrzymana odpowiedź
AT >> OK
A >> OK
AT+L >> AT+LED = (0-1)
AT+LED >> AT+LED = (0-1)
itd.

Jak widać mimo podania nie pełnej komendy otrzymujemy poprawną odpowiedź. Co zauważyłem w przypadku nie pełnej komendy to to że wybierana jest pierwsza z zgodnym ciągiem znaków. I nie ma losowości, jeśli podamy tylko A to odpowie jako AT, a jeśli podamy AT+ to zostanie zrestartowane urządzenie. Zgodnie z poniższą tablicą

Code:
      {"AT",       at_service},
      {"ATI",    ati_service},
      {"AT+RST",    at_rst_service},
      {"AT+IR",    at_ir_service},
      {"AT+LED",    at_led_service},
      {"AT+LCD",    at_lcd_service},


Jeśli pozmieniamy kolejność pozycji to wybierze pierwszą komendę pokrywając się.

Rozwiązanie jakie znalazłem żeby rozpoznawał pełne nazwy a nie po części.

Oryginalna część kodu z Blue Booka
Code:
      else {
         for(i=0;i<AT_CNT;i++) {
            if ( 0 == strncasecmp_P(pBuf, polecenia_at[i].polecenie_at, strlen(pBuf)) ) {
               if( pgm_read_word(polecenia_at[i].polecenie_at) ) { // <--- UWAGA! w tekście książki zabrakło pgm_read_word()
                  _at_srv = (void *)pgm_read_word( &polecenia_at[i].at_service );
                  if( _at_srv) _at_srv(2,0);
               }
               break;
            }
         }
      }


Po moich zmianach

Code:
      else {
         for(i=0;i<AT_CNT;i++) {
            if ( strlen(pBuf) == strlen(polecenia_at[i].polecenie_at) ){
               if ( 0 == strncasecmp_P(pBuf, polecenia_at[i].polecenie_at, strlen(pBuf)) ) {
                  if( pgm_read_word(polecenia_at[i].polecenie_at) ) { // <--- UWAGA! w tekście książki zabrakło pgm_read_word()
                     _at_srv = (void *)pgm_read_word( &polecenia_at[i].at_service );
                     if( _at_srv) _at_srv(2,0);
                  }
                  break;
               }
            }
         }
      }


Po pętli for wstawiłem dodatkowego if-a takie obejście robocze.

Ma ktoś pomysł jak to rozwiązać bardziej efektywniej.

Nie podaje pełnego kody gdyż nie jestem jej autorem. Autorem kodu jest Mirosław Kardaś.

Statystyki: Napisane przez EmCom5 — 29 kwi 2014, o 21:26

]]>