On 9/17/25 16:17, Kory Maincent wrote:
> This reverts commit d37641c61ba212241c38e3fd22f0335fc0bebc96.
>
> Restore support for environment storage in EXT4 filesystem on eMMC boot.
> The previous cleanup incorrectly removed this fallback option which is
> needed for boards that store their environment in an EXT4 partition.
>
> This configuration is OS-specific rather than board-dependent and should
> remain as it is configurable via menuconfig. Even if it is not described
> in ST defconfigs people may have enabled it in their defconfig.
>
> Signed-off-by: Kory Maincent <kory.mainc...@bootlin.com>
> ---
>
> Debugging this issue raises another issue that need investigation.
> If env_get_location return ENVL_IS_NOWHERE but CONFIG_ENV_IS_NOWHERE is
> not set, U-boot stall without any log. There might be an ENV check issue
> here.
> ---
> board/st/stm32mp1/stm32mp1.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/board/st/stm32mp1/stm32mp1.c b/board/st/stm32mp1/stm32mp1.c
> index ab7af69d472..cb5cab9f36a 100644
> --- a/board/st/stm32mp1/stm32mp1.c
> +++ b/board/st/stm32mp1/stm32mp1.c
> @@ -751,6 +751,8 @@ enum env_location env_get_location(enum env_operation op,
> int prio)
> case BOOT_FLASH_EMMC:
> if (CONFIG_IS_ENABLED(ENV_IS_IN_MMC))
> return ENVL_MMC;
> + else if (CONFIG_IS_ENABLED(ENV_IS_IN_EXT4))
> + return ENVL_EXT4;
> else
> return ENVL_NOWHERE;
>
Reviewed-by: Patrice Chotard <patrice.chot...@foss.st.com>
Thanks
Patrice
