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 <[email protected]> > > Reviewed-by: Tom Rini <[email protected]> > > Reviewed-by: Simon Glass <[email protected]> > > LGTM > > Reviewed-by: Alexander Sverdlin <[email protected]> > > > --- > > 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 <[email protected]>
