Re: [PATCH] hw/sysbus: Inline and remove sysbus_add_io()

2024-02-20 Thread Peter Maydell
On Fri, 16 Feb 2024 at 15:05, Philippe Mathieu-Daudé wrote: > > sysbus_add_io(...) is a simple wrapper to > memory_region_add_subregion(get_system_io(), ...). > It is used in 3 places; inline it directly. > > Signed-off-by: Philippe Mathieu-Daudé > --- > include/hw/sysbus.h | 2 -- > hw/core/sys

Re: [PATCH] hw/sysbus: Inline and remove sysbus_add_io()

2024-02-18 Thread Philippe Mathieu-Daudé
On 16/2/24 16:04, Philippe Mathieu-Daudé wrote: sysbus_add_io(...) is a simple wrapper to memory_region_add_subregion(get_system_io(), ...). It is used in 3 places; inline it directly. Rationale here is we want to move to an explicit I/O bus, rather that an implicit one. Besides in heterogeneou

[PATCH] hw/sysbus: Inline and remove sysbus_add_io()

2024-02-16 Thread Philippe Mathieu-Daudé
sysbus_add_io(...) is a simple wrapper to memory_region_add_subregion(get_system_io(), ...). It is used in 3 places; inline it directly. Signed-off-by: Philippe Mathieu-Daudé --- include/hw/sysbus.h | 2 -- hw/core/sysbus.c| 6 -- hw/i386/kvmvapic.c | 2 +- hw/mips/mipssim.c | 2 +- h