]> git.dujemihanovic.xyz Git - u-boot.git/commit
env: common: accept flags on reset to default env
authorYaniv Levinsky <yaniv.levinsky@compulab.co.il>
Sun, 24 Jun 2018 16:16:57 +0000 (19:16 +0300)
committerTom Rini <trini@konsulko.com>
Thu, 19 Jul 2018 20:17:58 +0000 (16:17 -0400)
commitc5d548a9f881b2268f1d81dab9cd329ae336607e
tree5a0e567d718a3d1816ab6ce8145be83067bca46b
parent5a04264ecf5f0f853f2d562b4c5efb139d3d4a9d
env: common: accept flags on reset to default env

The function set_default_env() sets the hashtable flags for import_r().
Formally set_default_env() doesn't accept flags from its callers. In
practice the caller can (un)set the H_INTERACTIVE flag, but it has to be
done using the first character of the function's string argument. Other
flags like H_FORCE can't be set by the caller.

Change the function to accept flags argument. The benefits are:
1. The caller will have to explicitly set the H_INTERACTIVE flag,
   instead of un-setting it using a special char in a string.
2. Add the ability to propagate flags from the caller to himport(),
   especially the H_FORCE flag from do_env_default() in nvedit.c that
   currently gets ignored for "env default -a -f" commands.
3. Flags and messages will not be coupled together. A caller will be
   able to set flags without passing a string and vice versa.

Please note:
The propagation of H_FORCE from do_env_default() does not introduce any
functional changes, because currently himport_r() is set to destroy the
old environment regardless if H_FORCE flag is set or not. More changes
are needed to utilize the propagation of H_FORCE.

Signed-off-by: Yaniv Levinsky <yaniv.levinsky@compulab.co.il>
Acked-by: Igor Grinberg <grinberg@compulab.co.il>
13 files changed:
arch/arm/mach-imx/mx6/opos6ul.c
cmd/nvedit.c
common/board_r.c
common/spl/spl_dfu.c
env/common.c
env/ext4.c
env/fat.c
env/mmc.c
env/nand.c
env/sata.c
env/sf.c
env/ubi.c
include/environment.h