Język C dla Atmeli: ograniczenie ilości wpisów w tablicy const char

Dostosowuję program przykładowy z książki "AVR Praktyczne projekty" dotyczący USART-DMA. Jest tam całkiem fajna funkcja która zamienia napis na numer indeksu tego napisu w tablicy.

Nie mogę jednak zrozumieć w czym tkwi ograniczenie polegające na niemożności zwiększenia ilości wpisów w tabeli.

Fragment oryginalnego kodu:


Komendy mogą być maksymalnie 10 znakowe (jeśli dobrze rozumiem), chociaż w moim przypadku raczej powinny być do 5 znaków każda (od 3 do 5) ewentualnie ograniczę się do takich samych długości np. wszystkie 3 znakowe.

Dlaczego jeśli dopiszę kolejną komendę np. to program głupieje i nie widzi już żadnego z elementów tablicy?

Potrzebuję sterować zachowaniem urządzenia z PC na zasadzie przekazywania prostych instrukcji lub lepiej poprzez prostego bash'a jak w linuxie. Chciał bym przesłać komendę np. "komenda arg1 arg2" lub "Komenda" i poniżej argument1, a jeszcze niżej argument2 (druga opcja jak w zwykłym C przy użyciu scanf) tyle, że nie bardzo wiem jak to zrobić optymalnie. W językach wyższego poziomu zapewne pobrał bym tekst i podzielił go po spacji uzyskując 3 teksty: komenda, arg1, arg2, niestety taki zapis jest zapewne nieoptymalny w przypadku mikrokontrolerów.

Cały sens stosowania TranslateCommand jest taki by móc zastosować instrukcję switch i przełączać zadania w zależności od przesłanej komendy. Komend jest ok. 10 i poza jedną wszystkie są bezargumentowe. Zakładam, że błąd który popełniam jest dość trywialny, ale nie mogę sobie z tym poradzić. Ciężko mi przejść z języków wysokiego poziomu na C i to jeszcze tragiczne C dla dla Atmeli (w porównaniu do mikroC ten kompilator jest mało przyjemny).

Przy okazji zapytam jeszcze: Czy można używać dwóch portów UART/USART w trybie DMA? Przykładowo USARTC0 i USARTF0 (xmega128A1)? Aktualnie USARTC0 używam z DMA, natomiast USARTF0 bez DMA. Drugi port nie pobiera informacji i nie wyzwala przerwań (pracuje tam tylko Tx).

kspro napisał:

zamiast cmd3[ ] dopisałeś omyłkowo po raz drugi cmd2[ ]

Już poprawiłem wpis w powyższym poście.

Wstawiałem 8 wpisów od cmd1 do cmd8, wypełniałem tablicę normalnie czyli wpisywałem treść bezpośrednio do cmds i wszystko kompilowało się prawidłowo. Problem w tym, że po uruchomieniu programu program już nie widział komend które wpisałem. Zmieniałem wpisy i przeglądałem kod kilka razy, ale nie znalazłem błędu. Tak jakby w pozostałej części kodu Tomasza Francuza (tmf na elektroda.pl) był jakiś wpis określający maksymalną ilość wpisów w cmds. Naprawdę nie rozumiem czemu tak się dzieje.

Tak to jest jak się używa nie swojego kodu do własnych zastosowań.
Jak wpiszę tak: to też nie działa. Pisałem analogiczny kod na DevC++ i tam oczywiście wszystko działa (inne zmienne itp.) może w tym PROGMEM jest coś nie tak?

Może zamiast const char powinno być prog_char?

vonar napisał:

teoretycznie powinno być wszystko w porządku. Bez pełnych źródeł trudno powiedzieć, w czym tkwi problem.

Pełne źródła to AVR Praktyczne projekty - Materiały dodatkowe, przykłady. (źródło linka) oraz zmodyfikowany plik usart-DMA.c:


Naprawdę nie rozumiem czemu to nie działa... cały czas TranslateCommand() zwraca indeks 8. Jak zredukuję wpis cmds do tego:

To zawsze działa, niezależnie od tego co skopię w pozostałym kodzie. Nie rozumiem gdzie jest ograniczenie cmds? Zmianiałem MAX_CMD_LEN na długość wszystkich przekazywanych znaków (nielogiczne, ale wolałem to sprawdzić) i to nic nie pomaga. Coś jest źle napisane w TranslateCommand ale ja błędu nie widzę.

Dlaczego kod przykładowy z książki może nie działać po tak drobnej modyfikacji?

