Re: [Buildroot] [PATCH v8 01/10] package/binutils-bare-metal: new package

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

Hello,

On Thu, 14 Dec 2023 07:51:03 +0000
"Frager, Neal" <neal.frager@amd.com> wrote:

> Since we are not sure about this, I will go ahead and try to upstream
> the patch to the newlib community, and see what they say.

Thing is you will have to explain very clearly what we need a different
value, and I'm not sure we have such a clear explanation (at least I
don't have it).

> If there is a reason, perhaps they can help us with implementing a
> configurable solution, as you suggest.  In any case, I think working
> to upstream a proper solution is still better than my fixup solution.
>  I only implemented the fixup solution as it was quicker than
> spending many hours learning the newlib configure script.  Now that
> you have found the root of the problem, I think patching it upstream
> is the best option.

Many hours to add a --with-option? It should take 10 minutes or so.

> By the way, since the bare-metal toolchain no longer has a default
> architecture, all of the config files in patches 7 and 9 need the
> following line added:
> BR2_TOOLCHAIN_BARE_METAL_BUILDROOT_ARCH="microblazeel-xilinx-elf"

Correct. There was no reason to have this as the default.

> Will I need to submit a v9 for this, or could you just add this
> config for me to each of the zynqmp and versal config files?

We'll see how we merge things. Perhaps a v9 will not be needed, and
I'll fix things up when applying.

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