On Mon, Jan 21, 2019 at 02:41:07PM +0100, Alexander Graf wrote:
> On 01/21/2019 08:49 AM, AKASHI Takahiro wrote:
> >"env [print|set] -e" allows for handling uefi variables without
> >knowing details about mapping to corresponding u-boot variables.
> >
> >Signed-off-by: AKASHI Takahiro <takahiro.aka...@linaro.org>
> >---
> > cmd/nvedit.c | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++-
> > 1 file changed, 58 insertions(+), 1 deletion(-)
> >
> >diff --git a/cmd/nvedit.c b/cmd/nvedit.c
> >index ce746bbf1b3e..1b971d7233bc 100644
> >--- a/cmd/nvedit.c
> >+++ b/cmd/nvedit.c
> >@@ -27,6 +27,8 @@
> > #include <cli.h>
> > #include <command.h>
> > #include <console.h>
> >+#include <efi.h>
> >+#include <efi_loader.h>
> > #include <environment.h>
> > #include <search.h>
> > #include <errno.h>
> >@@ -119,6 +121,25 @@ static int do_env_print(cmd_tbl_t *cmdtp, int flag, int
> >argc,
> > int rcode = 0;
> > int env_flag = H_HIDE_DOT;
> >+#if defined(CONFIG_EFI_LOADER)
> >+ if (argc > 1 && argv[1][0] == '-' && argv[1][1] == 'e') {
> >+ efi_status_t r;
> >+
> >+ argc--;
> >+ argv++;
> >+
> >+ /* Initialize EFI drivers */
> >+ r = efi_init_obj_list();
> >+ if (r != EFI_SUCCESS) {
> >+ printf("Error: Cannot set up EFI drivers, r = %lu\n",
> >+ r & ~EFI_ERROR_MASK);
> >+ return CMD_RET_FAILURE;
> >+ }
>
> No need for efi_init_obj_list() here, as it's already done by
> do_efi_dump_var_ext(), no?
Ah, right. (That's why do_efi_dump_var_ext() was introduced.)
Thanks,
-Takahiro Akashi
> >+
> >+ return do_efi_dump_var_ext(cmdtp, flag, argc, argv);
> >+ }
> >+#endif
> >+
> > if (argc > 1 && argv[1][0] == '-' && argv[1][1] == 'a') {
> > argc--;
> > argv++;
> >@@ -216,6 +237,26 @@ static int _do_env_set(int flag, int argc, char * const
> >argv[], int env_flag)
> > ENTRY e, *ep;
> > debug("Initial value for argc=%d\n", argc);
> >+
> >+#if defined(CONFIG_EFI_LOADER)
> >+ if (argc > 1 && argv[1][0] == '-' && argv[1][1] == 'e') {
> >+ efi_status_t r;
> >+
> >+ argc--;
> >+ argv++;
> >+
> >+ /* Initialize EFI drivers */
> >+ r = efi_init_obj_list();
> >+ if (r != EFI_SUCCESS) {
> >+ printf("Error: Cannot set up EFI drivers, r = %lu\n",
> >+ r & ~EFI_ERROR_MASK);
> >+ return CMD_RET_FAILURE;
> >+ }
>
> Same here.
>
>
> Alex
>
> >+
> >+ return do_efi_set_var_ext(NULL, flag, argc, argv);
> >+ }
> >+#endif
> >+
> > while (argc > 1 && **(argv + 1) == '-') {
> > char *arg = *++argv;
> >@@ -1262,15 +1303,23 @@ static char env_help_text[] =
> > #if defined(CONFIG_CMD_IMPORTENV)
> > "env import [-d] [-t [-r] | -b | -c] addr [size] [var ...] - import
> > environment\n"
> > #endif
> >+#if defined(CONFIG_EFI_LOADER)
> >+ "env print [-a | -e [name] | name ...] - print environment\n"
> >+#else
> > "env print [-a | name ...] - print environment\n"
> >+#endif
> > #if defined(CONFIG_CMD_RUN)
> > "env run var [...] - run commands in an environment variable\n"
> > #endif
> > #if defined(CONFIG_CMD_SAVEENV) && !defined(CONFIG_ENV_IS_NOWHERE)
> > "env save - save environment\n"
> > #endif
> >+#if defined(CONFIG_EFI_LOADER)
> >+ "env set [-e | -f] name [arg ...]\n";
> >+#else
> > "env set [-f] name [arg ...]\n";
> > #endif
> >+#endif
> > U_BOOT_CMD(
> > env, CONFIG_SYS_MAXARGS, 1, do_env,
> >@@ -1295,6 +1344,10 @@ U_BOOT_CMD_COMPLETE(
> > printenv, CONFIG_SYS_MAXARGS, 1, do_env_print,
> > "print environment variables",
> > "[-a]\n - print [all] values of all environment variables\n"
> >+#if defined(CONFIG_EFI_LOADER)
> >+ "printenv -e [<name>]\n"
> >+ " - print UEFI variable 'name' or all the variables\n"
> >+#endif
> > "printenv name ...\n"
> > " - print value of environment variable 'name'",
> > var_complete
> >@@ -1322,7 +1375,11 @@ U_BOOT_CMD_COMPLETE(
> > U_BOOT_CMD_COMPLETE(
> > setenv, CONFIG_SYS_MAXARGS, 0, do_env_set,
> > "set environment variables",
> >- "[-f] name value ...\n"
> >+#if defined(CONFIG_EFI_LOADER)
> >+ "-e <name> [<value>]\n"
> >+ " - set UEFI variable 'name' to 'value' ...'\n"
> >+#endif
> >+ "setenv [-f] name value ...\n"
> > " - [forcibly] set environment variable 'name' to 'value ...'\n"
> > "setenv [-f] name\n"
> > " - [forcibly] delete environment variable 'name'",
>
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
