summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAleksandar Mitev2015-09-18 13:08:28 +0200
committerGreg Kroah-Hartman2015-10-04 18:36:10 +0200
commit9ab76556608665c35080f641c73025297d0a2813 (patch)
tree462a73c489f6a4bd0a997b28cf1e386cb62cf917
parentserial: sh-sci: Stop calling sci_start_rx() from sci_request_dma() (diff)
downloadkernel-qcow2-linux-9ab76556608665c35080f641c73025297d0a2813.tar.gz
kernel-qcow2-linux-9ab76556608665c35080f641c73025297d0a2813.tar.xz
kernel-qcow2-linux-9ab76556608665c35080f641c73025297d0a2813.zip
serial: sh-sci: Remove timer on shutdown of port
This prevents DMA timer timeout that can trigger after the port has been closed. Signed-off-by: Aleksandar Mitev <amitev@visteon.com> [geert: Move del_timer_sync() outside spinlock to avoid circular locking dependency between rx_timer_fn() and del_timer_sync()] Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/tty/serial/sh-sci.c8
1 files changed, 8 insertions, 0 deletions
diff --git a/drivers/tty/serial/sh-sci.c b/drivers/tty/serial/sh-sci.c
index ac9ce8f1ff79..36a11110acf4 100644
--- a/drivers/tty/serial/sh-sci.c
+++ b/drivers/tty/serial/sh-sci.c
@@ -1810,6 +1810,14 @@ static void sci_shutdown(struct uart_port *port)
sci_stop_tx(port);
spin_unlock_irqrestore(&port->lock, flags);
+#ifdef CONFIG_SERIAL_SH_SCI_DMA
+ if (s->chan_rx) {
+ dev_dbg(port->dev, "%s(%d) deleting rx_timer\n", __func__,
+ port->line);
+ del_timer_sync(&s->rx_timer);
+ }
+#endif
+
sci_free_dma(port);
sci_free_irq(s);
}