bootmeth: use efi_loader interfaces instead of bootefi command

Now that efi_loader subsystem provides interfaces that are equivalent
with bootefi command, we can replace command invocations with APIs.

Signed-off-by: AKASHI Takahiro <takahiro.akashi@linaro.org>

diff --git a/boot/Kconfig b/boot/Kconfig
index b438002059c33f78833588b3d203c318d49ea529..987ca73141174a9cfe4c51f82728c5435e5ce5fb 100644 (file)
--- a/boot/Kconfig
+++ b/boot/Kconfig
@@ -523,7 +523,7 @@ config BOOTMETH_EXTLINUX_PXE
 
 config BOOTMETH_EFILOADER
        bool "Bootdev support for EFI boot"
-       depends on CMD_BOOTEFI
+       depends on BOOTEFI_BOOTMGR
        default y
        help
          Enables support for EFI boot using bootdevs. This makes the
@@ -558,7 +558,7 @@ config BOOTMETH_DISTRO
        select BOOTMETH_SCRIPT if CMDLINE # E.g. Armbian uses scripts
        select BOOTMETH_EXTLINUX  # E.g. Debian uses these
        select BOOTMETH_EXTLINUX_PXE if CMD_PXE && CMD_NET && DM_ETH
-       select BOOTMETH_EFILOADER if CMD_BOOTEFI # E.g. Ubuntu uses this
+       select BOOTMETH_EFILOADER if BOOTEFI_BOOTMGR # E.g. Ubuntu uses this
 
 config SPL_BOOTMETH_VBE
        bool "Bootdev support for Verified Boot for Embedded (SPL)"

diff --git a/boot/Makefile b/boot/Makefile
index de0eafed14b1e0fd56058a1d8f7782cbc5d899ae..a90ebea5a867f3ca1de357cb8c29883c320365f3 100644 (file)
--- a/boot/Makefile
+++ b/boot/Makefile
@@ -34,7 +34,7 @@ obj-$(CONFIG_$(SPL_TPL_)BOOTMETH_CROS) += bootm.o bootm_os.o bootmeth_cros.o
 obj-$(CONFIG_$(SPL_TPL_)BOOTMETH_SANDBOX) += bootmeth_sandbox.o
 obj-$(CONFIG_$(SPL_TPL_)BOOTMETH_SCRIPT) += bootmeth_script.o
 ifdef CONFIG_$(SPL_TPL_)BOOTSTD_FULL
-obj-$(CONFIG_CMD_BOOTEFI_BOOTMGR) += bootmeth_efi_mgr.o
+obj-$(CONFIG_BOOTEFI_BOOTMGR) += bootmeth_efi_mgr.o
 obj-$(CONFIG_$(SPL_TPL_)EXPO) += bootflow_menu.o
 obj-$(CONFIG_$(SPL_TPL_)BOOTSTD) += bootflow_menu.o
 obj-$(CONFIG_$(SPL_TPL_)CEDIT) += cedit.o

