Subject: Re: [PATCH] mmc: fsl_esdhc_imx: add wait_dat0() support
On 11/2/20 8:17 PM, haibo.chen@nxp.com wrote:
From: Haibo Chen haibo.chen@nxp.com
Add wait_dat0() support, upper layer will use this callback.
Signed-off-by: Haibo Chen haibo.chen@nxp.com
drivers/mmc/fsl_esdhc_imx.c | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+)
diff --git a/drivers/mmc/fsl_esdhc_imx.c b/drivers/mmc/fsl_esdhc_imx.c index 22040c67a8..dc6a6006fa 100644 --- a/drivers/mmc/fsl_esdhc_imx.c +++ b/drivers/mmc/fsl_esdhc_imx.c @@ -1646,6 +1646,28 @@ static int fsl_esdhc_set_enhanced_strobe(struct udevice *dev) } #endif
+static int fsl_esdhc_wait_dat0(struct udevice *dev, int state,
int timeout_us)+{
- int ret = -ETIMEDOUT;
- bool dat0_high;
- bool target_dat0_high = !!state;
- struct fsl_esdhc_priv *priv = dev_get_priv(dev);
- struct fsl_esdhc *regs = priv->esdhc_regs;
- timeout_us = DIV_ROUND_UP(timeout_us, 10); /* check every 10 us. */
- while (timeout_us--) {
dat0_high = !!(esdhc_read32(®s->prsstat) & PRSSTAT_DAT0);if (dat0_high == target_dat0_high) {ret = 0;break;}- udelay(10);
Fix indent. And can't use wait_for_bit_xx()?
+1, read_poll_timeout or similar should be used here.
Thanks, Peng.
Best Regards, Jaehoon Chung
- }
- return ret;
+}
static const struct dm_mmc_ops fsl_esdhc_ops = { .get_cd = fsl_esdhc_get_cd, .send_cmd = fsl_esdhc_send_cmd, @@ -1656,6 +1678,7 @@ static const struct dm_mmc_ops fsl_esdhc_ops =
{
#if CONFIG_IS_ENABLED(MMC_HS400_ES_SUPPORT) .set_enhanced_strobe = fsl_esdhc_set_enhanced_strobe, #endif
- .wait_dat0 = fsl_esdhc_wait_dat0,
}; #endif