if you're using strlen() on it, as we are in tp_serial_send_cmd(), it had better be a plain char. make it so, killing one warning. no reason to sprintf() into an unsigned char buffer, so make it not, killing the other. this cleans all warnings from synaptic.c using gcc 3.7.2