With this patch applied, we will be able to selectively execute
an EFI application by specifying a load option, say "1" for Boot0001,
"2" for Boot0002 and so on.

  => bootefi bootmgr <fdt addr> 1, or
     bootefi bootmgr - 1

Please note that BootXXXX need not be included in "BootOrder".

Signed-off-by: AKASHI Takahiro <takahiro.aka...@linaro.org>
---
 cmd/bootefi.c | 26 +++++++++++++++++++-------
 1 file changed, 19 insertions(+), 7 deletions(-)

diff --git a/cmd/bootefi.c b/cmd/bootefi.c
index 796ca6ee69ec..2fc52e3056d2 100644
--- a/cmd/bootefi.c
+++ b/cmd/bootefi.c
@@ -471,13 +471,13 @@ static efi_status_t bootefi_test_prepare
 
 #endif /* CONFIG_CMD_BOOTEFI_SELFTEST */
 
-static int do_bootefi_bootmgr_exec(void)
+static int do_bootefi_bootmgr_exec(int boot_id)
 {
        struct efi_device_path *device_path, *file_path;
        void *addr;
        efi_status_t r;
 
-       addr = efi_bootmgr_load(-1, &device_path, &file_path);
+       addr = efi_bootmgr_load(boot_id, &device_path, &file_path);
        if (!addr)
                return 1;
 
@@ -545,10 +545,22 @@ static int do_bootefi(cmd_tbl_t *cmdtp, int flag, int 
argc, char * const argv[])
        } else
 #endif
        if (!strcmp(argv[1], "bootmgr")) {
-               if (efi_handle_fdt(argc > 2 ? argv[2] : NULL))
-                       return CMD_RET_FAILURE;
+               char *endp;
+               int boot_id = -1;
+
+               if (argc > 2)
+                       if (efi_handle_fdt((argv[2][0] == '-') ?
+                                          NULL : argv[2]))
+                               return CMD_RET_FAILURE;
+
+               if (argc > 3) {
+                       boot_id = (int)simple_strtoul(argv[3], &endp, 0);
+                       if ((argv[3] + strlen(argv[3]) != endp) ||
+                           boot_id > 0xffff)
+                               return CMD_RET_USAGE;
+               }
 
-               return do_bootefi_bootmgr_exec();
+               return do_bootefi_bootmgr_exec(boot_id);
        } else {
                saddr = argv[1];
 
@@ -589,7 +601,7 @@ static char bootefi_help_text[] =
        "    Use environment variable efi_selftest to select a single test.\n"
        "    Use 'setenv efi_selftest list' to enumerate all tests.\n"
 #endif
-       "bootefi bootmgr [fdt addr]\n"
+       "bootefi bootmgr [<fdt addr>|'-' [<boot id>]]\n"
        "  - load and boot EFI payload based on BootOrder/BootXXXX variables.\n"
        "\n"
        "    If specified, the device tree located at <fdt address> gets\n"
@@ -597,7 +609,7 @@ static char bootefi_help_text[] =
 #endif
 
 U_BOOT_CMD(
-       bootefi, 3, 0, do_bootefi,
+       bootefi, 5, 0, do_bootefi,
        "Boots an EFI payload from memory",
        bootefi_help_text
 );
-- 
2.19.1

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to