[PATCH 00/10] efi: Move some efi-loader code into a new shared dir
Heinrich Schuchardt
xypron.glpk at gmx.de
Fri May 23 16:18:59 CEST 2025
On 23.05.25 15:06, Simon Glass wrote:
> Some functions provided in lib/efi_loader are actually useful for the
> app as well. This series refactors the Kconfig and directories a little
> so that code is easier to share.
>
> As a starting point, it moves some filename and device-path functions to
> the new directory.
>
> The next step would be to move device-path code over, but this will need
> some discussion.
Hello Simon,
Overall the ideas in this series look fine to me. But this series does
not apply to origin/next.
Applying: efi_loader: Separate device path into its own header
Patch failed at 0001 efi_loader: Separate device path into its own header
error: patch failed: cmd/efidebug.c:8
error: cmd/efidebug.c: patch does not apply
error: patch failed: include/efi_loader.h:967
error: include/efi_loader.h: patch does not apply
error: patch failed: lib/efi_loader/efi_bootmgr.c:12
error: lib/efi_loader/efi_bootmgr.c: patch does not apply
error: patch failed: lib/efi_loader/efi_device_path.c:10
error: lib/efi_loader/efi_device_path.c: patch does not apply
error: patch failed: lib/efi_loader/efi_helper.c:6
error: lib/efi_loader/efi_helper.c: patch does not apply
Please, resend the series based on origin/next.
Patches that are not based on upstream U-Boot cannot be reviewed via
this mailing list.
Best regards
Heinrich
>
>
> Simon Glass (10):
> efi_loader: Separate device path into its own header
> efi: Include device-path functions in the EFI API docs
> efi: Rename END to EFI_DP_END
> efi: Rename the lib/efi directory
> efi: Rename CONFIG_EFI to CONFIG_EFI_CLIENT
> efi: Create a new CONFIG_EFI
> efi: Update maintainers for EFI_CLIENT
> efi: Move the base name and PXE info into lib/efi
> efi: Adjust copyright and mention constant source
> efi: Move FDT-filename discovery into lib/efi
>
> MAINTAINERS | 5 +-
> Makefile | 4 +-
> arch/arm/include/asm/global_data.h | 2 +-
> arch/x86/Kconfig | 6 +-
> arch/x86/cpu/Makefile | 2 +-
> arch/x86/cpu/efi/Kconfig | 2 +-
> arch/x86/cpu/x86_64/Makefile | 2 +-
> cmd/Makefile | 2 +-
> cmd/bootefi.c | 1 +
> cmd/eficonfig.c | 5 +-
> cmd/efidebug.c | 7 +-
> cmd/part_find.c | 10 +-
> configs/efi-arm_app64_defconfig | 2 +-
> configs/efi-x86_app32_defconfig | 2 +-
> configs/efi-x86_app64_defconfig | 2 +-
> configs/efi-x86_payload32_defconfig | 2 +-
> configs/efi-x86_payload64_defconfig | 2 +-
> doc/api/efi.rst | 6 +
> doc/develop/uefi/u-boot_on_efi.rst | 18 +-
> drivers/block/Kconfig | 2 +-
> include/efi_device_path.h | 420 +++++++++++++++++++++
> include/efi_loader.h | 57 +--
> include/init.h | 2 +-
> lib/Kconfig | 2 +-
> lib/Makefile | 1 +
> lib/efi/Kconfig | 86 +----
> lib/efi/Makefile | 28 +-
> lib/efi/basename.c | 153 ++++++++
> lib/efi_client/Kconfig | 83 ++++
> lib/efi_client/Makefile | 28 ++
> lib/{efi => efi_client}/efi.c | 0
> lib/{efi => efi_client}/efi_app.c | 0
> lib/{efi => efi_client}/efi_app_init.c | 0
> lib/{efi => efi_client}/efi_dtb.S | 0
> lib/{efi => efi_client}/efi_info.c | 0
> lib/{efi => efi_client}/efi_stub_arm64.c | 0
> lib/{efi => efi_client}/efi_stub_x86.c | 0
> lib/{efi => efi_client}/efi_vars.c | 0
> lib/{efi => efi_client}/sdram.c | 0
> lib/efi_driver/efi_uclass.c | 1 +
> lib/efi_loader/Kconfig | 3 +
> lib/efi_loader/efi_bootbin.c | 1 +
> lib/efi_loader/efi_bootmgr.c | 4 +-
> lib/efi_loader/efi_boottime.c | 1 +
> lib/efi_loader/efi_capsule.c | 1 +
> lib/efi_loader/efi_console.c | 1 +
> lib/efi_loader/efi_device_path.c | 175 +++------
> lib/efi_loader/efi_device_path_utilities.c | 3 +-
> lib/efi_loader/efi_disk.c | 1 +
> lib/efi_loader/efi_fdt.c | 64 +---
> lib/efi_loader/efi_helper.c | 81 +---
> lib/efi_loader/efi_net.c | 1 +
> lib/efi_loader/efi_tcg2.c | 1 +
> lib/uuid.c | 4 +-
> test/lib/efi_device_path.c | 1 +
> 55 files changed, 814 insertions(+), 473 deletions(-)
> create mode 100644 include/efi_device_path.h
> create mode 100644 lib/efi/basename.c
> create mode 100644 lib/efi_client/Kconfig
> create mode 100644 lib/efi_client/Makefile
> rename lib/{efi => efi_client}/efi.c (100%)
> rename lib/{efi => efi_client}/efi_app.c (100%)
> rename lib/{efi => efi_client}/efi_app_init.c (100%)
> rename lib/{efi => efi_client}/efi_dtb.S (100%)
> rename lib/{efi => efi_client}/efi_info.c (100%)
> rename lib/{efi => efi_client}/efi_stub_arm64.c (100%)
> rename lib/{efi => efi_client}/efi_stub_x86.c (100%)
> rename lib/{efi => efi_client}/efi_vars.c (100%)
> rename lib/{efi => efi_client}/sdram.c (100%)
>
More information about the U-Boot
mailing list