On 27/02/2020 12:27, Jerome Brunet wrote: > Until now, the mmc clock was left in a good enough state by the ROM > code to be used by the controller. However on some SoC, if the ROM > code finds a bootloader on USB or SPI, it might leave the MMC clock > in state the controller cannot work with. > > Enable the input clocks provided to the mmc controller. While the > u-boot mmc controller driver is not doing fancy settings like the Linux, > it at least needs to make these clocks are running. > > Reviewed-by: Neil Armstrong <[email protected]> > Signed-off-by: Jerome Brunet <[email protected]> > --- > drivers/mmc/meson_gx_mmc.c | 14 +++++++++++++- > 1 file changed, 13 insertions(+), 1 deletion(-) > > diff --git a/drivers/mmc/meson_gx_mmc.c b/drivers/mmc/meson_gx_mmc.c > index 031cc79ccb1f..7060f15787f6 100644 > --- a/drivers/mmc/meson_gx_mmc.c > +++ b/drivers/mmc/meson_gx_mmc.c > @@ -4,6 +4,7 @@ > */ > > #include <common.h> > +#include <clk.h> > #include <dm.h> > #include <fdtdec.h> > #include <malloc.h> > @@ -240,12 +241,23 @@ static int meson_mmc_probe(struct udevice *dev) > struct mmc_uclass_priv *upriv = dev_get_uclass_priv(dev); > struct mmc *mmc = &pdata->mmc; > struct mmc_config *cfg = &pdata->cfg; > + struct clk_bulk clocks; > uint32_t val; > + int ret; > + > #ifdef CONFIG_PWRSEQ > struct udevice *pwr_dev; > - int ret; > #endif > > + /* Enable the clocks feeding the MMC controller */ > + ret = clk_get_bulk(dev, &clocks); > + if (ret) > + return ret; > + > + ret = clk_enable_bulk(&clocks); > + if (ret) > + return ret; > + > cfg->voltages = MMC_VDD_33_34 | MMC_VDD_32_33 | > MMC_VDD_31_32 | MMC_VDD_165_195; > cfg->host_caps = MMC_MODE_8BIT | MMC_MODE_4BIT | >
This doesn't apply on master, will fix while applying. Neil
