[PATCH v2 2/2] env: mmc: Apply GPT only on eMMC user HW partition

[Marek Vasut]

Apply the GPT U-Boot environment GUID type look up only on eMMC user
HW partition, do not apply the look up on eMMC boot HW partitions as
mmc_offset_try_partition() assumes either SD partitions or eMMC user
HW partition.

This fixes environment operation on systems where CONFIG_SYS_MMC_ENV_PART
is non-zero and CONFIG_SYS_REDUNDAND_ENVIRONMENT is set.

Fixes: 80105d8fd52 ("env: mmc: select GPT env partition by type guid")
Signed-off-by: Marek Vasut <ma...@denx.de>
---
Cc: Patrice Chotard <patrice.chot...@foss.st.com>
Cc: Patrick Delaunay <patrick.delau...@foss.st.com>
Cc: Tom Rini <tr...@konsulko.com>
---
V2: Rebase on changes in 1/2
---
 env/mmc.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/env/mmc.c b/env/mmc.c
index d51a5579128..88f8a9a8978 100644
--- a/env/mmc.c
+++ b/env/mmc.c
@@ -92,7 +92,7 @@ static inline int mmc_offset_try_partition(const char *str, 
int copy, s64 *val)
        return 0;
 }
 
-static inline s64 mmc_offset(int copy)
+static inline s64 mmc_offset(struct mmc *mmc, int copy)
 {
        const struct {
                const char *offset_redund;
@@ -106,8 +106,12 @@ static inline s64 mmc_offset(int copy)
        s64 val = 0, defvalue;
        const char *propname;
        const char *str;
+       int hwpart = 0;
        int err;
 
+       if (IS_ENABLED(CONFIG_SYS_MMC_ENV_PART))
+               hwpart = mmc_get_env_part(mmc);
+
        /* look for the partition in mmc CONFIG_SYS_MMC_ENV_DEV */
        str = ofnode_conf_read_str(dt_prop.partition);
        if (str) {
@@ -119,7 +123,7 @@ static inline s64 mmc_offset(int copy)
        }
 
        /* try the GPT partition with "U-Boot ENV" TYPE GUID */
-       if (IS_ENABLED(CONFIG_PARTITION_TYPE_GUID)) {
+       if (IS_ENABLED(CONFIG_PARTITION_TYPE_GUID) && hwpart == 0) {
                err = mmc_offset_try_partition(NULL, copy, &val);
                if (!err)
                        return val;
@@ -136,7 +140,7 @@ static inline s64 mmc_offset(int copy)
        return ofnode_conf_read_int(propname, defvalue);
 }
 #else
-static inline s64 mmc_offset(int copy)
+static inline s64 mmc_offset(struct mmc *mmc, int copy)
 {
        s64 offset = ENV_MMC_OFFSET;
 
@@ -149,7 +153,7 @@ static inline s64 mmc_offset(int copy)
 
 __weak int mmc_get_env_addr(struct mmc *mmc, int copy, u32 *env_addr)
 {
-       s64 offset = mmc_offset(copy);
+       s64 offset = mmc_offset(mmc, copy);
 
        if (offset == ENV_MMC_INVALID_OFFSET) {
                printf("Invalid ENV offset in MMC, copy=%d\n", copy);
-- 
2.39.1