Re: [PATCH v3] iio: accel: adxl355: replace usleep_range() with fsleep()
From: Jonathan Cameron
Date: Mon May 11 2026 - 12:38:48 EST
On Mon, 11 May 2026 10:27:40 +0300
Andy Shevchenko <andriy.shevchenko@xxxxxxxxx> wrote:
> On Sun, May 10, 2026 at 04:38:52PM +0500, Stepan Ionichev wrote:
> > The "at least 5ms" wait after software reset has no specific upper
> > bound. Use fsleep() with 5 * USEC_PER_MSEC to make the unit
> > explicit at the call site.
>
> > No functional change.
>
> Strictly speaking the upper limit is not set as 10 milliseconds now,
> but defined by the internals of fsleep(), usually +25%.
I'm not following. There doesn't seem to be a statement of anything
about the upper limit that is used - just one on it not being specified
for the part.
>
> Perhaps Jonathan can just drop this sentence from the commit message
> whilst applying.
>
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>
>
Given I didn't really get the comment - applied as is.
J