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

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello Neal,

On Wed, 23 Nov 2022 00:09:39 +0000
"Frager, Neal" <neal.frager@amd.com> wrote:

> I am not sure creating a separate BR2 config is the right solution.
> Both the u-boot and u-boot.elf file are in fact elf formatted files.
> One just has all the debug symbols and one does not.

I don't understand why you think having a separate option is a problem.
We already have separate options for each file that can be installed by
U-Boot. Adding another option for another file is completely logical,
and actually much more logical than having a weird option that
magically installs 2 files - sometimes.

> What do you think of this solution?  Basically, we check that the
> file exists before copying.  It is probably good to have this check
> anyway.

Checks like this are actually not great, because you expect something
to happen, there is no error, but the thing you were expecting did not
happen. Clearly confusing for the user.

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot