Re: [PATCH] iio: frequency: adrf6780: replace usleep_range() with fsleep()
From: Joshua Crofts
Date: Mon May 11 2026 - 09:25:36 EST
On Mon, 11 May 2026 at 15:09, Stepan Ionichev <sozdayvek@xxxxxxxxx> wrote:
>
> The ADRF6780 datasheet (Rev. D, page 23, ADC section) says:
> "Wait approximately 200 us for the ADC to be ready."
>
> fsleep(200) expands to the same usleep_range(200, 250). Use the
> flexible sleep helper, which picks the right primitive for the
> given microsecond delay.
>
> Replace the generic "Recommended delay for the ADC to be ready"
> comment with the datasheet reference so the "why" of the wait is
> visible at the call site.
>
> No functional change.
>
> Signed-off-by: Stepan Ionichev <sozdayvek@xxxxxxxxx>
> ---
> drivers/iio/frequency/adrf6780.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
This seems fine to me.
Reviewed-by: Joshua Crofts <joshua.crofts1@xxxxxxxxx>
--
Kind regards
CJD