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.vigu...@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)
