On Thu, Jun 2, 2022 at 4:03 PM Nuno Sá <nuno.sa@xxxxxxxxxx> wrote: > > The main goal of this patchset is to convert the iio inkern interface to > be firmware agnostic. > > In the process of doing it, I discover the bug solved in patch 1/6 which > also served as motivation for patch 2/6. > > Note that patch 3/6 is touching multiple files at once since the changes > are very simple. I can separate it in several patches if that is > preferred for better tagging though. > > The same can be said for patch 5/6 even though in this case it will be > more cumbersome to separate it in several patches (we will need to > temporarily support both options; convert each driver separately; > remove of). I think the changes in the drivers are simple enough that > we can have it like it is now... > > Lastly, this is only compile tested with allyesconfig for arm and arm64. > While I surely can come up with some dummy devices to make sure I can still > properly get IIO channels, having proper tested-by tags on platforms > relying on this interface would be very appreciated (and I suspect Jonathan > will require it). Thanks! I will definitely have a look into it later on. > Nuno Sá (6): > iio: inkern: fix return value in devm_of_iio_channel_get_by_name() > iio: inkern: only return error codes in iio_channel_get_*() APIs > iio: treewide: explicitly add proper header files > iio: inkern: split of_iio_channel_get_by_name() > iio: inkern: move to fwnode properties > iio: inkern: fix coding style warnings > > drivers/iio/adc/ab8500-gpadc.c | 7 +- > drivers/iio/adc/ad7606.c | 1 + > drivers/iio/adc/ad7606_par.c | 1 + > drivers/iio/adc/at91-sama5d2_adc.c | 7 +- > drivers/iio/adc/berlin2-adc.c | 2 + > drivers/iio/adc/imx7d_adc.c | 1 + > drivers/iio/adc/imx8qxp-adc.c | 1 + > drivers/iio/adc/ingenic-adc.c | 10 +- > drivers/iio/adc/mp2629_adc.c | 1 + > drivers/iio/adc/mt6360-adc.c | 1 + > drivers/iio/adc/npcm_adc.c | 1 + > drivers/iio/adc/qcom-pm8xxx-xoadc.c | 14 +- > drivers/iio/adc/qcom-spmi-adc5.c | 13 +- > drivers/iio/adc/qcom-spmi-vadc.c | 7 +- > drivers/iio/adc/rzg2l_adc.c | 1 + > drivers/iio/adc/stm32-adc.c | 7 +- > .../cros_ec_sensors/cros_ec_lid_angle.c | 1 + > .../common/cros_ec_sensors/cros_ec_sensors.c | 1 + > drivers/iio/dac/stm32-dac.c | 2 + > drivers/iio/dac/vf610_dac.c | 1 + > drivers/iio/humidity/hts221_buffer.c | 1 + > drivers/iio/inkern.c | 240 ++++++++++-------- > drivers/iio/light/cros_ec_light_prox.c | 1 + > drivers/iio/pressure/cros_ec_baro.c | 1 + > drivers/iio/trigger/stm32-lptimer-trigger.c | 1 + > drivers/thermal/qcom/qcom-spmi-adc-tm5.c | 3 +- > include/linux/iio/consumer.h | 28 +- > include/linux/iio/iio.h | 7 +- > 28 files changed, 200 insertions(+), 162 deletions(-) > > -- > 2.36.1 > -- With Best Regards, Andy Shevchenko