On Mon, 2023-11-20 at 22:32 +0100, Philippe Mathieu-Daudé wrote: > Mechanical change using the following coccinelle script: > > @@ > expression mr, owner, arg3, arg4, errp; > @@ > - memory_region_init_ram(mr, owner, arg3, arg4, &errp); > if ( > - errp > + !memory_region_init_ram(mr, owner, arg3, arg4, &errp) > ) { > ... > return; > } > > and removing the local Error variable. > > Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> > --- > hw/arm/aspeed_ast2400.c | 6 ++---- > hw/arm/aspeed_ast2600.c | 6 ++----
For my own benefit it looks like the motivating thread for this series is: https://lore.kernel.org/qemu-devel/936e1ac4-cef8-08b4-c688-e5b1e0572...@linaro.org/ Anyway, Reviewed-by: Andrew Jeffery <and...@codeconstruct.com.au> # aspeed