1. Patchew
  2. linux
  3. View series

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

Zhang, Lixu posted 9 patches 4 weeks, 1 day ago
Only 0 patches received!
There is a newer version of this series
RE: [PATCH v3 0/9] iio: introduce devm_ API for hid sensro setup and cleanup
Posted by Zhang, Lixu 4 weeks, 1 day ago 
Tested-by: Zhang Lixu <lixu.zhang@intel.com>

>-----Original Message-----
>From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
>Sent: Saturday, May 9, 2026 6:11 PM
>To: jikos@kernel.org; jic23@kernel.org; srinivas.pandruvada@linux.intel.com
>Cc: Lechner, David <dlechner@baylibre.com>; nuno.sa@analog.com;
>andy@kernel.org; sanjayembeddedse@gmail.com;
>sakari.ailus@linux.intel.com; linux-input@vger.kernel.org; linux-
>iio@vger.kernel.org; linux-kernel@vger.kernel.org
>Subject: [PATCH v3 0/9] iio: introduce devm_ API for hid sensro setup and
>cleanup
>
>From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
>
>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@gmail.com/
>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@gmail.com/
>
>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
>
Re: [PATCH v3 0/9] iio: introduce devm_ API for hid sensro setup and cleanup
Posted by Sanjay Chitroda 3 weeks, 4 days ago 
On Thu, May 14, 2026 at 02:47:52AM +0000, Zhang, Lixu wrote:
> Tested-by: Zhang Lixu <lixu.zhang@intel.com>

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@gmail.com>
> >Sent: Saturday, May 9, 2026 6:11 PM
> >To: jikos@kernel.org; jic23@kernel.org; srinivas.pandruvada@linux.intel.com
> >Cc: Lechner, David <dlechner@baylibre.com>; nuno.sa@analog.com;
> >andy@kernel.org; sanjayembeddedse@gmail.com;
> >sakari.ailus@linux.intel.com; linux-input@vger.kernel.org; linux-
> >iio@vger.kernel.org; linux-kernel@vger.kernel.org
> >Subject: [PATCH v3 0/9] iio: introduce devm_ API for hid sensro setup and
> >cleanup
> >
> >From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
> >
> >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@gmail.com/
> >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@gmail.com/
> >
> >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
> >
>
RE: [PATCH v3 0/9] iio: introduce devm_ API for hid sensro setup and cleanup
Posted by Zhang, Lixu 3 weeks, 3 days ago 
>-----Original Message-----
>From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
>Sent: Monday, May 18, 2026 4:20 AM
>To: Zhang, Lixu <lixu.zhang@intel.com>
>Cc: jikos@kernel.org; jic23@kernel.org; srinivas.pandruvada@linux.intel.com;
>Lechner, David <dlechner@baylibre.com>; nuno.sa@analog.com;
>andy@kernel.org; sakari.ailus@linux.intel.com; linux-input@vger.kernel.org;
>linux-iio@vger.kernel.org; linux-kernel@vger.kernel.org
>Subject: Re: [PATCH v3 0/9] iio: introduce devm_ API for hid sensro setup and
>cleanup
>
>On Thu, May 14, 2026 at 02:47:52AM +0000, Zhang, Lixu wrote:
>> Tested-by: Zhang Lixu <lixu.zhang@intel.com>
>
>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 Sanjay,

Yes, I validated the complete series. The unbind flow was tested as Srinivas previously
suggested - performing the unbind while iio-sensor-proxy has an open session. No
regression issues were observed.

Since iio-sensor-proxy does not typically open a gyro session, I applied the same changes
from hid-sensor-gyro-3d to hid-sensor-accel-3d and tested the unbind flow on that driver
as well. Again, no regression issues were found.

Thanks,
Lixu

>
>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@gmail.com>
>> >Sent: Saturday, May 9, 2026 6:11 PM
>> >To: jikos@kernel.org; jic23@kernel.org;
>> >srinivas.pandruvada@linux.intel.com
>> >Cc: Lechner, David <dlechner@baylibre.com>; nuno.sa@analog.com;
>> >andy@kernel.org; sanjayembeddedse@gmail.com;
>> >sakari.ailus@linux.intel.com; linux-input@vger.kernel.org; linux-
>> >iio@vger.kernel.org; linux-kernel@vger.kernel.org
>> >Subject: [PATCH v3 0/9] iio: introduce devm_ API for hid sensro setup
>> >and cleanup
>> >
>> >From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
>> >
>> >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@gmail.com/
>> >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@gmail.com/
>> >
>> >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
>> >
>>
Re: [PATCH v3 0/9] iio: introduce devm_ API for hid sensro setup and cleanup
Posted by Jonathan Cameron 3 weeks, 4 days ago 
On Mon, 18 May 2026 01:49:51 +0530
Sanjay Chitroda <sanjayembeddedse@gmail.com> wrote:

> On Thu, May 14, 2026 at 02:47:52AM +0000, Zhang, Lixu wrote:
> > Tested-by: Zhang Lixu <lixu.zhang@intel.com>  
> 
> 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?

There is just enough minor feedback on various patches that it needs
a v4 I think, or response to those bits of review feedback to argue
against the requested changes.  

I'd like to see the new API applied everywhere unless there is a reason
it's not suitable.  Key aim being that the last patch should remove
the now unused non devm functions or maybe make them private if
they are still useful from a code structure point of view.

We may be waiting on testing though for some of them so make sure
it is clear what Zhang has tested (obviously not the stuff you haven't
sent yet :)

Jonathan




> 
> Thanks, Sanjay
> 
> >   
> > >-----Original Message-----
> > >From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
> > >Sent: Saturday, May 9, 2026 6:11 PM
> > >To: jikos@kernel.org; jic23@kernel.org; srinivas.pandruvada@linux.intel.com
> > >Cc: Lechner, David <dlechner@baylibre.com>; nuno.sa@analog.com;
> > >andy@kernel.org; sanjayembeddedse@gmail.com;
> > >sakari.ailus@linux.intel.com; linux-input@vger.kernel.org; linux-
> > >iio@vger.kernel.org; linux-kernel@vger.kernel.org
> > >Subject: [PATCH v3 0/9] iio: introduce devm_ API for hid sensro setup and
> > >cleanup
> > >
> > >From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
> > >
> > >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@gmail.com/
> > >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@gmail.com/
> > >
> > >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
> > >  
> >   
>

Patchew 2.1-devel-ea86937

© 2016 - 2026 Red Hat, Inc.