[U-Boot] [PATCH] env: Add CONFIG_ENV_SUPPORT
patrick.delaunay at st.com
Thu Sep 5 07:56:45 UTC 2019
> From: Lukasz Majewski <lukma at denx.de>
> Sent: mercredi 4 septembre 2019 10:52
> Hi Patrick,
> > Add a new flag CONFIG_ENV_SUPPORT to compile all the environment
> > features in U-Boot (attributes, callbacks and flags). It is the
> > equivalent of the 2 existing flags
> > - CONFIG_SPL_ENV_SUPPORT for SPL
> > - CONFIG_TPL_ENV_SUPPORT for TPL
> Shouldn't it be the "supplement" ?
> I guess that it is possible to define
> CONFIG_SPL_ENV_SUPPORT to have ENV support in SPL
> CONFIG_TPL_ENV_SUPPORT to have ENV support in TPL
> and there is a third flag - CONFIG_ENV_SUPPORT to enable envs support in U-
> Boot proper?
> In that way one can enable ENV support only in SPL (via SPL_ENV_SUPPORT)
> and disable envs in U-Boot proper (by NOT defining ENV_SUPPORT) and use
> build in envs (in the binary).
Yes exactly the support for U-Boot, SPL or TPL are now independent,
I will update the commit message in V2, "supplement" is more clear.
> > This new configuration allows to use the macro
> > CONFIG_IS_ENABLED(ENV_SUPPORT) in the code without issue and solves
> > the regression introduced by commit 7d4776545b0f ("env: solve
> > compilation error in SPL"); change_ok was always NULL in U-Boot.
> > Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
> > ---
> > cmd/Kconfig | 2 ++
> > env/Kconfig | 7 +++++++
> > env/Makefile | 11 ++++-------
> > include/env_callback.h | 4 ++++
> > include/env_flags.h | 4 ++++
> > 5 files changed, 21 insertions(+), 7 deletions(-)
> Best regards,
> Lukasz Majewski
More information about the U-Boot