On Mon, 29 Jul 2024 17:20:54 +0530 Abhash Jha <abhashkumarjha123@xxxxxxxxx> wrote: > Add new ALS channel and allow reading raw and scale values. > Also provide gain and resolution configuration for ALS channel. > Add automatic mode switching between the UVS and ALS channel > based on which channel is being accessed. > The default mode in which the sensor start is ALS mode. > > Signed-off-by: Abhash Jha <abhashkumarjha123@xxxxxxxxx> Hi Abhash, Patch looks good but one quick question. Why not present an IIO_LIGHT channel? Needs to be converted to be illuminance (with scale / offset applied) rather than IIO_INTENSITY which we use when the frequency response is different from the requirements to measure Lux (and the units get very vague!) Looks like what you have here is close, but not quite the right scale factor as not including integration time and the mysterious 0.6 on the datasheet. If we can provide a signal scaled to illuminance that tends to be a lot more useful for things like screen brightness control because it should be close at least to other light sensors. Jonathan