On 5/4/22 04:25, Mark Cave-Ayland wrote:
Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk>
Acked-by: Helge Deller <del...@gmx.de>
---
  hw/hppa/lasi.c    | 8 --------
  hw/hppa/machine.c | 7 +++++++
  2 files changed, 7 insertions(+), 8 deletions(-)


In that it is code movement,
Reviewed-by: Richard Henderson <richard.hender...@linaro.org>

+    if (serial_hd(1)) {
+        /* Serial port */
+        serial_mm_init(addr_space, LASI_UART_HPA + 0x800, 0,
+                qdev_get_gpio_in(lasi_dev, LASI_IRQ_UART_HPA), 8000000 / 16,
+                serial_hd(1), DEVICE_BIG_ENDIAN);
+    }

Although I believe there's a bug: both of these serial ports are being registered at LASI_UART_HPA + 0x800. I suspect this bug was hidden by the bug fixed in patch 31, in that serial_hd(0) should be at 0x800, and serial_hd(1) should be elsewhere.


r~

Reply via email to