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

[Marek Vasut]

On 2/11/22 12:16, haibo.c...@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@gate@gated@ , s@expect@expected@ (past tense)

With those small details fixed:

Reviewed-by: Marek Vasut <ma...@denx.de>