With this patch, "env flags" commands shows information about storage attribute as well as other attributes.
Signed-off-by: AKASHI Takahiro <takahiro.aka...@linaro.org> --- cmd/nvedit.c | 37 ++++++++++++++++++++++++------------- 1 file changed, 24 insertions(+), 13 deletions(-) diff --git a/cmd/nvedit.c b/cmd/nvedit.c index 70fbb5dd8f2f..77e8846b601a 100644 --- a/cmd/nvedit.c +++ b/cmd/nvedit.c @@ -618,10 +618,12 @@ static int print_static_flags(const char *var_name, const char *flags, { enum env_flags_vartype type = env_flags_parse_vartype(flags); enum env_flags_varaccess access = env_flags_parse_varaccess(flags); + enum env_flags_varstorage storage = env_flags_parse_varstorage(flags); - printf("\t%-20s %-20s %-20s\n", var_name, + printf("\t%-20s %-20s %-20s %-20s\n", var_name, env_flags_get_vartype_name(type), - env_flags_get_varaccess_name(access)); + env_flags_get_varaccess_name(access), + env_flags_get_varstorage_name(storage)); return 0; } @@ -630,16 +632,25 @@ static int print_active_flags(ENTRY *entry) { enum env_flags_vartype type; enum env_flags_varaccess access; + enum env_flags_varstorage storage; - if (entry->flags == 0) +#ifdef CONFIG_EFI_LOADER + if (entry->context == ENVCTX_UEFI && + entry->flags == ENV_FLAGS_VARSTORAGE_NON_VOLATILE_AUTOSAVE) + return 0; +#endif + /* ENVCTX_UBOOT */ + if (entry->flags == ENV_FLAGS_VARSTORAGE_NON_VOLATILE) return 0; type = (enum env_flags_vartype) (entry->flags & ENV_FLAGS_VARTYPE_BIN_MASK); access = env_flags_parse_varaccess_from_binflags(entry->flags); - printf("\t%-20s %-20s %-20s\n", entry->key, + storage = env_flags_parse_varstorage_from_binflags(entry->flags); + printf("\t%-20s %-20s %-20s %-20s\n", entry->key, env_flags_get_vartype_name(type), - env_flags_get_varaccess_name(access)); + env_flags_get_varaccess_name(access), + env_flags_get_varstorage_name(storage)); return 0; } @@ -667,19 +678,19 @@ int do_env_flags(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) /* Print the static flags that may exist */ puts("Static flags:\n"); - printf("\t%-20s %-20s %-20s\n", "Variable Name", "Variable Type", - "Variable Access"); - printf("\t%-20s %-20s %-20s\n", "-------------", "-------------", - "---------------"); + printf("\t%-20s %-20s %-20s %-20s\n", "Variable Name", "Variable Type", + "Variable Access", "Variable Storage"); + printf("\t%-20s %-20s %-20s %-20s\n", "-------------", "-------------", + "---------------", "-------------"); env_attr_walk(ENV_FLAGS_LIST_STATIC, print_static_flags, NULL); puts("\n"); /* walk through each variable and print the flags if non-default */ puts("Active flags:\n"); - printf("\t%-20s %-20s %-20s\n", "Variable Name", "Variable Type", - "Variable Access"); - printf("\t%-20s %-20s %-20s\n", "-------------", "-------------", - "---------------"); + printf("\t%-20s %-20s %-20s %-20s\n", "Variable Name", "Variable Type", + "Variable Access", "Variable Storage"); + printf("\t%-20s %-20s %-20s %-20s\n", "-------------", "-------------", + "---------------", "-------------"); hwalk_r(&env_htab, print_active_flags); return 0; } -- 2.21.0 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot