From 816282fb8bd0298e224b5f89c82bae12ca628193 Mon Sep 17 00:00:00 2001 From: Hyeongsik Min Date: Wed, 12 Jul 2017 14:04:43 +0900 Subject: [PATCH] Fix log message format in uart.c - Remove unnecessay info in log message. - Remove unnecessay new line(\n). - Print error description string instead of showing errno. Change-Id: Id0dccd0434ee37a7d9aa278e2295ec224a6dba1c Signed-off-by: Hyeongsik Min --- src/interface/uart.c | 65 +++++++++++++++++++++++++++++++++++----------------- 1 file changed, 44 insertions(+), 21 deletions(-) diff --git a/src/interface/uart.c b/src/interface/uart.c index f070b8b..e262a1a 100644 --- a/src/interface/uart.c +++ b/src/interface/uart.c @@ -35,6 +35,7 @@ #ifndef ARRAY_SIZE #define ARRAY_SIZE(x) (sizeof(x)/sizeof((x)[0])) #endif +#define MAX_ERR_LEN 128 char *sysfs_uart_path[] = { "/dev/ttyS", @@ -63,9 +64,11 @@ int uart_open(int port, int *file_hndl) if (access(uart_dev, F_OK) == 0) break; } - + _D("uart_dev : %s", uart_dev); if ((fd = open(uart_dev, O_RDWR | O_NOCTTY | O_NONBLOCK)) < 0) { - _E("Error[%d]: can't open %s, %s--[%d]\n", errno, uart_dev, __FUNCTION__, __LINE__); + char errmsg[MAX_ERR_LEN]; + strerror_r(errno, errmsg, MAX_ERR_LEN); + _E("can't open %s, errmsg : %s", uart_dev, errmsg); return -ENXIO; } *file_hndl = fd; @@ -77,7 +80,7 @@ int uart_close(int file_hndl) _D("file_hndl : %d", file_hndl); if (!file_hndl) { - _E("Error[%d]: Invalid parameter, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + _E("Invalid NULL parameter"); return -EINVAL; } close(file_hndl); @@ -89,13 +92,15 @@ int uart_flush(int file_hndl) int ret; if (!file_hndl) { - _E("Error[%d]: Invalid parameter, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + _E("Invalid NULL parameter"); return -EINVAL; } ret = tcflush(file_hndl, TCIOFLUSH); if (ret < 0) { - _E("FAILED[%d]: tcflush, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + char errmsg[MAX_ERR_LEN]; + strerror_r(errno, errmsg, MAX_ERR_LEN); + _E("tcflush failed, errmsg : %s", errmsg); return -1; } @@ -111,18 +116,20 @@ int uart_set_baudrate(int file_hndl, uart_baudrate_e baud) memset(&tio, 0, sizeof(tio)); if (!file_hndl) { - _E("Error[%d]: Invalid parameter, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + _E("Invalid NULL parameter"); return -EINVAL; } if (baud > UART_BAUDRATE_230400) { - _E("Error[%d]: Invalid parameter, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + _E("Invalid parameter"); return -EINVAL; } ret = tcgetattr(file_hndl, &tio); if (ret) { - _E("Error[%d]: tcgetattr, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + char errmsg[MAX_ERR_LEN]; + strerror_r(errno, errmsg, MAX_ERR_LEN); + _E("tcgetattr failed, errmsg : %s", errmsg); return -1; } tio.c_cflag = peripheral_uart_br[baud]; @@ -135,7 +142,9 @@ int uart_set_baudrate(int file_hndl, uart_baudrate_e baud) uart_flush(file_hndl); ret = tcsetattr(file_hndl, TCSANOW, &tio); if (ret) { - _E("Error[%d]: tcsetattr, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + char errmsg[MAX_ERR_LEN]; + strerror_r(errno, errmsg, MAX_ERR_LEN); + _E("tcsetattr failed, errmsg: %s", errmsg); return -1; } @@ -150,19 +159,21 @@ int uart_set_mode(int file_hndl, uart_bytesize_e bytesize, uart_parity_e parity, _D("file_hndl : %d, bytesize : %d, parity : %d, stopbits : %d", file_hndl, bytesize, parity, stopbits); if (!file_hndl) { - _E("Error[%d]: Invalid parameter, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + _E("Invalid NULL parameter"); return -EINVAL; } ret = tcgetattr(file_hndl, &tio); if (ret) { - _E("Error[%d]: tcgetattr, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + char errmsg[MAX_ERR_LEN]; + strerror_r(errno, errmsg, MAX_ERR_LEN); + _E("tcgetattr failed, errmsg: %s", errmsg); return -1; } /* set byte size */ if (bytesize < UART_BYTESIZE_5BIT || bytesize > UART_BYTESIZE_8BIT) { - _E("Error[%d]: Invalid parameter bytesize, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + _E("Invalid parameter bytesize"); return -EINVAL; } tio.c_cflag &= ~CSIZE; @@ -195,14 +206,16 @@ int uart_set_mode(int file_hndl, uart_bytesize_e bytesize, uart_parity_e parity, tio.c_cflag |= CSTOPB; break; default: - _E("Error[%d]: Invalid parameter stopbits, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + _E("Invalid parameter stopbits"); return -EINVAL; } uart_flush(file_hndl); ret = tcsetattr(file_hndl, TCSANOW, &tio); if (ret) { - _E("Error[%d]: tcsetattr, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + char errmsg[MAX_ERR_LEN]; + strerror_r(errno, errmsg, MAX_ERR_LEN); + _E("tcsetattr failed, errmsg : %s", errmsg); return -1; } @@ -217,13 +230,15 @@ int uart_set_flowcontrol(int file_hndl, bool xonxoff, bool rtscts) _D("file_hndl : %d, xonxoff : %d, rtscts : %d", file_hndl, xonxoff, rtscts); if (!file_hndl) { - _E("Error[%d]: Invalid parameter, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + _E("Invalid NULL parameter"); return -EINVAL; } ret = tcgetattr(file_hndl, &tio); if (ret) { - _E("Error[%d]: tcgetattr, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + char errmsg[MAX_ERR_LEN]; + strerror_r(errno, errmsg, MAX_ERR_LEN); + _E("tcgetattr failed, errmsg : %s", errmsg); return -1; } @@ -241,7 +256,9 @@ int uart_set_flowcontrol(int file_hndl, bool xonxoff, bool rtscts) ret = tcsetattr(file_hndl, TCSANOW, &tio); if (ret) { - _E("Error[%d]: tcsetattr, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + char errmsg[MAX_ERR_LEN]; + strerror_r(errno, errmsg, MAX_ERR_LEN); + _E("tcsetattr failed, errmsg : %s", errmsg); return -1; } @@ -251,14 +268,17 @@ int uart_set_flowcontrol(int file_hndl, bool xonxoff, bool rtscts) int uart_read(int file_hndl, uint8_t *buf, unsigned int length) { int ret; + if (!file_hndl) { - _E("Error[%d]: Invalid parameter, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + _E("Invalid NULL parameter"); return -EINVAL; } ret = read(file_hndl, (void *)buf, length); if ((errno != EAGAIN && errno != EINTR) && ret < 0) { - _E("Error[%d]: read, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + char errmsg[MAX_ERR_LEN]; + strerror_r(errno, errmsg, MAX_ERR_LEN); + _E("read failed, errmsg : %s", errmsg); return -EIO; } @@ -268,14 +288,17 @@ int uart_read(int file_hndl, uint8_t *buf, unsigned int length) int uart_write(int file_hndl, uint8_t *buf, unsigned int length) { int ret; + if (!file_hndl) { - _E("Error[%d]: Invalid parameter, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + _E("Invalid NULL parameter"); return -EINVAL; } ret = write(file_hndl, buf, length); if (ret < 0) { - _E("Error[%d]: write, %s--[%d]\n", errno, __FUNCTION__, __LINE__); + char errmsg[MAX_ERR_LEN]; + strerror_r(errno, errmsg, MAX_ERR_LEN); + _E("write failed, errmsg : %s", errmsg); return -EIO; } -- 2.7.4