| Message ID | 20230919174931.1417681-1-u.kleine-koenig@pengutronix.de |
|---|---|
| Headers | show |
| Series | iio: Convert to platform remove callback returning void | expand |
On Tue, 19 Sep 2023 19:48:42 +0200 Uwe Kleine-König <u.kleine-koenig@pengutronix.de> wrote: > this series converts all platform drivers below drivers/iio to use > .remove_new(). The motivation is to get rid of an integer return code > that is (mostly) ignored by the platform driver core and error prone on > the driver side. As all platform drivers return zero unconditionally in their > remove callback up to now, the conversions are "trivial". > > See commit 5c5a7680e67b ("platform: Provide a remove callback that > returns no value") for an extended explanation and the eventual goal. > > There are no interdependencies between the patches. As there are still > quite a few drivers to convert, I'm happy about every patch that makes > it in. So even if there is a merge conflict with one patch until you > apply or I picked a wrong subject prefix, please apply the remainder of > this series anyhow. > Series applied to the togreg branch of iio.git and pushed out as testing to see if 0-day finds anything we are missing. I've picked up all tags give as of early today. Thanks, Jonathan > Best regards > Uwe > > Uwe Kleine-König (49): > iio: accel: hid-sensor-accel-3d: Convert to platform remove callback > returning void > iio: adc: ab8500-gpadc: Convert to platform remove callback returning > void > iio: adc: at91-sama5d2: Convert to platform remove callback returning > void > iio: adc: at91: Convert to platform remove callback returning void > iio: adc: axp20x: Convert to platform remove callback returning void > iio: adc: bcm_iproc: Convert to platform remove callback returning > void > iio: adc: dln2: Convert to platform remove callback returning void > iio: adc: ep93xx: Convert to platform remove callback returning void > iio: adc: exynos: Convert to platform remove callback returning void > iio: adc: fsl-imx25-gcq: Convert to platform remove callback returning > void > iio: adc: hx711: Convert to platform remove callback returning void > iio: adc: imx8qxp: Convert to platform remove callback returning void > iio: adc: imx93: Convert to platform remove callback returning void > iio: adc: meson_saradc: Convert to platform remove callback returning > void > iio: adc: mp2629: Convert to platform remove callback returning void > iio: adc: mxs-lradc: Convert to platform remove callback returning > void > iio: adc: npcm: Convert to platform remove callback returning void > iio: adc: qcom-pm8xxx-xoadc: Convert to platform remove callback > returning void > iio: adc: rcar-gyroadc: Convert to platform remove callback returning > void > iio: adc: stm32-adc-core: Convert to platform remove callback > returning void > iio: adc: stm32-adc: Convert to platform remove callback returning > void > iio: adc: stm32-dfsdm-adc: Convert to platform remove callback > returning void > iio: adc: stm32-dfsdm-core: Convert to platform remove callback > returning void > iio: adc: sun4i-gpadc-iio: Convert to platform remove callback > returning void > iio: adc: ti_am335x_adc: Convert to platform remove callback returning > void > iio: adc: twl4030-madc: Convert to platform remove callback returning > void > iio: adc: twl6030-gpadc: Convert to platform remove callback returning > void > iio: adc: vf610_adc: Convert to platform remove callback returning > void > iio: dac: dpot-dac: Convert to platform remove callback returning void > iio: dac: lpc18xx_dac: Convert to platform remove callback returning > void > iio: dac: stm32-dac-core: Convert to platform remove callback > returning void > iio: dac: stm32-dac: Convert to platform remove callback returning > void > iio: dac: vf610: Convert to platform remove callback returning void > iio: gyro: hid-sensor-gyro-3d: Convert to platform remove callback > returning void > iio: humidity: hid-sensor-humidity: Convert to platform remove > callback returning void > iio: light: cm3605: Convert to platform remove callback returning void > iio: light: hid-sensor-als: Convert to platform remove callback > returning void > iio: light: hid-sensor-prox: Convert to platform remove callback > returning void > iio: light: lm3533-als: Convert to platform remove callback returning > void > iio: magnetometer: hid-sensor-magn-3d: Convert to platform remove > callback returning void > iio: orientation: hid-sensor-incl-3d: Convert to platform remove > callback returning void > iio: orientation: hid-sensor-rotation: Convert to platform remove > callback returning void > iio: position: hid-sensor-custom-intel-hinge: Convert to platform > remove callback returning void > iio: pressure: hid-sensor: Convert to platform remove callback > returning void > iio: proximity: cros_ec_mkbp: Convert to platform remove callback > returning void > iio: proximity: srf04: Convert to platform remove callback returning > void > iio: temperature: hid-sensor: Convert to platform remove callback > returning void > iio: trigger: iio-trig-interrupt: Convert to platform remove callback > returning void > iio: trigger: stm32-timer: Convert to platform remove callback > returning void > > drivers/iio/accel/hid-sensor-accel-3d.c | 6 ++---- > drivers/iio/adc/ab8500-gpadc.c | 6 ++---- > drivers/iio/adc/at91-sama5d2_adc.c | 6 ++---- > drivers/iio/adc/at91_adc.c | 6 ++---- > drivers/iio/adc/axp20x_adc.c | 6 ++---- > drivers/iio/adc/bcm_iproc_adc.c | 6 ++---- > drivers/iio/adc/dln2-adc.c | 5 ++--- > drivers/iio/adc/ep93xx_adc.c | 6 ++---- > drivers/iio/adc/exynos_adc.c | 6 ++---- > drivers/iio/adc/fsl-imx25-gcq.c | 6 ++---- > drivers/iio/adc/hx711.c | 6 ++---- > drivers/iio/adc/imx8qxp-adc.c | 6 ++---- > drivers/iio/adc/imx93_adc.c | 6 ++---- > drivers/iio/adc/meson_saradc.c | 6 ++---- > drivers/iio/adc/mp2629_adc.c | 6 ++---- > drivers/iio/adc/mxs-lradc-adc.c | 6 ++---- > drivers/iio/adc/npcm_adc.c | 6 ++---- > drivers/iio/adc/qcom-pm8xxx-xoadc.c | 6 ++---- > drivers/iio/adc/rcar-gyroadc.c | 6 ++---- > drivers/iio/adc/stm32-adc-core.c | 6 ++---- > drivers/iio/adc/stm32-adc.c | 6 ++---- > drivers/iio/adc/stm32-dfsdm-adc.c | 6 ++---- > drivers/iio/adc/stm32-dfsdm-core.c | 6 ++---- > drivers/iio/adc/sun4i-gpadc-iio.c | 8 +++----- > drivers/iio/adc/ti_am335x_adc.c | 6 ++---- > drivers/iio/adc/twl4030-madc.c | 6 ++---- > drivers/iio/adc/twl6030-gpadc.c | 6 ++---- > drivers/iio/adc/vf610_adc.c | 6 ++---- > drivers/iio/dac/dpot-dac.c | 6 ++---- > drivers/iio/dac/lpc18xx_dac.c | 6 ++---- > drivers/iio/dac/stm32-dac-core.c | 6 ++---- > drivers/iio/dac/stm32-dac.c | 6 ++---- > drivers/iio/dac/vf610_dac.c | 6 ++---- > drivers/iio/gyro/hid-sensor-gyro-3d.c | 6 ++---- > drivers/iio/humidity/hid-sensor-humidity.c | 6 ++---- > drivers/iio/light/cm3605.c | 6 ++---- > drivers/iio/light/hid-sensor-als.c | 6 ++---- > drivers/iio/light/hid-sensor-prox.c | 6 ++---- > drivers/iio/light/lm3533-als.c | 6 ++---- > drivers/iio/magnetometer/hid-sensor-magn-3d.c | 6 ++---- > drivers/iio/orientation/hid-sensor-incl-3d.c | 6 ++---- > drivers/iio/orientation/hid-sensor-rotation.c | 6 ++---- > drivers/iio/position/hid-sensor-custom-intel-hinge.c | 6 ++---- > drivers/iio/pressure/hid-sensor-press.c | 6 ++---- > drivers/iio/proximity/cros_ec_mkbp_proximity.c | 6 ++---- > drivers/iio/proximity/srf04.c | 6 ++---- > drivers/iio/temperature/hid-sensor-temperature.c | 6 ++---- > drivers/iio/trigger/iio-trig-interrupt.c | 6 ++---- > drivers/iio/trigger/stm32-timer-trigger.c | 6 ++---- > 49 files changed, 99 insertions(+), 196 deletions(-) > > > base-commit: 29e400e3ea486bf942b214769fc9778098114113
Hello: This patch was applied to chrome-platform/linux.git (for-kernelci) by Jonathan Cameron <Jonathan.Cameron@huawei.com>: On Tue, 19 Sep 2023 19:48:42 +0200 you wrote: > this series converts all platform drivers below drivers/iio to use > .remove_new(). The motivation is to get rid of an integer return code > that is (mostly) ignored by the platform driver core and error prone on > the driver side. As all platform drivers return zero unconditionally in their > remove callback up to now, the conversions are "trivial". > > See commit 5c5a7680e67b ("platform: Provide a remove callback that > returns no value") for an extended explanation and the eventual goal. > > [...] Here is the summary with links: - [45/49] iio: proximity: cros_ec_mkbp: Convert to platform remove callback returning void https://git.kernel.org/chrome-platform/c/2df694f710d2 You are awesome, thank you!
Hello: This patch was applied to chrome-platform/linux.git (for-next) by Jonathan Cameron <Jonathan.Cameron@huawei.com>: On Tue, 19 Sep 2023 19:48:42 +0200 you wrote: > this series converts all platform drivers below drivers/iio to use > .remove_new(). The motivation is to get rid of an integer return code > that is (mostly) ignored by the platform driver core and error prone on > the driver side. As all platform drivers return zero unconditionally in their > remove callback up to now, the conversions are "trivial". > > See commit 5c5a7680e67b ("platform: Provide a remove callback that > returns no value") for an extended explanation and the eventual goal. > > [...] Here is the summary with links: - [45/49] iio: proximity: cros_ec_mkbp: Convert to platform remove callback returning void https://git.kernel.org/chrome-platform/c/2df694f710d2 You are awesome, thank you!
this series converts all platform drivers below drivers/iio to use .remove_new(). The motivation is to get rid of an integer return code that is (mostly) ignored by the platform driver core and error prone on the driver side. As all platform drivers return zero unconditionally in their remove callback up to now, the conversions are "trivial". See commit 5c5a7680e67b ("platform: Provide a remove callback that returns no value") for an extended explanation and the eventual goal. There are no interdependencies between the patches. As there are still quite a few drivers to convert, I'm happy about every patch that makes it in. So even if there is a merge conflict with one patch until you apply or I picked a wrong subject prefix, please apply the remainder of this series anyhow. Best regards Uwe Uwe Kleine-König (49): iio: accel: hid-sensor-accel-3d: Convert to platform remove callback returning void iio: adc: ab8500-gpadc: Convert to platform remove callback returning void iio: adc: at91-sama5d2: Convert to platform remove callback returning void iio: adc: at91: Convert to platform remove callback returning void iio: adc: axp20x: Convert to platform remove callback returning void iio: adc: bcm_iproc: Convert to platform remove callback returning void iio: adc: dln2: Convert to platform remove callback returning void iio: adc: ep93xx: Convert to platform remove callback returning void iio: adc: exynos: Convert to platform remove callback returning void iio: adc: fsl-imx25-gcq: Convert to platform remove callback returning void iio: adc: hx711: Convert to platform remove callback returning void iio: adc: imx8qxp: Convert to platform remove callback returning void iio: adc: imx93: Convert to platform remove callback returning void iio: adc: meson_saradc: Convert to platform remove callback returning void iio: adc: mp2629: Convert to platform remove callback returning void iio: adc: mxs-lradc: Convert to platform remove callback returning void iio: adc: npcm: Convert to platform remove callback returning void iio: adc: qcom-pm8xxx-xoadc: Convert to platform remove callback returning void iio: adc: rcar-gyroadc: Convert to platform remove callback returning void iio: adc: stm32-adc-core: Convert to platform remove callback returning void iio: adc: stm32-adc: Convert to platform remove callback returning void iio: adc: stm32-dfsdm-adc: Convert to platform remove callback returning void iio: adc: stm32-dfsdm-core: Convert to platform remove callback returning void iio: adc: sun4i-gpadc-iio: Convert to platform remove callback returning void iio: adc: ti_am335x_adc: Convert to platform remove callback returning void iio: adc: twl4030-madc: Convert to platform remove callback returning void iio: adc: twl6030-gpadc: Convert to platform remove callback returning void iio: adc: vf610_adc: Convert to platform remove callback returning void iio: dac: dpot-dac: Convert to platform remove callback returning void iio: dac: lpc18xx_dac: Convert to platform remove callback returning void iio: dac: stm32-dac-core: Convert to platform remove callback returning void iio: dac: stm32-dac: Convert to platform remove callback returning void iio: dac: vf610: Convert to platform remove callback returning void iio: gyro: hid-sensor-gyro-3d: Convert to platform remove callback returning void iio: humidity: hid-sensor-humidity: Convert to platform remove callback returning void iio: light: cm3605: Convert to platform remove callback returning void iio: light: hid-sensor-als: Convert to platform remove callback returning void iio: light: hid-sensor-prox: Convert to platform remove callback returning void iio: light: lm3533-als: Convert to platform remove callback returning void iio: magnetometer: hid-sensor-magn-3d: Convert to platform remove callback returning void iio: orientation: hid-sensor-incl-3d: Convert to platform remove callback returning void iio: orientation: hid-sensor-rotation: Convert to platform remove callback returning void iio: position: hid-sensor-custom-intel-hinge: Convert to platform remove callback returning void iio: pressure: hid-sensor: Convert to platform remove callback returning void iio: proximity: cros_ec_mkbp: Convert to platform remove callback returning void iio: proximity: srf04: Convert to platform remove callback returning void iio: temperature: hid-sensor: Convert to platform remove callback returning void iio: trigger: iio-trig-interrupt: Convert to platform remove callback returning void iio: trigger: stm32-timer: Convert to platform remove callback returning void drivers/iio/accel/hid-sensor-accel-3d.c | 6 ++---- drivers/iio/adc/ab8500-gpadc.c | 6 ++---- drivers/iio/adc/at91-sama5d2_adc.c | 6 ++---- drivers/iio/adc/at91_adc.c | 6 ++---- drivers/iio/adc/axp20x_adc.c | 6 ++---- drivers/iio/adc/bcm_iproc_adc.c | 6 ++---- drivers/iio/adc/dln2-adc.c | 5 ++--- drivers/iio/adc/ep93xx_adc.c | 6 ++---- drivers/iio/adc/exynos_adc.c | 6 ++---- drivers/iio/adc/fsl-imx25-gcq.c | 6 ++---- drivers/iio/adc/hx711.c | 6 ++---- drivers/iio/adc/imx8qxp-adc.c | 6 ++---- drivers/iio/adc/imx93_adc.c | 6 ++---- drivers/iio/adc/meson_saradc.c | 6 ++---- drivers/iio/adc/mp2629_adc.c | 6 ++---- drivers/iio/adc/mxs-lradc-adc.c | 6 ++---- drivers/iio/adc/npcm_adc.c | 6 ++---- drivers/iio/adc/qcom-pm8xxx-xoadc.c | 6 ++---- drivers/iio/adc/rcar-gyroadc.c | 6 ++---- drivers/iio/adc/stm32-adc-core.c | 6 ++---- drivers/iio/adc/stm32-adc.c | 6 ++---- drivers/iio/adc/stm32-dfsdm-adc.c | 6 ++---- drivers/iio/adc/stm32-dfsdm-core.c | 6 ++---- drivers/iio/adc/sun4i-gpadc-iio.c | 8 +++----- drivers/iio/adc/ti_am335x_adc.c | 6 ++---- drivers/iio/adc/twl4030-madc.c | 6 ++---- drivers/iio/adc/twl6030-gpadc.c | 6 ++---- drivers/iio/adc/vf610_adc.c | 6 ++---- drivers/iio/dac/dpot-dac.c | 6 ++---- drivers/iio/dac/lpc18xx_dac.c | 6 ++---- drivers/iio/dac/stm32-dac-core.c | 6 ++---- drivers/iio/dac/stm32-dac.c | 6 ++---- drivers/iio/dac/vf610_dac.c | 6 ++---- drivers/iio/gyro/hid-sensor-gyro-3d.c | 6 ++---- drivers/iio/humidity/hid-sensor-humidity.c | 6 ++---- drivers/iio/light/cm3605.c | 6 ++---- drivers/iio/light/hid-sensor-als.c | 6 ++---- drivers/iio/light/hid-sensor-prox.c | 6 ++---- drivers/iio/light/lm3533-als.c | 6 ++---- drivers/iio/magnetometer/hid-sensor-magn-3d.c | 6 ++---- drivers/iio/orientation/hid-sensor-incl-3d.c | 6 ++---- drivers/iio/orientation/hid-sensor-rotation.c | 6 ++---- drivers/iio/position/hid-sensor-custom-intel-hinge.c | 6 ++---- drivers/iio/pressure/hid-sensor-press.c | 6 ++---- drivers/iio/proximity/cros_ec_mkbp_proximity.c | 6 ++---- drivers/iio/proximity/srf04.c | 6 ++---- drivers/iio/temperature/hid-sensor-temperature.c | 6 ++---- drivers/iio/trigger/iio-trig-interrupt.c | 6 ++---- drivers/iio/trigger/stm32-timer-trigger.c | 6 ++---- 49 files changed, 99 insertions(+), 196 deletions(-) base-commit: 29e400e3ea486bf942b214769fc9778098114113