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.
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. Best regards Heinrich
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(-)
