HI Liu,

This is generic mmc.c, not dw_mmc.

On 9/21/2026 3:16 PM, baozhu.liu wrote:
In the mmc_select_hs400es function, mmc_set_card_speed(mmc, MMC_HS,
true) is executed.

Also mention |mmc_select_mode_and_width()|, which already passes |true| for HS400-ES.

The third parameter is set to true, but the judgment branch for the
hsdowngrade parameter does not include MMC_HS400_ES_SUPPORT.

Not a missing runtime if case - if (hsdowngrade) never checks the mode.
Modify the message.

Signed-off-by: baozhu.liu<[email protected]>
---
  drivers/mmc/mmc.c | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c
index 670e92ee12b..2bc36c1405c 100644
--- a/drivers/mmc/mmc.c
+++ b/drivers/mmc/mmc.c
@@ -1007,7 +1007,8 @@ static int mmc_set_card_speed(struct mmc *mmc, enum 
bus_mode mode,
                return err;
#if CONFIG_IS_ENABLED(MMC_HS200_SUPPORT) || \
-    CONFIG_IS_ENABLED(MMC_HS400_SUPPORT)
+       CONFIG_IS_ENABLED(MMC_HS400_SUPPORT) || \
+       CONFIG_IS_ENABLED(MMC_HS400_ES_SUPPORT)
        /*
         * In case the eMMC is in HS200/HS400 mode and we are downgrading

Update this comment to mention HS400-ES as well.

Reply via email to