Re: [PATCH 00/19] vbe: Series part E

[Caleb Connolly <caleb.connolly@linaro.org>]

Hi Simon,

On 29/08/2024 15:57, Simon Glass wrote:
> 
> This includes various patches towards implementing the VBE abrec
> bootmeth in U-Boot. It mostly focuses on SPL tweaks and adjusting what
> fatures are available in VPL.

It would be helpful if you could include some context as to what VBE is, 
how this series fits in to previous ones, etc... If this is part of some 
overarching changes then having some common background justification / 
context which is copy-pasted to all the relevant patch series would make 
sense imo.

Thanks and regards,
> 
> 
> Simon Glass (19):
>    image: Add a prototype for fit_image_get_phase()
>    serial: ns16550: Allow clocks to be missing
>    boot: Allow FIT to fall back from best-match option
>    bootstd: Avoid sprintf() in SPL when creating bootdevs
>    boot: Respect the load_op in fit_image_load()
>    malloc: Show amount of used space when memory runs out
>    malloc: Provide a simple malloc for VPL
>    Support setting a maximum size for the VPL image
>    spl: Report a loader failure
>    spl: Allow serial to be disabled in any XPL phase
>    spl: Support a relocated stack in any XPL phase
>    spl: Drop use of uintptr_t
>    spl: Drop a duplicate variable in boot_from_devices()
>    spl: Add some more debugging to load_simple_fit()
>    spl: lib: Allow for decompression in any SPL build
>    boot: Allow use of FIT in TPL and VPL
>    lib: Allow crc8 in TPL and VPL
>    boot: Imply CRC8 with VBE
>    hash: Plumb crc8 into the hash functions
> 
>   boot/Kconfig                  | 71 ++++++++++++++++++++++++++++++++++-
>   boot/Makefile                 |  4 +-
>   boot/bootdev-uclass.c         | 10 ++++-
>   boot/image-fit.c              | 29 ++++++++------
>   common/hash.c                 |  8 ++++
>   common/malloc_simple.c        |  3 +-
>   common/spl/Kconfig.vpl        | 17 +++++++++
>   common/spl/spl.c              | 15 +++++---
>   common/spl/spl_atf.c          | 36 +++++++++---------
>   common/spl/spl_fit.c          | 12 +++++-
>   common/spl/spl_legacy.c       |  8 ++--
>   configs/sandbox_vpl_defconfig |  3 +-
>   drivers/serial/ns16550.c      |  2 +-
>   include/image.h               | 16 +++++++-
>   include/spl.h                 | 28 +++++++-------
>   include/u-boot/crc.h          |  3 ++
>   lib/Kconfig                   | 53 ++++++++++++++++++++++++++
>   lib/Makefile                  |  8 ++--
>   lib/crc8.c                    |  6 +++
>   19 files changed, 263 insertions(+), 69 deletions(-)
> 

-- 
// Caleb (they/them)