Skip to content

Commit a67d263

Browse files
root3315jic23
authored andcommitted
iio: adc: ad7793: replace usleep_range() with fsleep()
The AD7792/AD7793 datasheet (Rev. B, page 25, RESET section) says: "When a reset is initiated, the user must allow a period of 500 us before accessing any of the on-chip registers." Use fsleep(500) instead of usleep_range(500, 2000). The 500 us minimum stays the same; fsleep() picks the upper slack itself (about +25% on a default config -- narrower than the original 2000 us). Add a code comment with the datasheet reference so the "why" of the wait is visible at the call site. Signed-off-by: Stepan Ionichev <sozdayvek@gmail.com> Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
1 parent 8e9c394 commit a67d263

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

drivers/iio/adc/ad7793.c

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,12 @@ static int ad7793_setup(struct iio_dev *indio_dev,
268268
ret = ad_sd_reset(&st->sd);
269269
if (ret < 0)
270270
goto out;
271-
usleep_range(500, 2000); /* Wait for at least 500us */
271+
272+
/*
273+
* Per AD7792/AD7793 datasheet (Rev. B, page 25, RESET section),
274+
* allow 500 us after a reset before accessing on-chip registers.
275+
*/
276+
fsleep(500);
272277

273278
/* write/read test for device presence */
274279
ret = ad_sd_read_reg(&st->sd, AD7793_REG_ID, 1, &id);

0 commit comments

Comments
 (0)