On Sat, Mar 09, 2024 at 09:20:26PM +1030, Subhajit Ghosh wrote: > Support for Avago APDS9306 Ambient Light Sensor. > > Driver support for Avago (Broadcom) APDS9306 Ambient Light Sensor. > It has two channels - ALS and CLEAR. The ALS (Ambient Light Sensor) > channel approximates the response of the human-eye providing direct > read out where the output count is proportional to ambient light levels. > It is internally temperature compensated and rejects 50Hz and 60Hz flicker > caused by artificial light sources. Hardware interrupt configuration is > optional. It is a low power device with 20 bit resolution and has > configurable adaptive interrupt mode and interrupt persistence mode. > The device also features inbuilt hardware gain, multiple integration time > selection options and sampling frequency selection options. > > This driver also uses the IIO GTS (Gain Time Scale) Helpers Namespace for > Scales, Gains and Integration time implementation. > > Link: https://docs.broadcom.com/doc/AV02-4755EN ... > - Removed 'ret' from iio_gts_find_new_gain_by_old_gain_time() > as it is not used. The current implementaion of the above > function passes all my tests of changing integration times, > scales and gains from userspace. This is weird. I believe we have to fix the API first before adding / dropping the return value from it. This odd API which makes everybody confused. -- With Best Regards, Andy Shevchenko