[PATCH 17/54] powerpc/shared/console: Fix warnings
Joel Sherrill
joel.sherrill at oarcorp.com
Mon Oct 13 15:37:01 UTC 2014
---
c/src/lib/libbsp/powerpc/shared/console/console.c | 4 +++
c/src/lib/libbsp/powerpc/shared/console/uart.c | 35 +++++++----------------
2 files changed, 15 insertions(+), 24 deletions(-)
diff --git a/c/src/lib/libbsp/powerpc/shared/console/console.c b/c/src/lib/libbsp/powerpc/shared/console/console.c
index 59b0804..b68e076 100644
--- a/c/src/lib/libbsp/powerpc/shared/console/console.c
+++ b/c/src/lib/libbsp/powerpc/shared/console/console.c
@@ -148,6 +148,7 @@ rtems_device_driver console_initialize(
return RTEMS_SUCCESSFUL;
} /* console_initialize */
+#if !defined(USE_POLLED_IO)
static int console_first_open(int major, int minor, void *arg)
{
rtems_status_code status;
@@ -174,12 +175,15 @@ static int console_first_open(int major, int minor, void *arg)
return 0;
}
+#endif
+#if !defined(USE_POLLED_IO)
static int console_last_close(int major, int minor, void *arg)
{
BSP_uart_remove_isr(minor, ttyS[minor].isr);
return 0;
}
+#endif
/*-------------------------------------------------------------------------+
| Console device driver OPEN entry point
diff --git a/c/src/lib/libbsp/powerpc/shared/console/uart.c b/c/src/lib/libbsp/powerpc/shared/console/uart.c
index 67816e3..917fd5b 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;
--
1.9.3
More information about the devel
mailing list