diff options
| author | Govindraj.R <govindraj.raja@ti.com> | 2011-12-14 21:24:11 +0530 |
|---|---|---|
| committer | Kevin Hilman <khilman@ti.com> | 2011-12-14 16:05:27 -0800 |
| commit | da27468655540b083525177f5dc6f3b1f6e3b869 (patch) | |
| tree | 7ec8e5044d48cc33cc7126a36021bd3fe4b57d90 /drivers/tty | |
| parent | 2fd149645eb46d26130d7070c6de037dddf34880 (diff) | |
| download | blackbird-op-linux-da27468655540b083525177f5dc6f3b1f6e3b869.tar.gz blackbird-op-linux-da27468655540b083525177f5dc6f3b1f6e3b869.zip | |
ARM: OMAP2+: UART: Fix compilation/sparse warnings
Fixes below compilation warning.
drivers/tty/serial/omap-serial.c: In function 'serial_omap_irq':
drivers/tty/serial/omap-serial.c:228:29: warning: 'ch' may be used uninitialized in this function [-Wuninitialized]
Fix below sparse warning.
drivers/tty/serial/omap-serial.c:392:52: warning: incorrect type in argument 2 (different signedness)
drivers/tty/serial/omap-serial.c:392:52: expected int *status
drivers/tty/serial/omap-serial.c:392:52: got unsigned int *<noident>
Reported-by: Kevin Hilman <khilman@ti.com>
Signed-off-by: Govindraj.R <govindraj.raja@ti.com>
Acked-by: Greg Kroah-Hartman <gregkh@suse.de> (for drivers/tty changes)
Signed-off-by: Kevin Hilman <khilman@ti.com>
Diffstat (limited to 'drivers/tty')
| -rw-r--r-- | drivers/tty/serial/omap-serial.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/tty/serial/omap-serial.c b/drivers/tty/serial/omap-serial.c index f3ff0ca377c5..7b0303d34021 100644 --- a/drivers/tty/serial/omap-serial.c +++ b/drivers/tty/serial/omap-serial.c @@ -166,11 +166,12 @@ static void serial_omap_stop_rx(struct uart_port *port) pm_runtime_put_autosuspend(&up->pdev->dev); } -static inline void receive_chars(struct uart_omap_port *up, int *status) +static inline void receive_chars(struct uart_omap_port *up, + unsigned int *status) { struct tty_struct *tty = up->port.state->port.tty; - unsigned int flag; - unsigned char ch, lsr = *status; + unsigned int flag, lsr = *status; + unsigned char ch = 0; int max_count = 256; do { |
