Re: [PATCH 1/3] mmc: sdhci-of-dwcmshc: improve delay handling with fsleep()

From: Artem Shimko

Date: Tue May 19 2026 - 11:55:36 EST


Hi Andy,

On Mon, May 18, 2026 at 9:35 PM Andy Shevchenko
<andriy.shevchenko@xxxxxxxxx> wrote:
> Yep, the hint was basically about uses of usleep_range().
Just to clarify before sending V2, for example (midpoint of the range):
s/usleep_range(1000, 2000)/fsleep(1500)/ ?

> microseconds is fine as is.
Since they occur in non-atomic context
(inside set_clock and reset functions), would it be worth replacing
them with fsleep(1) as well for consistency? =)

Thank you!
--
Best regards,
Artem Shimko



>
> > Or is that the commit description mistake?
> >
> > Thank you for reviewing it.
> >
> > [1] https://lore.kernel.org/all/ad-mCsvOWWgqt-rc@ashevche-desk.local/
>
> --
> With Best Regards,
> Andy Shevchenko
>
>