Re: [PATCH 03/12] aspeed: Use the boot_rom region of the fby35 machine

[Cédric Le Goater]

On 5/30/23 23:27, Philippe Mathieu-Daudé wrote:
> On 8/5/23 09:58, Cédric Le Goater wrote:
> > This change completes commits 5aa281d757 ("aspeed: Introduce a
> > spi_boot region under the SoC") and 8b744a6a47 ("aspeed: Add a
> > boot_rom overlap region in the SoC spi_boot container") which
> > introduced a spi_boot container at the SoC level to map the boot rom
> > region as an overlap.
>
> 2nd patch,
>
> > It also fixes a Coverity report (CID 1508061) for a memory leak
> > warning when the QEMU process exits by using an bmc_boot_rom
> > MemoryRegion available at the machine level.
>
> 1st patch?

That would be superfluous.

> > Cc: Peter Delevoryas <peter@pjd.dev>
> > Signed-off-by: Cédric Le Goater <clg@kaod.org>
> > ---
> >   hw/arm/fby35.c | 29 +++++++++++++++--------------
> >   1 file changed, 15 insertions(+), 14 deletions(-)