Re: [PATCH 1/2] distro_bootcmd: Add support for customizable find_distro_rootpart

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 1932 bytes --]

On Fri, Nov 07, 2025 at 10:05:17AM +0100, Kory Maincent wrote:
> On Thu, 6 Nov 2025 12:23:29 -0600
> Tom Rini <trini@konsulko.com> wrote:
> 
> > On Thu, Nov 06, 2025 at 07:00:01PM +0100, Kory Maincent wrote:
> > > On Tue, 4 Nov 2025 10:38:16 -0600
> > > Tom Rini <trini@konsulko.com> wrote:
> > > 
> > > Other question: Is it possible to not add the environment to SPL image using
> > > the new text environment format. I have tried to update am335_evm board to
> > > it but I got a SPL too big error.
> > > I have tried to add "#if !defined(XPL_BUILD)" but it is not working.
> > > It seems the same generated environment file is used for all images.
> > > I still haven't found were it is included in the build of the SPL image.   
> > 
> > It shouldn't change the environment size, but also environment is likely
> > being pulled in to SPL for network boot support which does require
> > environment. What do you mean by "same generated environment file is
> > used for all images" ? And are you building in separate object
> > directories for a given board or is it not being re-generated on config
> > changes? Or did I misunderstand? Thanks.
> 
> Sorry if I was not clear.
> For the am335x_evm most of the environment is only in the main u-boot image
> (i.e. not an xPL image).
> https://elixir.bootlin.com/u-boot/v2025.10/source/include/configs/am335x_evm.h#L78
> 
> I would like to do the same but with the new text environment format. Is it
> possible? Something like: board/ti/am335x/am335x.env for main image
> and board/ti/am335x/spl_am335x.env for SPL image.
> If that is not the case could you point me where the change need to
> happen to help me on that. I didn't find yet where the text environment is
> pulled into the images.

Ah. It sounds like we need to add missing logic to generate the text
environment in each phase rather than once and use it in all phases?

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]