Re: [PATCH 0/1] hw/arm/aspeed: Automatically zero-extend flash images

From: Peter Delevoryas <peter@pjd.dev>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: clg@kaod.org, andrew@aj.id.au, joel@jms.id.au,
	qemu-arm@nongnu.org, qemu-devel@nongnu.org, patrick@stwcx.xyz,
	Markus Armbruster <armbru@redhat.com>
Subject: Re: [PATCH 0/1] hw/arm/aspeed: Automatically zero-extend flash images
Date: Tue, 15 Nov 2022 10:30:38 -0800	[thread overview]
Message-ID: <Y3PaznQgV800nX6K@pdel-fedora-MJ0HJWH9> (raw)
In-Reply-To: <CAFEAcA9itEDTrznX1KRVEza__Dch95aBpPzbdTJ0-tuxwiHtoQ@mail.gmail.com>

On Tue, Nov 15, 2022 at 10:48:40AM +0000, Peter Maydell wrote:
> On Mon, 14 Nov 2022 at 19:08, Peter Delevoryas <peter@pjd.dev> wrote:
> >
> > I've been using this patch for a long time so that I don't have to use
> > dd to zero-extend stuff all the time. It's just doing what people are
> > doing already, right? I hope it's not controversial.
> 
> We just had a thread about this kind of thing for one of the
> riscv boards (although there the attempted approach was to
> change the size of the flash device to match the provided
> file, rather than changing the file to match the flash device):
> https://lore.kernel.org/qemu-devel/20221107130217.2243815-1-sunilvl@ventanamicro.com/

Thanks for linking this!

> 
> The short summary is (a) yes, it's controversial and
> (b) if we do something for this we need to have a standard
> approach that we do on all boards, not "some boards do
> some weird magic in different ways from everything else"...

I see, that's ok, I'll investigate option (b) then.

Thanks,
Peter

> 
> thanks
> -- PMM