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
