On Sat, Oct 23, 2021 at 05:25:54PM -0600, Simon Glass wrote:

> The bootflow feature provide a built-in way for U-Boot to automatically
> boot an Operating System without custom scripting and other customisation.
> This is called 'standard boot' since it provides a standard way for
> U-Boot to boot a distro, without scripting.
> 
> It introduces the following concepts:
> 
>    - bootdev - a device which can hold a distro
>    - bootmeth - a method to scan a bootdev to find bootflows (owned by
>                 U-Boot)
>    - bootflow - a description of how to boot (owned by the distro)
> 
> This series provides an implementation of these, enabled to scan for
> bootflows from MMC, USB and Ethernet. It supports the existing distro
> boot as well as the EFI loader flow (bootefi/bootmgr). It works
> similiarly to the existing script-based approach, but is native to
> U-Boot.

I'm going to break my feedback down in to a few threads, to hopefully
not confuse things too much.  My first comment is that rpi_arm64 grows
in size by 17 kilobytes, with the whole series (pxe, env, this) applied.
And while there's a few small changes in the pxe cleanup I'm going to
re-investigate on their own, it's really just this series, right here,
adding tons of code.  To replace an admittedly complex bit of
environment scripting, with C.  It's not even the earlier parts of the
series to clean up / prepare, it starts at "bootstd: Add the bootstd
uclass and core implementation" and keeps going from there.

-- 
Tom