[PATCH v3 0/7] efi: Start to chip away at the EFI workaround

[Simon Glass]

Hi Heinrich,

On Sun, 22 Dec 2024 at 05:54, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
> On 12/19/24 03:38, Simon Glass wrote:
> > At present there is a function, efi_set_bootdev(), which is used in
> > various places to tell the EFI loader which device a file came from.
>
> As long as we have a command line to download images we won't be able to
> avoid remembering the boot device in a global.

Watch this space.

>
> >
> > With bootstd, this information is available in the bootflow.
> >
> > This little series provides a way for bootstd to provide the bootflow to
> > the EFI loader, so that it is able to select the correct paths.
> >
> > For now only the EFI bootmeth is updated.
>
> This seems to be the wrong focus. The EFI boot method should be removed.
> It just duplicates the parts of the EFI boot manager boot method.

I don't agree with that at all. The boot manager doesn't play well
with other bootmeths and needs to be looked at more closely, as the
sunxi experience shows.

I'm going to resend this series with the changes you requested, except
for the networking side (recently introduced) which needs to be
handled separately.

> >
> > Changes in v3:
> > - Make calculate_paths() static and add a comment
> > - Add new patch to support PXE booting
> > - Rebase on Tom's tree (i.e. without the pointers/addresses series)
> >    https://patchwork.ozlabs.org/project/uboot/list/?series=436241
> >
> > Changes in v2:
> > - Adjust argument ordering for efi_run_image()
> > - Use efi_loader tag instead of efi
> > - Drop unnecessary path removal
> > - Fix 'require' typo
> > - Move calculation of dev-name into a separate function
> >
> > Simon Glass (7):
> >    efi_loader: Refactor device and image paths into a function
> >    efi_loader: Make efi_run_image() static
> >    efi_loader: Update efi_run_image() to accept image and device path
> >    efi_loader: Add a version of efi_binary_run() with more parameters
> >    efi_loader: Move the fallback code from efi_run_image()
> >    efi_loader: Pass in the required parameters from EFI bootmeth
> >    bootmeth_efi: Support PXE booting
> >
> >   boot/bootmeth_efi.c          |  64 +-------
> >   include/efi_loader.h         |  10 ++
> >   lib/efi_loader/efi_bootbin.c | 276 ++++++++++++++++++++++++++---------
> >   3 files changed, 218 insertions(+), 132 deletions(-)
> >
>

Regards,
Simon