altera_uart: Don't take spinlock in already protected functions
authorTobias Klauser <tklauser@distanz.ch>
Tue, 25 May 2010 14:59:55 +0000 (16:59 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 4 Jun 2010 20:37:17 +0000 (13:37 -0700)
commitd8d721f4c005f9a69bd1b5d5c6ba99b7e1d464de
tree4ff679f2f17605139bdf209179519a03ff911566
parent328be395a396b1333b56e04571365dc614c96e46
altera_uart: Don't take spinlock in already protected functions

Don't take the port spinlock in uart functions where the serial core
already takes care of locking/unlocking them.

The code would actually lock up on architectures where spinlocks are
implemented (not the case on nios2 where this driver is primarily used
for now, thus this bug didn't trigger).

Also protect calling altera_uart_rx_chars/altera_uart_tx_chars in the
interrupt handler by the port spinlock.

Thanks to Ian Abbott for pointing these issues out.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Thomas Chou <thomas@wytron.com.tw>
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/serial/altera_uart.c