From 9ca6799b0e78d74e390663a4b70a53dc9a11e2fb Mon Sep 17 00:00:00 2001 From: Joel Sherrill Date: Sun, 12 Oct 2014 10:07:07 -0500 Subject: powerpc/shared/console: Fix warnings --- c/src/lib/libbsp/powerpc/shared/console/uart.c | 35 ++++++++------------------ 1 file changed, 11 insertions(+), 24 deletions(-) (limited to 'c/src/lib/libbsp/powerpc/shared/console/uart.c') diff --git a/c/src/lib/libbsp/powerpc/shared/console/uart.c b/c/src/lib/libbsp/powerpc/shared/console/uart.c index 67816e38db..917fd5bf7c 100644 --- a/c/src/lib/libbsp/powerpc/shared/console/uart.c +++ b/c/src/lib/libbsp/powerpc/shared/console/uart.c @@ -82,8 +82,7 @@ uwrite(int uart, int reg, unsigned int val) } -#ifdef UARTDEBUG - static void +static void uartError(int uart, void *termiosPrivate) { unsigned char uartStatus, dummy; @@ -92,6 +91,7 @@ uartError(int uart, void *termiosPrivate) uartStatus = uread(uart, LSR); dummy = uread(uart, RBR); +#ifdef UARTDEBUG if (uartStatus & OE) printk("********* Over run Error **********\n"); if (uartStatus & PE) @@ -100,32 +100,18 @@ uartError(int uart, void *termiosPrivate) printk("********* Framing Error **********\n"); if (uartStatus & BI) { printk("********* BREAK INTERRUPT *********\n"); - if ((h=uart_data[uart].breakCallback.handler)) - h(uart, - (dummy<<8)|uartStatus, - termiosPrivate, - uart_data[uart].breakCallback.private); - +#endif + if ((h=uart_data[uart].breakCallback.handler)) { + h(uart, + (dummy<<8)|uartStatus, + termiosPrivate, + uart_data[uart].breakCallback.private); } +#ifdef UARTDEBUG if (uartStatus & ERFIFO) printk("********* Error receive Fifo **********\n"); - -} -#else -inline void uartError(int uart, void *termiosPrivate) -{ - unsigned char uartStatus,dummy; - BSP_UartBreakCbProc h; - - uartStatus = uread(uart, LSR); - dummy = uread(uart, RBR); - if ((uartStatus & BI) && (h=uart_data[uart].breakCallback.handler)) - h(uart, - (dummy<<8)|uartStatus, - termiosPrivate, - uart_data[uart].breakCallback.private); -} #endif +} /* * Uart initialization, it is hardcoded to 8 bit, no parity, @@ -197,6 +183,7 @@ BSP_uart_init(int uart, int baud, int hwFlow) tmp = uread(uart, LSR); tmp = uread(uart, RBR); tmp = uread(uart, MSR); + (void) tmp; /* avoid set but not used warning */ /* Remember state */ uart_data[uart].hwFlow = hwFlow; -- cgit v1.2.3