On 9/24/22 07:29, Marek Vasut wrote:
> From: Ye Li <ye...@nxp.com>
>
> Move init_clk_usdhc to non-clk driver case, since assigned-clocks properties
> will initialize the clocks by clk driver.
>
> Signed-off-by: Ye Li <ye...@nxp.com>
> Signed-off-by: Marek Vasut <ma...@denx.de>
Reviewed-by: Jaehoon Chung <jh80.ch...@samsung.com>
Best Regards,
Jaehoon Chung
> ---
> NOTE: Was MLK-21848-5 fsl_esdhc: fix problem when using clk driver
> NOTE: Fixes SD SDR104 mode instability on MX8MP
> ---
> Cc: Fabio Estevam <feste...@denx.de>
> Cc: Jaehoon Chung <jh80.ch...@samsung.com>
> Cc: Marcel Ziswiler <marcel.ziswi...@toradex.com>
> Cc: Max Krummenacher <max.krummenac...@toradex.com>
> Cc: Peng Fan <peng....@nxp.com>
> Cc: Stefano Babic <sba...@denx.de>
> ---
> drivers/mmc/fsl_esdhc_imx.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mmc/fsl_esdhc_imx.c b/drivers/mmc/fsl_esdhc_imx.c
> index 9befb190bdf..92b152fc979 100644
> --- a/drivers/mmc/fsl_esdhc_imx.c
> +++ b/drivers/mmc/fsl_esdhc_imx.c
> @@ -1519,8 +1519,6 @@ static int fsl_esdhc_probe(struct udevice *dev)
> * work as expected.
> */
>
> - init_clk_usdhc(dev_seq(dev));
> -
> #if CONFIG_IS_ENABLED(CLK)
> /* Assigned clock already set clock */
> ret = clk_get_by_name(dev, "per", &priv->per_clk);
> @@ -1536,6 +1534,8 @@ static int fsl_esdhc_probe(struct udevice *dev)
>
> priv->sdhc_clk = clk_get_rate(&priv->per_clk);
> #else
> + init_clk_usdhc(dev_seq(dev));
> +
> priv->sdhc_clk = mxc_get_clock(MXC_ESDHC_CLK + dev_seq(dev));
> if (priv->sdhc_clk <= 0) {
> dev_err(dev, "Unable to get clk for %s\n", dev->name);
