On Mon, 4 Dec 2023 12:23:06 +0100
Marek Vasut <marex@xxxxxxx> wrote:
> On 12/4/23 12:20, Jonathan Cameron wrote:
> > On Mon, 27 Nov 2023 22:26:53 +0100
> > Marek Vasut <marex@xxxxxxx> wrote:
> >
> >> The ISL76682 is very basic ALS which only supports ALS or IR mode
> >> in four ranges, 1k/4k/16k/64k LUX. There is no IRQ support or any
> >> other fancy functionality.
> >>
> >> Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> >> Reviewed-by: Matti Vaittinen <mazziesaccount@xxxxxxxxx>
> >> Signed-off-by: Marek Vasut <marex@xxxxxxx>
> > Hi Marek,
> >
> > Discussion around available on v5 made me look closer at that aspect.
> > You are providing all the available entries in the callback but they
> > shouldn't be exposed to actually read unless the *_available bitmap
> > bits corresponding to them are set.
> >
> > If you like I can just rip the unused code out whilst applying?
> > Or if you'd prefer to send a v7 that's great too.
> >
> > Otherwise everything looks good to me.
>
> Maybe just do that while applying and I'll test it right after to see
> whether something broke, that's probably fastest. Just let me know where
> this got applied. I have the device on my desk .
Diff is below. Applied to the togreg branch of iio.git and initially pushed out
as testing for normal reasons + for you to test.
Thanks,
Jonathan
diff --git a/drivers/iio/light/isl76682.c b/drivers/iio/light/isl76682.c
index 15a68609985b..8605187bfb62 100644
--- a/drivers/iio/light/isl76682.c
+++ b/drivers/iio/light/isl76682.c
@@ -184,8 +184,6 @@ static int intensity_scale_available[] = {
0, 673000,
};
-static int integration_time_available[] = { 0, ISL76682_INT_TIME_US };
-
static int isl76682_read_avail(struct iio_dev *indio_dev,
struct iio_chan_spec const *chan,
const int **vals, int *type,
@@ -207,11 +205,6 @@ static int isl76682_read_avail(struct iio_dev *indio_dev,
default:
return -EINVAL;
}
- case IIO_CHAN_INFO_INT_TIME:
- *vals = integration_time_available;
- *length = ARRAY_SIZE(integration_time_available);
- *type = IIO_VAL_INT_PLUS_MICRO;
- return IIO_AVAIL_LIST;
default:
return -EINVAL;
}
