[PATCH 1/3] mmc: fsl_esdhc_imx: use VENDORSPEC_FRC_SDCLK_ON when necessary

[Marek Vasut]

On 2/11/22 12:16, haibo.chen at nxp.com wrote:

Hi,

[...]

> @@ -897,6 +900,11 @@ static int fsl_esdhc_execute_tuning(struct udevice *dev, uint32_t opcode)
>   
>   	esdhc_stop_tuning(mmc);
>   
> +	/* change to default setting, let host control the card clock */
> +	esdhc_clrbits32(&regs->vendorspec, VENDORSPEC_FRC_SDCLK_ON);
> +	if (readx_poll_timeout(esdhc_read32, &regs->prsstat, tmp, tmp & PRSSTAT_SDOFF, 100))

Please propagate the error in both cases:

ret = readx_poll..();
if (ret)
   dev_warn(...);

return ret;

> +		pr_warn("fsl_esdhc_imx: card clock not gate off as expect.\n");

btw. s at gate@gated@ , s at expect@expected@ (past tense)

With those small details fixed:

Reviewed-by: Marek Vasut <marex at denx.de>