On Mon, 10 Mar 2025 17:05:50 +1030 Subhajit Ghosh <subhajit.ghosh@xxxxxxxxxxxxxx> wrote: > On 10/3/25 03:36, Jonathan Cameron wrote: > > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > > > These new functions allow sparse to find failures to release > > direct mode reducing chances of bugs over the claim_direct_mode() > > functions that are deprecated. > > > > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > Cc: Subhajit Ghosh <subhajit.ghosh@xxxxxxxxxxxxxx> > > --- > > drivers/iio/light/apds9306.c | 7 +++---- > > 1 file changed, 3 insertions(+), 4 deletions(-) > > > > diff --git a/drivers/iio/light/apds9306.c b/drivers/iio/light/apds9306.c > > index 69a0d609cffc..fca7e73a905c 100644 > > --- a/drivers/iio/light/apds9306.c > > +++ b/drivers/iio/light/apds9306.c > > @@ -831,11 +831,10 @@ static int apds9306_read_raw(struct iio_dev *indio_dev, > > * Changing device parameters during adc operation, resets > > * the ADC which has to avoided. > > */ > > - ret = iio_device_claim_direct_mode(indio_dev); > > - if (ret) > > - return ret; > > + if (!iio_device_claim_direct(indio_dev)) > > + return -EBUSY; > > ret = apds9306_read_data(data, val, reg); > > - iio_device_release_direct_mode(indio_dev); > > + iio_device_release_direct(indio_dev); > > if (ret) > > return ret; > > > Thank you Jonathan. Maybe a tag if it looks fine to you? Reviewed-by or Acked-by Thanks, Jonathan