On 5/12/22 03:27, Pali Rohár wrote: > Currently default fallback SDHC mode is 1-bit. Add new config option > CONFIG_SYS_FSL_ESDHC_DEFAULT_BUS_WIDTH to allow specifying default fallback > mode. This is useful e.g. for SPL builds which loads other parts from SD > card during boot process. > > Signed-off-by: Pali Rohár <p...@kernel.org> > --- > drivers/mmc/Kconfig | 5 +++++ > drivers/mmc/fsl_esdhc.c | 1 + > 2 files changed, 6 insertions(+) > > diff --git a/drivers/mmc/Kconfig b/drivers/mmc/Kconfig > index f04cc44e1973..df15dff6248f 100644 > --- a/drivers/mmc/Kconfig > +++ b/drivers/mmc/Kconfig > @@ -826,6 +826,11 @@ config FSL_ESDHC_VS33_NOT_SUPPORT > For eSDHC, power supply is through peripheral circuit. 3.3V support is > common. Select this if 3.3V power supply not supported. > > +config SYS_FSL_ESDHC_DEFAULT_BUS_WIDTH > + int > + depends on FSL_ESDHC > + default 1
Is there any reason not to use dt? Best Regards, Jaehoon Chung > + > config FSL_ESDHC_IMX > bool "Freescale/NXP i.MX eSDHC controller support" > help > diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c > index 834f8933b0c6..a4c22942c2be 100644 > --- a/drivers/mmc/fsl_esdhc.c > +++ b/drivers/mmc/fsl_esdhc.c > @@ -988,6 +988,7 @@ int fsl_esdhc_mmc_init(struct bd_info *bis) > > cfg = calloc(sizeof(struct fsl_esdhc_cfg), 1); > cfg->esdhc_base = CONFIG_SYS_FSL_ESDHC_ADDR; > + cfg->max_bus_width = CONFIG_SYS_FSL_ESDHC_DEFAULT_BUS_WIDTH; > /* Prefer peripheral clock which provides higher frequency. */ > if (gd->arch.sdhc_per_clk) > cfg->sdhc_clk = gd->arch.sdhc_per_clk;