[PATCH v3 1/2] env: migrate static flags list to Kconfig

[Schweizer, Walter]

On Tue, 2026-05-12 at 12:15 +0000, Sverdlin, Alexander (SI B PRO TI EAC
CCP) wrote:
> Hi James,
> 
> On Mon, 2026-05-11 at 12:20 -0600, James Hilliard wrote:
> > Environment callbacks can already be configured from Kconfig with
> > CONFIG_ENV_CALLBACK_LIST_STATIC, but static environment flags still
> > require board headers to define CFG_ENV_FLAGS_LIST_STATIC.
> > 
> > Add CONFIG_ENV_FLAGS_LIST_STATIC and use it as the only board-
> > provided
> > static environment flags list. Convert the remaining default-config
> > users
> > from CFG_ENV_FLAGS_LIST_STATIC to defconfig settings and drop the
> > legacy
> > header macro from ENV_FLAGS_LIST_STATIC.
> > 
> > Move the environment flags format documentation out of README and
> > into
> > the developer environment documentation. Include the format in the
> > Kconfig help as well.
> > 
> > This lets boards configure writeable-list policy and type
> > validation
> > from defconfig without adding a config header solely for env flags.
> > 
> > This preserves the behavior of default configs. Header-only cases
> > that
> > were inactive in upstream defconfigs are not converted into
> > defconfig
> > entries: iot2050 can add its list when enabling ENV_WRITEABLE_LIST,
> > and
> > smegw01 can add mmcdev:dw support if the unlocked SYS_BOOT_LOCKED=n
> > configuration is needed.
> > 
> > Signed-off-by: James Hilliard <james.hilliard1 at gmail.com>
> > Reviewed-by: Tom Rini <trini at konsulko.com>
> > Reviewed-by: Simon Glass <sjg at chromium.org>
> 
> LGTM
> 
> Reviewed-by: Alexander Sverdlin <alexander.sverdlin at siemens.com>
> 
> > ---
> > Changes v2 -> v3:
> >   - Note that inactive header-only iot2050 flags and the smegw01
> >     SYS_BOOT_LOCKED=n mmcdev:dw flag are not converted into
> > defconfigs.
> > 
> > Changes v1 -> v2:
> >   - Convert existing CFG_ENV_FLAGS_LIST_STATIC users to
> >     CONFIG_ENV_FLAGS_LIST_STATIC in defconfigs.
> >   - Drop CFG_ENV_FLAGS_LIST_STATIC support instead of appending the
> > new
> >     Kconfig value after the legacy C macro.
> >   - Remove the trailing-comma requirement for board-specific static
> > flags.
> >   - Place the migrated defconfig entries in savedefconfig order.
> >   - Move environment flags documentation from README to
> >     doc/develop/environment.rst.
> >   - Add the flags-list format to the CONFIG_ENV_FLAGS_LIST_STATIC
> > Kconfig
> >     help text.
> > ---
> >  README                               | 45 +-----------------------
> > ----
> >  configs/aristainetos2c_defconfig     |  1 +
> >  configs/aristainetos2ccslb_defconfig |  1 +
> >  configs/hmibsc_defconfig             |  1 +
> >  configs/imx6q_bosch_acc_defconfig    |  1 +
> >  configs/imx8qxp_capricorn.config     |  1 +
> >  configs/smegw01_defconfig            |  1 +
> >  configs/socrates_defconfig           |  1 +
> >  doc/develop/environment.rst          | 40
> > +++++++++++++++++++++++++
> >  env/Kconfig                          | 23 ++++++++++++++
> >  include/configs/aristainetos2.h      |  3 --
> >  include/configs/capricorn-common.h   | 13 --------
> >  include/configs/hmibsc.h             |  4 ---
> >  include/configs/imx6q-bosch-acc.h    | 12 --------
> >  include/configs/iot2050.h            |  8 -----
> >  include/configs/smegw01.h            | 15 ----------
> >  include/configs/socrates.h           |  2 --
> >  include/env_flags.h                  |  6 +---
> >  18 files changed, 72 insertions(+), 106 deletions(-)
> 
Thanks James

Reviewed-by: Walter Schweizer <walter.schweizer at siemens.com>