The default serial and rtc IO region overlaps with PCI IO bar
region leading bar 0 activation to fail. Moving these devices
to MMIO region similar to ARM.
Given serial has been moved from 0x3f8 to 0x10000000, this
requires us to now pass earlycon=uart8250,mmio,0x10000000
from cmdline rather than earlycon=uart8250,mmio,0x3f8.
To avoid the need to change the address every time the tool
is updated, we can also just pass "earlycon" from cmdline
and guest then finds the type and base address by following
the Device Tree's stdout-path property.
Signed-off-by: Rajnesh Kanwal <rkanwal@xxxxxxxxxxxx>
Tested-by: Atish Patra <atishp@xxxxxxxxxxxx>
Reviewed-by: Atish Patra <atishp@xxxxxxxxxxxx>
---
v4: Incorporated feedback from Alexandru Elisei. Mainly rebased the
change on main with the IRQ changes which haven't landed yet. Also
a fix in KVM_VIRTIO_MMIO_AREA macro.
v3: https://lore.kernel.org/all/20230202191301.588804-1-rkanwal@xxxxxxxxxxxx/
Incorporated feedback from Andre Przywara and Alexandru Elisei.
Mainly updated the commit message to specify that we can simply pass
just "earlycon" from cmdline and avoid the need to specify uart address.
Also added Tested-by and Reviewed-by tags by Atish Patra.
v2: https://lore.kernel.org/all/20230201160137.486622-1-rkanwal@xxxxxxxxxxxx/
Added further details in the commit message regarding the
UART address change required in kernel cmdline parameter.
v1: https://lore.kernel.org/all/20230124155251.1417682-1-rkanwal@xxxxxxxxxxxx/
hw/rtc.c | 3 +++
hw/serial.c | 4 ++++
riscv/include/kvm/kvm-arch.h | 8 +++++++-
3 files changed, 14 insertions(+), 1 deletion(-)
diff --git a/hw/rtc.c b/hw/rtc.c
index 9b8785a..da696e1 100644
--- a/hw/rtc.c
+++ b/hw/rtc.c
@@ -9,6 +9,9 @@
#if defined(CONFIG_ARM) || defined(CONFIG_ARM64)
#define RTC_BUS_TYPE DEVICE_BUS_MMIO
#define RTC_BASE_ADDRESS ARM_RTC_MMIO_BASE
+#elif defined(CONFIG_RISCV)
+#define RTC_BUS_TYPE DEVICE_BUS_MMIO
+#define RTC_BASE_ADDRESS RISCV_RTC_MMIO_BASE
#else
/* PORT 0070-007F - CMOS RAM/RTC (REAL TIME CLOCK) */
#define RTC_BUS_TYPE DEVICE_BUS_IOPORT
diff --git a/hw/serial.c b/hw/serial.c
index 3d53362..b6263a0 100644
--- a/hw/serial.c
+++ b/hw/serial.c
@@ -17,6 +17,10 @@
#define serial_iobase(nr) (ARM_UART_MMIO_BASE + (nr) * 0x1000)
#define serial_irq(nr) (32 + (nr))
#define SERIAL8250_BUS_TYPE DEVICE_BUS_MMIO
+#elif defined(CONFIG_RISCV)
+#define serial_iobase(nr) (RISCV_UART_MMIO_BASE + (nr) * 0x1000)
+#define serial_irq(nr) (1 + (nr))
+#define SERIAL8250_BUS_TYPE DEVICE_BUS_MMIO
#else
#define serial_iobase_0 (KVM_IOPORT_AREA + 0x3f8)
#define serial_iobase_1 (KVM_IOPORT_AREA + 0x2f8)
diff --git a/riscv/include/kvm/kvm-arch.h b/riscv/include/kvm/kvm-arch.h
index 2cf41c5..3e7dd3e 100644
--- a/riscv/include/kvm/kvm-arch.h
+++ b/riscv/include/kvm/kvm-arch.h
@@ -35,10 +35,16 @@
#define RISCV_MAX_MEMORY(kvm) RISCV_LOMAP_MAX_MEMORY
#endif
+#define RISCV_UART_MMIO_BASE RISCV_MMIO
+#define RISCV_UART_MMIO_SIZE 0x10000
+
+#define RISCV_RTC_MMIO_BASE (RISCV_UART_MMIO_BASE + RISCV_UART_MMIO_SIZE)
+#define RISCV_RTC_MMIO_SIZE 0x10000
+
#define KVM_IOPORT_AREA RISCV_IOPORT
#define KVM_PCI_CFG_AREA RISCV_PCI
#define KVM_PCI_MMIO_AREA (KVM_PCI_CFG_AREA + RISCV_PCI_CFG_SIZE)
-#define KVM_VIRTIO_MMIO_AREA RISCV_MMIO
+#define KVM_VIRTIO_MMIO_AREA (RISCV_RTC_MMIO_BASE + RISCV_RTC_MMIO_SIZE)
#define KVM_IOEVENTFD_HAS_PIO 0
--
2.25.1
