The Linux Kernel Mailing List
 help / color / mirror / Atom feed
* [PATCH] iio: magnetometer: ak8975: replace usleep_range() with fsleep()
@ 2026-05-10  7:19 Stepan Ionichev
  2026-05-10  8:10 ` Stepan Ionichev
  2026-05-10 15:17 ` Joshua Crofts
  0 siblings, 2 replies; 3+ messages in thread
From: Stepan Ionichev @ 2026-05-10  7:19 UTC (permalink / raw)
  To: jic23
  Cc: dlechner, nuno.sa, andy, gregkh, hcazarim, linux-iio,
	linux-kernel, sozdayvek

The "minimum 500us here" wait after power-up has no specific upper
bound. fsleep(500) expands to the same usleep_range(500, 1000)
internally.

No functional change.

Signed-off-by: Stepan Ionichev <sozdayvek@gmail.com>
---
 drivers/iio/magnetometer/ak8975.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iio/magnetometer/ak8975.c b/drivers/iio/magnetometer/ak8975.c
index b648b0afa..a88b19fc1 100644
--- a/drivers/iio/magnetometer/ak8975.c
+++ b/drivers/iio/magnetometer/ak8975.c
@@ -456,7 +456,7 @@ static int ak8975_power_on(const struct ak8975_data *data)
 	 * and the minimum wait time before mode setting is 100us, in
 	 * total 300us. Add some margin and say minimum 500us here.
 	 */
-	usleep_range(500, 1000);
+	fsleep(500);
 	return 0;
 }
 
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] iio: magnetometer: ak8975: replace usleep_range() with fsleep()
  2026-05-10  7:19 [PATCH] iio: magnetometer: ak8975: replace usleep_range() with fsleep() Stepan Ionichev
@ 2026-05-10  8:10 ` Stepan Ionichev
  2026-05-10 15:17 ` Joshua Crofts
  1 sibling, 0 replies; 3+ messages in thread
From: Stepan Ionichev @ 2026-05-10  8:10 UTC (permalink / raw)
  To: joshua.crofts1
  Cc: jic23, dlechner, nuno.sa, andy, gregkh, hcazarim, linux-iio,
	linux-kernel, sozdayvek

Sorry, missed your earlier post. Please drop mine; yours has priority.

Stepan

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] iio: magnetometer: ak8975: replace usleep_range() with fsleep()
  2026-05-10  7:19 [PATCH] iio: magnetometer: ak8975: replace usleep_range() with fsleep() Stepan Ionichev
  2026-05-10  8:10 ` Stepan Ionichev
@ 2026-05-10 15:17 ` Joshua Crofts
  1 sibling, 0 replies; 3+ messages in thread
From: Joshua Crofts @ 2026-05-10 15:17 UTC (permalink / raw)
  To: Stepan Ionichev
  Cc: jic23, dlechner, nuno.sa, andy, gregkh, hcazarim, linux-iio,
	linux-kernel

On Sun, 10 May 2026 at 16:35, Stepan Ionichev <sozdayvek@gmail.com> wrote:
>
> The "minimum 500us here" wait after power-up has no specific upper
> bound. fsleep(500) expands to the same usleep_range(500, 1000)
> internally.
>
> No functional change.
>
> Signed-off-by: Stepan Ionichev <sozdayvek@gmail.com>
> ---
>  drivers/iio/magnetometer/ak8975.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/iio/magnetometer/ak8975.c b/drivers/iio/magnetometer/ak8975.c
> index b648b0afa..a88b19fc1 100644
> --- a/drivers/iio/magnetometer/ak8975.c
> +++ b/drivers/iio/magnetometer/ak8975.c
> @@ -456,7 +456,7 @@ static int ak8975_power_on(const struct ak8975_data *data)
>          * and the minimum wait time before mode setting is 100us, in
>          * total 300us. Add some margin and say minimum 500us here.
>          */
> -       usleep_range(500, 1000);
> +       fsleep(500);
>         return 0;
>  }
>
> --
> 2.43.0
>
>

I've beaten you to it unfortunately...
https://lore.kernel.org/all/20260506173946.2b1907d7@jic23-huawei/

-- 
Kind regards

CJD

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-05-10 15:28 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-10  7:19 [PATCH] iio: magnetometer: ak8975: replace usleep_range() with fsleep() Stepan Ionichev
2026-05-10  8:10 ` Stepan Ionichev
2026-05-10 15:17 ` Joshua Crofts

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox