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