On Tue, 29 Apr 2025 at 21:56, Daniel Henrique Barboza <dbarb...@ventanamicro.com> wrote: > > > > 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 <j...@jms.id.au> > > --- > > 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,
Guilty! Thanks for the close review. You're correct, the first one is the correct one. I pushed a branch here if it helps: https://gitlab.com/shenki/qemu/-/tree/riscv-dt-props-v1
