Author: Awais Masood <[hidden email]>
AuthorDate: Fri Oct 6 18:01:50 2017 +0200
Commit: Jan Beulich <[hidden email]>
CommitDate: Fri Oct 6 18:01:50 2017 +0200
ns16550: fix ISR lockup on Allwinner uart
This patch fixes an ISR lockup seen on Allwinner uart
On Allwinner H5, serial driver goes into an infinite loop
when interrupts are enabled. The reason is a residual
"busy detect" interrupt. Since the condition UART_IIR_NOINT
will not be true unless this interrupt is cleared, the
interrupt handler will remain locked up in this while loop.
A HW quirk fix was previously added for designware uart under
It checks for a busy condition during setup and clears the
condition by reading UART_USR register.
On Allwinner hardware, the "busy detect" condition occurs
later because an LCR write is performed during setup 'after'
this clear and if uart is busy, the "busy detect" condition
will trigger again and cause the ISR lockup.
To solve this problem, the same UART_USR read operation needs
to be performed within the interrupt handler to clear this
Linux dw 8250 driver also handles this condition within
Tested on Orange Pi PC2 (H5). This issue is seen on H3
as well and the same fix works.
Signed-off-by: Awais Masood <[hidden email]>
Acked-by: Jan Beulich <[hidden email]>
xen/drivers/char/ns16550.c | 38 +++++++++++++++++++++++++++++---------
1 file changed, 29 insertions(+), 9 deletions(-)
diff --git a/xen/drivers/char/ns16550.c b/xen/drivers/char/ns16550.c
index 6ab5ec3..e0f8199 100644
@@ -505,6 +505,23 @@ static int ns16550_ioport_invalid(struct ns16550 *uart)
return ns_read_reg(uart, UART_IER) == 0xff;
+static void handle_dw_usr_busy_quirk(struct ns16550 *uart)
+ if ( uart->dw_usr_bsy &&
+ (ns_read_reg(uart, UART_IIR) & UART_IIR_BSY) == UART_IIR_BSY )
+ /* DesignWare 8250 detects if LCR is written while the UART is
+ * busy and raises a "busy detect" interrupt. Read the UART
+ * Status Register to clear this state.
+ * Allwinner/sunxi UART hardware is similar to DesignWare 8250
+ * and also contains a "busy detect" interrupt. So this quirk
+ * fix will also be used for Allwinner UART.
+ ns_read_reg(uart, UART_USR);
static void ns16550_interrupt(
int irq, void *dev_id, struct cpu_user_regs *regs)
@@ -521,6 +538,16 @@ static void ns16550_interrupt(
if ( lsr & UART_LSR_DR )
+ /* A "busy-detect" condition is observed on Allwinner/sunxi UART
+ * after LCR is written during setup. It needs to be cleared at
+ * this point or UART_IIR_NOINT will never be set and this loop
+ * will continue forever.
+ * This state can be cleared by calling the dw_usr_busy quirk
+ * handler that resolves "busy-detect" for DesignWare uart.
@@ -623,15 +650,8 @@ static void ns16550_setup_preirq(struct ns16550 *uart)
/* No interrupts. */
ns_write_reg(uart, UART_IER, 0);
- if ( uart->dw_usr_bsy &&
- (ns_read_reg(uart, UART_IIR) & UART_IIR_BSY) == UART_IIR_BSY )
- /* DesignWare 8250 detects if LCR is written while the UART is
- * busy and raises a "busy detect" interrupt. Read the UART
- * Status Register to clear this state.
- ns_read_reg(uart, UART_USR);
+ /* Handle the DesignWare 8250 'busy-detect' quirk. */
/* Line control and baud-rate generator. */
ns_write_reg(uart, UART_LCR, lcr | UART_LCR_DLAB);
generated by git-patchbot for /home/xen/git/xen.git#master
Xen-changelog mailing list
|Free forum by Nabble||Edit this page|