[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 2/6] pl011: reset the fifo when enabled or disabled
From: |
Peter Maydell |
Subject: |
[Qemu-devel] [PULL 2/6] pl011: reset the fifo when enabled or disabled |
Date: |
Wed, 19 Mar 2014 12:05:42 +0000 |
From: Rob Herring <address@hidden>
Intermittent issues have been seen where no serial input occurs. It
appears the pl011 gets in a state where the rx interrupt never fires
because the rx interrupt only asserts when crossing the fifo trigger
level. The fifo state appears to get out of sync when the pl011 is
re-configured. This combined with the rx timeout interrupt not being
modeled results in no more rx interrupts.
Disabling the fifo is the recommended way to clear the tx fifo in the
TRM (section 3.3.8). The behavior in this case for the rx fifo is
undefined in the TRM, but having fifo contents to be maintained during
configuration changes is not likely expected behavior. Reseting the
fifo state when the fifo size is changed is the simplest solution.
Signed-off-by: Rob Herring <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Message-id: address@hidden
Signed-off-by: Peter Maydell <address@hidden>
---
hw/char/pl011.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/hw/char/pl011.c b/hw/char/pl011.c
index a8ae6f4..8103e2e 100644
--- a/hw/char/pl011.c
+++ b/hw/char/pl011.c
@@ -162,6 +162,11 @@ static void pl011_write(void *opaque, hwaddr offset,
s->fbrd = value;
break;
case 11: /* UARTLCR_H */
+ /* Reset the FIFO state on FIFO enable or disable */
+ if ((s->lcr ^ value) & 0x10) {
+ s->read_count = 0;
+ s->read_pos = 0;
+ }
s->lcr = value;
pl011_set_read_trigger(s);
break;
--
1.9.0
- [Qemu-devel] [PULL 0/6] target-arm queue, Peter Maydell, 2014/03/19
- [Qemu-devel] [PULL 6/6] target-arm: A64: Add saturating accumulate ops (USQADD/SUQADD), Peter Maydell, 2014/03/19
- [Qemu-devel] [PULL 3/6] pl011: fix UARTRSR accesses corrupting the UARTCR value, Peter Maydell, 2014/03/19
- [Qemu-devel] [PULL 4/6] pl011: fix incorrect logic to set the RXFF flag, Peter Maydell, 2014/03/19
- [Qemu-devel] [PULL 2/6] pl011: reset the fifo when enabled or disabled,
Peter Maydell <=
- [Qemu-devel] [PULL 5/6] target-arm: A64: Add saturating int ops (SQNEG/SQABS), Peter Maydell, 2014/03/19
- [Qemu-devel] [PULL 1/6] ahci: fix sysbus support, Peter Maydell, 2014/03/19
- Re: [Qemu-devel] [PULL 0/6] target-arm queue, Peter Maydell, 2014/03/19