Sachin Kamat <sachin.kamat@xxxxxxxxxx> wrote:
>With the conversion of iio drivers to use device managed APIs, several
>drivers now have a remove function with only the
>iio_device_unregister()
>call. Thus introduce device managed version of iio_device_register() to
>clean up lot of such code. 3 patches also removes the unregister call
>in drivers
>where it happens to be the last call before return statement.
>
>This is only compile tested as I do not have any actual hardware to
>test ATM.
Hi all.
I definitely want to gather opinions on this idea.
It makes me nervous but the stats speak for themselves.
Jonathan
>
>Sachin Kamat (32):
> iio: core: Implement devm_iio_device_{register,unregister}
> iio: accel: kxsd9: Use devm_iio_device_register
> iio: adc: mcp3422: Use devm_iio_device_register
> iio: adc: twl6030-gpadc: Use devm_iio_device_register
> iio: adc: viperboard: Use devm_iio_device_register
> iio: dac: ad5421: Use devm_iio_device_register
> iio: dac: ad5755: Use devm_iio_device_register
> iio: dac: max517: Use devm_iio_device_register
> iio: dac: mcp4725: Use devm_iio_device_register
> iio: gyro: adis16130: Use devm_iio_device_register
> iio: gyro: adxrs450: Use devm_iio_device_register
> iio: light: vcnl4000: Use devm_iio_device_register
> staging: iio: adis16220: Use devm_iio_device_register
> staging: iio: ad7816: Use devm_iio_device_register
> staging: iio: lpc32xx_adc: Use devm_iio_device_register
> staging: iio: addac: Use devm_iio_device_register
> staging: iio: ad7150: Use devm_iio_device_register
> staging: iio: ad7746: Use devm_iio_device_register
> staging: iio: ad5930: Use devm_iio_device_register
> staging: iio: ad9850: Use devm_iio_device_register
> staging: iio: ad9852: Use devm_iio_device_register
> staging: iio: ad9910: Use devm_iio_device_register
> staging: iio: ad9951: Use devm_iio_device_register
> staging: iio: adis16060: Use devm_iio_device_register
> staging: iio: isl29018: Use devm_iio_device_register
> staging: iio: isl29028: Use devm_iio_device_register
> staging: iio: tsl2583: Use devm_iio_device_register
> staging: iio: tsl2x7x_core: Use devm_iio_device_register
> staging: iio: ade7854: Use devm_iio_device_register
> staging: iio: ad2s1200: Use devm_iio_device_register
> staging: iio: ad2s90: Use devm_iio_device_register
> staging: iio: ad7152: Use devm_iio_device_register
>
> Documentation/driver-model/devres.txt | 2 ++
> drivers/iio/accel/kxsd9.c | 10 +-------
> drivers/iio/adc/mcp3422.c | 9 +------
> drivers/iio/adc/twl6030-gpadc.c | 5 +---
> drivers/iio/adc/viperboard_adc.c | 12 +---------
> drivers/iio/dac/ad5421.c | 12 +---------
> drivers/iio/dac/ad5755.c | 12 +---------
> drivers/iio/dac/max517.c | 9 +------
> drivers/iio/dac/mcp4725.c | 9 +------
> drivers/iio/gyro/adis16130.c | 9 +------
> drivers/iio/gyro/adxrs450.c | 14 ++---------
>drivers/iio/industrialio-core.c | 35
>++++++++++++++++++++++++++++
> drivers/iio/light/vcnl4000.c | 9 +------
> drivers/staging/iio/accel/adis16220_core.c | 7 ++----
> drivers/staging/iio/adc/ad7816.c | 12 +---------
> drivers/staging/iio/adc/lpc32xx_adc.c | 12 +---------
> drivers/staging/iio/addac/adt7316-i2c.c | 6 -----
> drivers/staging/iio/addac/adt7316-spi.c | 6 -----
> drivers/staging/iio/addac/adt7316.c | 12 +---------
> drivers/staging/iio/addac/adt7316.h | 1 -
> drivers/staging/iio/cdc/ad7150.c | 12 +---------
> drivers/staging/iio/cdc/ad7152.c | 12 +---------
> drivers/staging/iio/cdc/ad7746.c | 12 +---------
> drivers/staging/iio/frequency/ad5930.c | 10 +-------
> drivers/staging/iio/frequency/ad9850.c | 10 +-------
> drivers/staging/iio/frequency/ad9852.c | 10 +-------
> drivers/staging/iio/frequency/ad9910.c | 10 +-------
> drivers/staging/iio/frequency/ad9951.c | 10 +-------
> drivers/staging/iio/gyro/adis16060_core.c | 10 +-------
> drivers/staging/iio/light/isl29018.c | 13 +----------
> drivers/staging/iio/light/isl29028.c | 11 +--------
> drivers/staging/iio/light/tsl2583.c | 10 +-------
> drivers/staging/iio/light/tsl2x7x_core.c | 4 +---
> drivers/staging/iio/meter/ade7854-i2c.c | 6 -----
> drivers/staging/iio/meter/ade7854-spi.c | 7 ------
> drivers/staging/iio/meter/ade7854.c | 14 +----------
> drivers/staging/iio/meter/ade7854.h | 1 -
> drivers/staging/iio/resolver/ad2s1200.c | 10 +-------
> drivers/staging/iio/resolver/ad2s90.c | 10 +-------
> include/linux/iio/iio.h | 25 ++++++++++++++++++++
> 40 files changed, 95 insertions(+), 315 deletions(-)
--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
