Re: [PATCH v3 0/9] iio: introduce devm_ API for hid sensro setup and cleanup

[Sanjay Chitroda]

On Thu, May 14, 2026 at 02:47:52AM +0000, Zhang, Lixu wrote:
> Tested-by: Zhang Lixu <lixu.zhang@xxxxxxxxx>

Hi Zhang,

Thanks for the testing and validaiton.
I hope you validated the complete series, including the unbind flow for the
drivers using devm API.

Hi Jonathan,

How should we take this series forward? should I extend the series to
cover remaining HID IIO drivers for this devm API, or this series can
applied to IIO tree first and sent followup patches on top of that?

Thanks, Sanjay

> 
> >-----Original Message-----
> >From: Sanjay Chitroda <sanjayembeddedse@xxxxxxxxx>
> >Sent: Saturday, May 9, 2026 6:11 PM
> >To: jikos@xxxxxxxxxx; jic23@xxxxxxxxxx; srinivas.pandruvada@xxxxxxxxxxxxxxx
> >Cc: Lechner, David <dlechner@xxxxxxxxxxxx>; nuno.sa@xxxxxxxxxx;
> >andy@xxxxxxxxxx; sanjayembeddedse@xxxxxxxxx;
> >sakari.ailus@xxxxxxxxxxxxxxx; linux-input@xxxxxxxxxxxxxxx; linux-
> >iio@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> >Subject: [PATCH v3 0/9] iio: introduce devm_ API for hid sensro setup and
> >cleanup
> >
> >From: Sanjay Chitroda <sanjayembeddedse@xxxxxxxxx>
> >
> >Key highlights:
> >- Prepare change as pre-requisite for devm conversion for HID IIO
> >  drivers by removing redundant argument
> >- Add devm API to setup trigger and clenaup resource using
> >  devm_add_action_or_reset()
> >- few cleanup and prepratory changes before updating driver for devm_
> >- few sample driver update using devm conversion to auto release resource
> >
> >changes in v3:
> >- Added cleanup and prepratory changes before adding devm_ API
> >  conversion based on self review: 0002, 0004, 0006, 0007 and 0008
> >- Address andy's review comment on commit message and coding style
> >- v2 series -> https://lore.kernel.org/all/20260429175918.2541914-1-
> >sanjayembedded@xxxxxxxxx/
> >changes in v2:
> >- Following input from Jonathan and Andy, squash initial patch v1
> >  series in single change as individual change should not break anything
> >- Add devm API support and two driver using the same
> >- v1 series -> https://lore.kernel.org/all/20260428071613.1134053-1-
> >sanjayembedded@xxxxxxxxx/
> >
> >Testing:
> >  - Compiled with W=1
> >  - Build-tested on QEMU x86_64
> >
> >Based on further feedback and reviews, I would extend this series to convert all
> >HID IIO driver to use devm_* API.
> >
> >Thanks,
> >Sanjay Chitroda
> >
> >
> >Sanjay Chitroda (9):
> >  iio: hid-sensors: drop redundant iio_dev argument
> >  iio: hid-sensors: cleanup codestyle warning
> >  iio: hid-sensors: introduce device managed API
> >  iio: gyro: hid-sensor-gyro-3d: cleanup codestyle warning
> >  iio: gyro: hid-sensor-gyro-3d: drop hid_sensor_remove_trigger() using
> >    devm API
> >  iio: humidity: hid-sensor-humidity: cleanup codestyle check
> >  iio: humidity: hid-sensor-humidity: use common device for devres
> >  iio: humidity: hid-sensor-humidity: use local struct device
> >  iio: humidity: hid-sensor-humidity: drop hid_sensor_remove_trigger()
> >    using devm API
> >
> > drivers/iio/accel/hid-sensor-accel-3d.c       |  4 +-
> > .../common/hid-sensors/hid-sensor-trigger.c   | 24 +++++++-
> > .../common/hid-sensors/hid-sensor-trigger.h   |  5 +-
> > drivers/iio/gyro/hid-sensor-gyro-3d.c         | 16 ++---
> > drivers/iio/humidity/hid-sensor-humidity.c    | 61 +++++++++----------
> > drivers/iio/light/hid-sensor-als.c            |  4 +-
> > drivers/iio/light/hid-sensor-prox.c           |  4 +-
> > drivers/iio/magnetometer/hid-sensor-magn-3d.c |  4 +-
> >drivers/iio/orientation/hid-sensor-incl-3d.c  |  4 +-  drivers/iio/orientation/hid-
> >sensor-rotation.c |  4 +-  .../position/hid-sensor-custom-intel-hinge.c  |  4 +-
> > drivers/iio/pressure/hid-sensor-press.c       |  4 +-
> > .../iio/temperature/hid-sensor-temperature.c  |  4 +-
> > 13 files changed, 78 insertions(+), 64 deletions(-)
> >
> >
> >base-commit: 39b80c5c9830d12d2d6531059001301c4265322a
> >--
> >2.34.1
> >
>