Revert "serial: 8250: Don't service RX FIFO if interrupts are disabled"
authorOliver Barta <o.barta89@gmail.com>
Wed, 19 Jun 2019 08:16:39 +0000 (10:16 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 21 Jun 2019 09:07:34 +0000 (11:07 +0200)
commit3f2640ed7be838c3f05c0d2b0f7c7508e7431e48
tree9630fa6628ec325530a9395e4dd5ee60d26e7330
parent4a96895f74c9633b51427fd080ab70fa62b65bc4
Revert "serial: 8250: Don't service RX FIFO if interrupts are disabled"

This reverts commit 2e9fe539108320820016f78ca7704a7342788380.

Reading LSR unconditionally but processing the error flags only if
UART_IIR_RDI bit was set before in IIR may lead to a loss of transmission
error information on UARTs where the transmission error flags are cleared
by a read of LSR. Information are lost in case an error is detected right
before the read of LSR while processing e.g. an UART_IIR_THRI interrupt.

Signed-off-by: Oliver Barta <o.barta89@gmail.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Fixes: 2e9fe5391083 ("serial: 8250: Don't service RX FIFO if interrupts are disabled")
Cc: stable <stable@vger.kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/8250/8250_port.c