serial/amba-pl011: Unconditionally poll for FIFO space before each TX char
authorDave Martin <Dave.Martin@arm.com>
Thu, 21 May 2015 15:37:49 +0000 (16:37 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 24 May 2015 19:43:29 +0000 (12:43 -0700)
commit43dd1f9a5b05d6db2cb258354a01ace63baa5c0b
tree341f4a2ef7d1cbc0009c28bbe3d7b3ff80c6ac82
parente26081808edadfd257c6c9d81014e3b25e9a6118
serial/amba-pl011: Unconditionally poll for FIFO space before each TX char

Commit 734745caeb9f155ab58918834a8c70e83fa6afd3 serial/amba-pl011:
(Activate TX IRQ passively) introduces a race which causes the driver
sometimes to attempt to write a character to the TX FIFO when the FIFO
is already full.

The PL011 does not guarantee its behaviour when the FIFO is overfilled.
In practice, this can cause duplicate and/or dropped characters to be
output on the wire.  The problem is common enough to be readily
observable on the ARM Juno platform when the PL011 UART is used as
the console and DMA is not in use.

This patch fixes this problem by always polling for space before each
character is written to the FIFO.

This will be amended to a less brute-force approach in a later commit,
but this patch should help ensure correct behaviour for now.

Signed-off-by: Dave Martin <Dave.Martin@arm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/amba-pl011.c