From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
To: jikos@kernel.org, jic23@kernel.org, srinivas.pandruvada@linux.intel.com
Cc: 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 v4 00/36] HID: iio: basic clean up and introduce devm_ API for HID sensors
Date: Mon, 25 May 2026 00:50:23 +0530 [thread overview]
Message-ID: <20260524192059.1193716-1-sanjayembedded@gmail.com> (raw)
From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
Key highlights:
- 0000-0024: General cleanup and kernel few coding style fixes across HID IIO drivers
- 0025: Remove unused iio_dev argument from HID IIO sensor helper
- 0026: Introduce devm_hid_sensor_setup_trigger() device-managed API
- 0027-0036: Convert HID IIO sensor drivers to use the new devm-based trigger setup
changes in v4:
- Extend the series to cover remaining HID IIO drivers with devm API usage
- Reorder patches to place cleanup and warning fix at beginning and,
devm-related changes toward the end based on feedback from David
- v3 series -> https://lore.kernel.org/all/20260509101040.791404-1-sanjayembedded@gmail.com/
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 for each patch in series
- Build-tested on QEMU x86_64
P.S.
- Sashiko reported an issue in a different driver and noted that it is not
introduced by this series. I have taken this feedback into account and
will address the actual issue in a separate series focus on that driver.
- Once this series is merged into the IIO tree, a number of HID IIO
drivers will become available to fully converted to devm API usage.
- The changes are organized across drivers to keep similar modifications
grouped together for consistency, making the series easier to review,
rather than grouping all changes per driver.
Thanks,
Sanjay Chitroda
Sanjay Chitroda (36):
iio: hid-sensors: add missing blank line after declarations
iio: gyro: hid-sensor-gyro-3d: use u32 instead of unsigned
iio: accel: hid-sensor-accel-3d: use u32 instead of unsigned
iio: light: hid-sensor-als: use u32 instead of unsigned
iio: light: hid-sensor-prox: use u32 instead of unsigned
iio: orientation: hid-sensor-incl-3d: use u32 instead of unsigned
iio: orientation: hid-sensor-rotation: use u32 instead of unsigned
iio: pressure: hid-sensor-press: use u32 instead of unsigned
iio: humidity: hid-sensor-humidity: align parenthesis for readability
iio: gyro: hid-sensor-gyro-3d: align parenthesis for readability
iio: magnetometer: hid-sensor-magn-3d: align parenthesis for
readability
iio: humidity: hid-sensor-humidity: use common device for devres
iio: position: hid-sensor-custom-intel-hinge: use common device for
devres
iio: temperature: hid-sensor-temperature: use common device for devres
iio: humidity: hid-sensor-humidity: use local struct device
iio: gyro: hid-sensor-gyro-3d: use local struct device
iio: accel: hid-sensor-accel-3d: use local struct device
iio: light: hid-sensor-als: use local struct device
iio: light: hid-sensor-prox: use local struct device
iio: magnetometer: hid-sensor-magn-3d: use local struct device
iio: orientation: hid-sensor-incl-3d: use local struct device
iio: orientation: hid-sensor-rotation: use local struct device
iio: position: hid-sensor-custom-intel-hinge: use local struct device
iio: pressure: hid-sensor-press: use local struct device
iio: hid-sensors: remove unused iio_dev argument
iio: hid-sensors: introduce device managed API
iio: gyro: hid-sensor-gyro-3d: drop hid_sensor_remove_trigger() using
devm API
iio: humidity: hid-sensor-humidity: drop hid_sensor_remove_trigger()
using devm API
iio: light: hid-sensor-prox: drop hid_sensor_remove_trigger() using
devm API
iio: light: hid-sensor-als: drop hid_sensor_remove_trigger() using
devm API
iio: magnetometer: hid-sensor-magn-3d: drop
hid_sensor_remove_trigger() using devm API
iio: orientation: hid-sensor-incl-3d: drop hid_sensor_remove_trigger()
using devm API
iio: orientation: hid-sensor-rotation: drop
hid_sensor_remove_trigger() using devm API
iio: position: hid-sensor-custom-intel-hinge: drop
hid_sensor_remove_trigger() using devm API
iio: pressure: hid-sensor-press: drop hid_sensor_remove_trigger()
using devm API
iio: temperature: hid-sensor-temperature: drop
hid_sensor_remove_trigger() using devm API
drivers/iio/accel/hid-sensor-accel-3d.c | 30 ++---
.../common/hid-sensors/hid-sensor-trigger.c | 24 +++-
.../common/hid-sensors/hid-sensor-trigger.h | 5 +-
drivers/iio/gyro/hid-sensor-gyro-3d.c | 96 ++++++++-------
drivers/iio/humidity/hid-sensor-humidity.c | 61 +++++-----
drivers/iio/light/hid-sensor-als.c | 31 +++--
drivers/iio/light/hid-sensor-prox.c | 30 ++---
drivers/iio/magnetometer/hid-sensor-magn-3d.c | 112 +++++++++---------
drivers/iio/orientation/hid-sensor-incl-3d.c | 36 +++---
drivers/iio/orientation/hid-sensor-rotation.c | 38 +++---
.../position/hid-sensor-custom-intel-hinge.c | 27 ++---
drivers/iio/pressure/hid-sensor-press.c | 36 +++---
.../iio/temperature/hid-sensor-temperature.c | 15 +--
13 files changed, 264 insertions(+), 277 deletions(-)
base-commit: 08297ca8422541dde6c8b7e6b1d68bd4aa4568ef
--
2.34.1
next reply other threads:[~2026-05-24 19:21 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-24 19:20 Sanjay Chitroda [this message]
2026-05-24 19:20 ` [PATCH v4 01/36] iio: hid-sensors: add missing blank line after declarations Sanjay Chitroda
2026-05-24 19:20 ` [PATCH v4 02/36] iio: gyro: hid-sensor-gyro-3d: use u32 instead of unsigned Sanjay Chitroda
2026-05-24 19:20 ` [PATCH v4 03/36] iio: accel: hid-sensor-accel-3d: " Sanjay Chitroda
2026-05-24 19:20 ` [PATCH v4 04/36] iio: light: hid-sensor-als: " Sanjay Chitroda
2026-05-24 19:20 ` [PATCH v4 05/36] iio: light: hid-sensor-prox: " Sanjay Chitroda
2026-05-24 19:20 ` [PATCH v4 06/36] iio: orientation: hid-sensor-incl-3d: " Sanjay Chitroda
2026-05-24 19:20 ` [PATCH v4 07/36] iio: orientation: hid-sensor-rotation: " Sanjay Chitroda
2026-05-24 19:20 ` [PATCH v4 08/36] iio: pressure: hid-sensor-press: " Sanjay Chitroda
2026-05-24 19:20 ` [PATCH v4 09/36] iio: humidity: hid-sensor-humidity: align parenthesis for readability Sanjay Chitroda
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260524192059.1193716-1-sanjayembedded@gmail.com \
--to=sanjayembeddedse@gmail.com \
--cc=andy@kernel.org \
--cc=dlechner@baylibre.com \
--cc=jic23@kernel.org \
--cc=jikos@kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=nuno.sa@analog.com \
--cc=sakari.ailus@linux.intel.com \
--cc=srinivas.pandruvada@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox