Re: [PATCH] iio: adc: ad7192: replace usleep_range() with fsleep()
From: Andy Shevchenko
Date: Sun May 10 2026 - 09:00:55 EST
On Sun, May 10, 2026 at 10:24:01AM +0500, Stepan Ionichev wrote:
> The "at least 500us" wait after serial interface reset has no specific
> upper bound. fsleep(500) expands to the same usleep_range(500, 1000)
> internally and is the kernel helper for plain "about N microseconds"
> delays.
>
> No functional change.
...
> - usleep_range(500, 1000); /* Wait for at least 500us */
> + fsleep(500); /* Wait for at least 500us */
This is useless comment, I can see it from the code. Better comment to explain
"why?" and ideally with a reference to a datasheet.
--
With Best Regards,
Andy Shevchenko