Hi all,
Please disregard this patch. I'm not sure if CONFIG_VAL is the correct macro when there are no SPL_,TPL_ versions for SYS_MMC_ENV_PART option.
So back on square one.
How do we solve this to support SYS_MMC_ENV_PART = 2?
We have a configuration where we want the environment on "mmcblk0boot1" (which is the **second** boot partition) and the IS_ENABLED evaluates to **0** which prevents the "hwpart = mmc_get_env_part(mmc);" to even run.
Thanks for your help.
-- Javier Viguera
On 1/12/23 12:09, Javier Viguera wrote:
SYS_MMC_ENV_PART is an integer option. If it is set to 2 to use the second hardware boot partition, the IS_ENABLED check fails as it expects a boolean, and then it does not get the correct hwpart.
Signed-off-by: Javier Viguera javier.viguera@digi.com
env/mmc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/env/mmc.c b/env/mmc.c index cb14bbb58f13..df82969d4231 100644 --- a/env/mmc.c +++ b/env/mmc.c @@ -111,7 +111,7 @@ static inline s64 mmc_offset(struct mmc *mmc, int copy) int hwpart = 0; int err;
- if (IS_ENABLED(CONFIG_SYS_MMC_ENV_PART))
if (CONFIG_VAL(SYS_MMC_ENV_PART)) hwpart = mmc_get_env_part(mmc);
#if defined(CONFIG_ENV_MMC_PARTITION)