On 4/29/25 3:12 AM, Joel Stanley wrote:
The address is a hardware address, so use hwaddr for consistency with the rest of the machine. Signed-off-by: Joel Stanley <[email protected]> ---
I believe you have 2 patches #1 in this series: [PATCH 01/13] hw/riscv/virt: Fix clint base address type [PATCH 01/13] riscv/virt: Fix address type in create_fdt_socket_clint You probably renamed the commit subject to "hw/riscv/virt" and forgot to remove the generated patch, since both patches do the same changes. I believe we can ignore this patch and stick withe the first #1. Thanks, Daniel
hw/riscv/virt.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/riscv/virt.c b/hw/riscv/virt.c index 17a790821484..e4c0ac8a2a9a 100644 --- a/hw/riscv/virt.c +++ b/hw/riscv/virt.c @@ -324,7 +324,7 @@ static void create_fdt_socket_clint(RISCVVirtState *s, int cpu; g_autofree char *clint_name = NULL; g_autofree uint32_t *clint_cells = NULL; - unsigned long clint_addr; + hwaddr clint_addr; MachineState *ms = MACHINE(s); static const char * const clint_compat[2] = { "sifive,clint0", "riscv,clint0" @@ -340,8 +340,8 @@ static void create_fdt_socket_clint(RISCVVirtState *s, }clint_addr = s->memmap[VIRT_CLINT].base +- (s->memmap[VIRT_CLINT].size * socket); - clint_name = g_strdup_printf("/soc/clint@%lx", clint_addr); + s->memmap[VIRT_CLINT].size * socket; + clint_name = g_strdup_printf("/soc/clint@%"HWADDR_PRIx, clint_addr); qemu_fdt_add_subnode(ms->fdt, clint_name); qemu_fdt_setprop_string_array(ms->fdt, clint_name, "compatible", (char **)&clint_compat,