Czy ktoś może podać inne rozwiązanie tego problemu? Czyli tak jak pisałem: Jak przekażę jedną z komend przez RS232 to chcę by został włączony odpowiedni program. Nie wiem tylko czy komenda STOP ma prawo działać w takim układzie programu. Ewentualnie jak użyć przerwania od USART? Tak by dało się tam sprawdzić czy wywołana komenda to STOP?
Właściwie to w tym przypadku lepszym układem programu było by użycie systemu czasu rzeczywistego, ale zupełnie nie mam pojęcia jak to zrobić w Atmelach i skąd taki system wziąć.

BlueDraco napisał:

Spróbuj wyświetlić tekst polecenia przed porównaniem go z tablicą nazw poleceń.

Niestety, ale właśnie to robię i to nawet w kodzie który zamieściłem fprintf wysyła tekst na jeden z dwóch portów USART z których korzystam. Tekst wyświetla się właściwie, a tablica nie zwraca indeksu czyli strcmp_P nigdy nie jest 0. W samej pętli z porównaniem też wyświetlałem już tekst pobrany i ten z tablicy i zawsze jak mam więcej niż dwa wpisy w tablicy to dostaję pustkę...

Raczej odpuszczę sobie ten sposób działania w celu obsługi switch'a. W VB.NET w którym programuję na codzień nie ma problemów z odpowiednikiem switch'a (select case) ale to jest język wysokiego poziomu więc nie ma oczywiście porównania z C.

Jak użyć odwrotnej notacji polskiej do obsługi takiego tekstu:

Z tego co pamiętam ze studiów ONP używało się do rozwiązywania równań w odpowiedniej kolejności. W tym przypadku pewnie prościej było by sprawdzić czy początek tekstu to PRINT lub prościej czy pierszy znak to P i jeśli tak to podzielić jakoś string na 3 mniejsze stringi. W VB napisał bym split, ale w C już bardzo dawno nie robiłem takich rzeczy więc nie bardzo wiem jak to zrobić. Stąd wpadłem na pomysł by użyć zapisu stosowanego w AutoCADzie gdzie wprowadzamy komendę po czym w następnych linijkach wprowadzamy jej argumenty np. współrzędne kolejnych punktów jakiejś figury.

W gruncie rzeczy zapis jest dowolny.

Start ma startować obsługę enkodera i wysyłać co 1mm kreskę na usart1. Natomiast po odliczeniu przekazanej przez "print" odległości ma wysłać komendę na usart2. Problem w tym, że nie wiem jak wymusić by po wpisaniu stop zliczanie zostało zatrzymane. Pewnie będzie potrzebna obsługa przerwania od USART.

michalko12 napisał:

Czyściłeś cały projekt?

Już chciałem napisać, że to genialna i jakże prosta odpowiedź, ale niestety to też nie pomogło ;(

Spróbowałem czegoś innego. Po następującej zmianie:

Wszystko ładnie działa. Pytanie tylko co właściwie się teraz dzieje i gdzie znajdują się wpisy. "cmd?" jest typu PROGMEM, ale już cmds jest tylko zwykłą tablicą stałych const char* pytanie więc brzmi czy cmd? znajdują się w RAMie czy też w pamięci programu. Czy też wystąpił najgorszy przypadek czyli znajdują się zarówno w RAMie jak i w pamięci programu?

--
hmm...

Data Memory Usage: 167 bytes 2% Full (przed zmianą)
Data Memory Usage: 183 bytes 2% Full (po zmianie)

Program Memory Usage: bez zmian.

Czyli dane znajdują się w RAMie, ale zajmują tak straszliwie kosmiczną przestrzeń, że chyba je ta zostawię
Ciekawe dlaczego to składowanie tablicy w pamięci programu nie działa właściwie. Może chodzi o to, że to jest jakby tablica znaków w tablicy znaków?

vonar napisał:

(PGM_P)pgm_read_word(cmds[indeks]))

To nic nie daje. No ale przeżyję te 0,2% RAM bez PROGRAMEM.

To itak sporo mniej niż:

Tomasz Francuz raczej przemyślał dość dobrze swoje programy, ale widocznie sam kompilator od AtmelStudio6 ma jakieś nieokreślone bug'i związane z PROGRAMEM i tablicą typu PROGRAMEM. Skoro działa to z dwoma elementami, a z większą ilością już nie to pewnie jest to jak święty granat ręczny z Monty Pytona (liczyć do trzech, ale nie do dwóch chyba, że licząc do trzech).

Zagadka zapewne pozostanie niewyjaśniona, no ale chociaż coś się wyjaśniło. Przypadkiem bo przypadkiem, ale zawsze to coś.

vonar napisał:

zapomniałem "&"

Fakt. Teraz wszystko działa jak należy.

Oj ten Atembler (miks słów asembler i Atmel) jest bardzo fikuśny. Grunt, że działa.
Temat rozwiązany przez vonar.

Dzięki wielkie!