Commit ea48ce854a for qemu.org
commit ea48ce854a2a560937a10cdc4091a9b17809f3a0
Author: Daniel Henrique Barboza <daniel.barboza@oss.qualcomm.com>
Date: Fri Aug 28 17:39:14 2026 -0300
hw/riscv/device-common, virt.c: add riscv_create_platform_bus()
There will be code repetition instances added by the 'riscv-server-ref'
board w.r.t device creation and initialization. Instead of waiting for
that code to land we're adding helpers beforehand. The idea is to make
the new board code shorter and more palatable for reviewers.
A new 'device-common' file is created for such helpers. We'll start
with the platform bus, a nice piece of software that only the 'virt'
board uses so far, and the 'riscv-server-ref' will use to support TPM
devices.
Signed-off-by: Daniel Henrique Barboza <daniel.barboza@oss.qualcomm.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@oss.qualcomm.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Chao Liu <chao.liu@processmission.com>
Message-ID: <20260828203918.350131-13-daniel.barboza@oss.qualcomm.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
diff --git a/hw/riscv/device-common.c b/hw/riscv/device-common.c
new file mode 100644
index 0000000000..d3c68a9e71
--- /dev/null
+++ b/hw/riscv/device-common.c
@@ -0,0 +1,48 @@
+/*
+ * RISC-V board helpers for FDT generation.
+ *
+ * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
+ *
+ * SPDX-License-Identifier: GPL-2.0-or-later
+ */
+
+#include "qemu/osdep.h"
+
+#include "hw/riscv/device-common.h"
+
+#include "hw/core/platform-bus.h"
+#include "hw/core/qdev.h"
+#include "hw/core/qdev-properties.h"
+#include "hw/core/sysbus.h"
+#include "qapi/error.h"
+#include "system/address-spaces.h"
+#include "system/memory.h"
+
+
+DeviceState *riscv_create_platform_bus(DeviceState *irqchip,
+ const MemMapEntry *platform_bus_mem,
+ int platform_bus_base_irq,
+ int platform_bus_num_irqs)
+{
+ MemoryRegion *sysmem = get_system_memory();
+ DeviceState *dev;
+ SysBusDevice *sysbus;
+
+ dev = qdev_new(TYPE_PLATFORM_BUS_DEVICE);
+ dev->id = g_strdup(TYPE_PLATFORM_BUS_DEVICE);
+ qdev_prop_set_uint32(dev, "num_irqs", platform_bus_num_irqs);
+ qdev_prop_set_uint32(dev, "mmio_size", platform_bus_mem->size);
+ sysbus_realize_and_unref(SYS_BUS_DEVICE(dev), &error_fatal);
+
+ sysbus = SYS_BUS_DEVICE(dev);
+ for (int i = 0; i < platform_bus_num_irqs; i++) {
+ int irq = platform_bus_base_irq + i;
+ sysbus_connect_irq(sysbus, i, qdev_get_gpio_in(irqchip, irq));
+ }
+
+ memory_region_add_subregion(sysmem,
+ platform_bus_mem->base,
+ sysbus_mmio_get_region(sysbus, 0));
+
+ return dev;
+}
diff --git a/hw/riscv/meson.build b/hw/riscv/meson.build
index 0d82ceacc4..7aa3c1578d 100644
--- a/hw/riscv/meson.build
+++ b/hw/riscv/meson.build
@@ -1,6 +1,7 @@
riscv_ss = ss.source_set()
riscv_ss.add(files('aia.c'))
riscv_ss.add(files('boot.c'))
+riscv_ss.add(files('device-common.c'))
riscv_ss.add(files('fdt-common.c'))
riscv_ss.add(when: 'CONFIG_RISCV_NUMA', if_true: files('numa.c'))
riscv_ss.add(files('riscv_hart.c'))
diff --git a/hw/riscv/virt.c b/hw/riscv/virt.c
index 3a9d1e0a13..64f75646d0 100644
--- a/hw/riscv/virt.c
+++ b/hw/riscv/virt.c
@@ -35,6 +35,7 @@
#include "hw/riscv/riscv-iommu-bits.h"
#include "hw/riscv/virt.h"
#include "hw/riscv/boot.h"
+#include "hw/riscv/device-common.h"
#include "hw/riscv/fdt-common.h"
#include "hw/riscv/numa.h"
#include "kvm/kvm_riscv.h"
@@ -682,31 +683,6 @@ static DeviceState *virt_create_plic(const MemMapEntry *memmap, int socket,
memmap[VIRT_PLIC].size);
}
-static void create_platform_bus(RISCVVirtState *s, DeviceState *irqchip)
-{
- DeviceState *dev;
- SysBusDevice *sysbus;
- int i;
- MemoryRegion *sysmem = get_system_memory();
-
- dev = qdev_new(TYPE_PLATFORM_BUS_DEVICE);
- dev->id = g_strdup(TYPE_PLATFORM_BUS_DEVICE);
- qdev_prop_set_uint32(dev, "num_irqs", VIRT_PLATFORM_BUS_NUM_IRQS);
- qdev_prop_set_uint32(dev, "mmio_size", s->memmap[VIRT_PLATFORM_BUS].size);
- sysbus_realize_and_unref(SYS_BUS_DEVICE(dev), &error_fatal);
- s->platform_bus_dev = dev;
-
- sysbus = SYS_BUS_DEVICE(dev);
- for (i = 0; i < VIRT_PLATFORM_BUS_NUM_IRQS; i++) {
- int irq = VIRT_PLATFORM_BUS_IRQ + i;
- sysbus_connect_irq(sysbus, i, qdev_get_gpio_in(irqchip, irq));
- }
-
- memory_region_add_subregion(sysmem,
- s->memmap[VIRT_PLATFORM_BUS].base,
- sysbus_mmio_get_region(sysbus, 0));
-}
-
static void virt_build_smbios(RISCVVirtState *s)
{
MachineClass *mc = MACHINE_GET_CLASS(s);
@@ -1029,7 +1005,9 @@ static void virt_machine_init(MachineState *machine)
gpex_pcie_init(system_memory, pcie_irqchip, s);
- create_platform_bus(s, mmio_irqchip);
+ s->platform_bus_dev = riscv_create_platform_bus(mmio_irqchip,
+ &s->memmap[VIRT_PLATFORM_BUS], VIRT_PLATFORM_BUS_IRQ,
+ VIRT_PLATFORM_BUS_NUM_IRQS);
serial_mm_init(system_memory, s->memmap[VIRT_UART0].base,
0, qdev_get_gpio_in(mmio_irqchip, UART0_IRQ), 399193,
diff --git a/include/hw/riscv/device-common.h b/include/hw/riscv/device-common.h
new file mode 100644
index 0000000000..12af031bbf
--- /dev/null
+++ b/include/hw/riscv/device-common.h
@@ -0,0 +1,19 @@
+/*
+ * RISC-V board helpers for FDT generation.
+ *
+ * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
+ *
+ * SPDX-License-Identifier: GPL-2.0-or-later
+ */
+
+#ifndef RISCV_DEVICE_COMMON_H
+#define RISCV_DEVICE_COMMON_H
+
+#include "exec/hwaddr.h"
+#include "hw/core/qdev.h"
+
+DeviceState *riscv_create_platform_bus(DeviceState *irqchip,
+ const MemMapEntry *platform_bus_mem,
+ int platform_bus_base_irq,
+ int platform_bus_num_irqs);
+#endif