Administrace komentářů

Ahoj, jestli ti to nevadí, tak se zase pokusím tvůj kód trochu okomentovat. Především jsi se ptal na funkci wait(). Ta má asi počkat daný počet milisekund. Mohl jsi si ušetřit práci, protože již existuje funkce usleep(), která to umí (údaj požaduje v mikrosekundách). Její výhodou je, že zbytečně nezatěžuje systém.

Také vidím, že v kódu používáš "unsigned short". Pravděpodobně chceš datový typ o osmi bytech (bajt). Nejsem si jistý, jestli je vždy zárukou, že "unsigned short" bude mít vždy 8 bitů. Co třeba na 64bitové platformě? Doporučoval bych raději buď "unsigned char" nebo "uint8_t" (je definován v hlavičkovém souboru stdint.h).

Co se týče tvých funkcí RTS() a DTR() - není nutné v nich zjišťovat stav bitů. Pro tento účel je vhodnější použít volání TIOCMBIS/TIOCMBIC, které nastaví/nuluje příslušné bity. Například
ioctl(fd, TIOCMBIS, &TIOCM_DTR); /* nastaví DTR */
ioctl(fd, TIOCMBIC, &TIOCM_DTR); /* nuluje DTR */

Při selhání vypisuješ málo informací, ale to jsem psal už v předchozím příspěvku. Asi to tak chceš mít, co se dá dělat.

K čemu je ve funkci main() dobré to druhé volání wait(10) ? Zbytečně to zdržuje, no ne?

Mohl bys vysvětlit funkci power()? Odkud vlastně bereš napájení? Někde tam vypadlo "ioctl". K čemu je tam to TIOCSBRK?

Pak mě napadá ještě pár dalších prkotin, například že dělení ve funkci mereni() lze nahradit bitovým posunem, a v téže funkci by ses mohl úplně obejít bez proměnné i. O návratových hodnotách jsem psal už minule - teď opravdu skřípu zuby, když vidím, že funkce openserial() vrací nulu, když se něco pokazilo a jedničku, když je vše v pořádku. Proč to neudělat jak je zvykem? Ve funkci main() přetypováváš na int - to ale vždy zaokrouhlí směrem dolů, což se v této situaci nehodí.

Napiš prosím, jaký AD převodník používáš. Díky.