diff --git a/boot/bootm_os.c b/boot/bootm_os.c
index b92422171a8499cbc0c9e27bc4786e20f75da0a4..dc4046ad082a2223def9e3f7baa48642c90d8e06 100644 (file)
--- a/boot/bootm_os.c
+++ b/boot/bootm_os.c
@@ -476,43 +476,27 @@ static int do_bootm_tee(int flag, int argc, char *const argv[],
 static int do_bootm_efi(int flag, int argc, char *const argv[],
                        struct bootm_headers *images)
 {
-       efi_status_t efi_ret;
+       int ret;
        void *image_buf;
 
        if (flag != BOOTM_STATE_OS_GO)
                return 0;
 
-       /* Initialize EFI drivers */
-       efi_ret = efi_init_obj_list();
-       if (efi_ret != EFI_SUCCESS) {
-               printf("## Failed to initialize UEFI sub-system: r = %lu\n",
-                      efi_ret & ~EFI_ERROR_MASK);
-               return 1;
-       }
+       /* We expect to return */
+       images->os.type = IH_TYPE_STANDALONE;
 
-       /* Install device tree */
-       efi_ret = efi_install_fdt(images->ft_len
-                                 ? images->ft_addr : EFI_FDT_USE_INTERNAL);
-       if (efi_ret != EFI_SUCCESS) {
-               printf("## Failed to install device tree: r = %lu\n",
-                      efi_ret & ~EFI_ERROR_MASK);
-               return 1;
-       }
+       image_buf = map_sysmem(images->ep, images->os.image_len);
 
        /* Run EFI image */
        printf("## Transferring control to EFI (at address %08lx) ...\n",
               images->ep);
        bootstage_mark(BOOTSTAGE_ID_RUN_OS);
 
-       /* We expect to return */
-       images->os.type = IH_TYPE_STANDALONE;
+       ret = efi_binary_run(image_buf, images->os.image_len,
+                            images->ft_len
+                            ? images->ft_addr : EFI_FDT_USE_INTERNAL);
 
-       image_buf = map_sysmem(images->ep, images->os.image_len);
-
-       efi_ret = efi_run_image(image_buf, images->os.image_len);
-       if (efi_ret != EFI_SUCCESS)
-               return 1;
-       return 0;
+       return ret;
 }
 #endif
 

diff --git a/boot/bootmeth_efi.c b/boot/bootmeth_efi.c
index 9ba7734911e13d1bdd52fbb885a7ef256a3d859d..d46bff51d8a7a6ec0fbcbf1d960c08e709673ccc 100644 (file)
--- a/boot/bootmeth_efi.c
+++ b/boot/bootmeth_efi.c
@@ -413,7 +413,6 @@ static int distro_efi_read_bootflow(struct udevice *dev, struct bootflow *bflow)
 static int distro_efi_boot(struct udevice *dev, struct bootflow *bflow)
 {
        ulong kernel, fdt;
-       char cmd[50];
        int ret;
 
        kernel = env_get_hex("kernel_addr_r", 0);
@@ -442,12 +441,7 @@ static int distro_efi_boot(struct udevice *dev, struct bootflow *bflow)
                fdt = env_get_hex("fdt_addr_r", 0);
        }
 
-       /*
-        * At some point we can add a real interface to bootefi so we can call
-        * this directly. For now, go through the CLI, like distro boot.
-        */
-       snprintf(cmd, sizeof(cmd), "bootefi %lx %lx", kernel, fdt);
-       if (run_command(cmd, 0))
+       if (efi_binary_run(map_sysmem(kernel, 0), 0, map_sysmem(fdt, 0)))
                return log_msg_ret("run", -EINVAL);
 
        return 0;

diff --git a/boot/bootmeth_efi_mgr.c b/boot/bootmeth_efi_mgr.c
index 6428c096d7f8c790551b12dd28b0252ea5293b62..ed29d7ef02104fef176d45e5ae90f66f6fc8cd01 100644 (file)
--- a/boot/bootmeth_efi_mgr.c
+++ b/boot/bootmeth_efi_mgr.c
@@ -87,7 +87,7 @@ static int efi_mgr_boot(struct udevice *dev, struct bootflow *bflow)
        int ret;
 
        /* Booting is handled by the 'bootefi bootmgr' command */
-       ret = run_command("bootefi bootmgr", 0);
+       ret = efi_bootmgr_run(EFI_FDT_USE_INTERNAL);
 
        return 0;
 }

diff --git a/cmd/Kconfig b/cmd/Kconfig
index b21a30c08b4c83b3240f7fb9580e6a82c87cc541..24bfbe505722dcd99713dda5ac49ac2887186015 100644 (file)
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -273,7 +273,7 @@ config CMD_BOOTMETH
 
 config BOOTM_EFI
        bool "Support booting UEFI FIT images"
-       depends on CMD_BOOTEFI && CMD_BOOTM && FIT
+       depends on BOOTEFI_BOOTMGR && CMD_BOOTM && FIT
        default y
        help
          Support booting UEFI FIT images via the bootm command.

diff --git a/test/boot/bootflow.c b/test/boot/bootflow.c
index 597f624d0369bcd0b82a3ea493fe22c260c6a50b..f3e5a839da4737fa76a6ca8b44e1f80a24ec9314 100644 (file)
--- a/test/boot/bootflow.c
+++ b/test/boot/bootflow.c
@@ -374,7 +374,7 @@ static int bootflow_system(struct unit_test_state *uts)
 {
        struct udevice *bootstd, *dev;
 
-       if (!IS_ENABLED(CONFIG_CMD_BOOTEFI_BOOTMGR))
+       if (!IS_ENABLED(CONFIG_BOOTEFI_BOOTMGR))
                return -EAGAIN;
        ut_assertok(uclass_first_device_err(UCLASS_BOOTSTD, &bootstd));
        ut_assertok(device_bind(bootstd, DM_DRIVER_GET(bootmeth_efi_mgr),