ARM: meson: serial: only disable tx irq on stop
authorBen Dooks <ben.dooks@codethink.co.uk>
Wed, 18 Nov 2015 14:41:15 +0000 (14:41 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 14 Dec 2015 03:59:48 +0000 (19:59 -0800)
Since disabling the transmit state machine still allows characters to
be transmitted when written to the UART write FIFO, simply disable the
transmit interrupt when the UART port is stopped.

This has not shown an improvement with the console issues when running
systemd, but seems like it should be done.

Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
Reported-by: Edward Cragg <ed.cragg@codethink.co.uk>
Tested-by: Carlo Caione <carlo@endlessm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/meson_uart.c

index c7bad2b..9327efd 100644 (file)
@@ -110,7 +110,7 @@ static void meson_uart_stop_tx(struct uart_port *port)
        u32 val;
 
        val = readl(port->membase + AML_UART_CONTROL);
-       val &= ~AML_UART_TX_EN;
+       val &= ~AML_UART_TX_INT_EN;
        writel(val, port->membase + AML_UART_CONTROL);
 }
 
@@ -133,7 +133,7 @@ static void meson_uart_shutdown(struct uart_port *port)
        spin_lock_irqsave(&port->lock, flags);
 
        val = readl(port->membase + AML_UART_CONTROL);
-       val &= ~(AML_UART_RX_EN | AML_UART_TX_EN);
+       val &= ~AML_UART_RX_EN;
        val &= ~(AML_UART_RX_INT_EN | AML_UART_TX_INT_EN);
        writel(val, port->membase + AML_UART_CONTROL);