Re: [PATCH 01/18] iio: light: apds9306: Switch to sparse friendly iio_device_claim/release_direct()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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






[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux