Re: [Buildroot] [PATCH v2 1/1] boot/uboot/uboot.mk: add stripped u-boot.elf support

From: Luca Ceresoli via buildroot <buildroot@buildroot.org>
To: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: "Frager, Neal" <neal.frager@amd.com>,
	"Frager, Neal via buildroot" <buildroot@buildroot.org>
Subject: Re: [Buildroot] [PATCH v2 1/1] boot/uboot/uboot.mk: add stripped u-boot.elf support
Date: Tue, 22 Nov 2022 22:31:25 +0100	[thread overview]
Message-ID: <20221122223125.33045c23@booty> (raw)
In-Reply-To: <20221122214713.1359fcac@windsurf>

Hi Thomas, Neal,

On Tue, 22 Nov 2022 21:47:13 +0100
Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote:

> On Tue, 22 Nov 2022 19:07:43 +0100
> Luca Ceresoli <luca.ceresoli@bootlin.com> wrote:
> 
> > I agree with the principle you are proposing.
> > 
> > However, according to the U-Boot source code, u-boot.elf appears as
> > u-boot.bin repackaged into an ELF file. Is "stripped" a correct word
> > for this?  
> 
> I used "stripped" because that's how Neal described this file.
> 
> Based on what you say, perhaps a better option name is:
> 
> BR2_TARGET_UBOOT_FORMAT_BIN_ELF
> 	bool "u-boot.elf"
> 
> Thoughts?

Seems better.

Neal, adding a short help text would also be useful IMO, as this "bin
elf" Frankenstein creation is really non intuitive.

-- 
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot