[PATCH v2 00/13] iio: Clean up acpi_match_device() use cases

Andy Shevchenko posted 13 patches 1 month ago
There is a newer version of this series
drivers/iio/accel/mma9551.c                | 19 ++-------
drivers/iio/accel/mma9553.c                | 19 ++-------
drivers/iio/adc/pac1934.c                  |  2 +-
drivers/iio/gyro/bmg160_core.c             | 15 --------
drivers/iio/gyro/bmg160_i2c.c              |  4 +-
drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c |  5 +--
drivers/iio/industrialio-acpi.c            | 45 ++++++++++++++++++++--
drivers/iio/light/isl29018.c               | 38 ++++++------------
drivers/iio/light/ltr501.c                 | 29 +++++---------
drivers/iio/magnetometer/bmc150_magn.c     | 15 --------
include/linux/iio/iio.h                    | 10 +++++
11 files changed, 86 insertions(+), 115 deletions(-)
[PATCH v2 00/13] iio: Clean up acpi_match_device() use cases
Posted by Andy Shevchenko 1 month ago
There are current uses of acpi_match_device():
- as strange way of checking if the device was enumerated via ACPI
- as a way to get IIO device name as ACPI device instance name
- as above with accompanying driver data

Deduplicate its use by providing two new helper functions in IIO ACPI
library and update the rest accordingly.

This also includes a rework of previously sent ltr501 patch.

In v2:
- collected tags (Hans, Jean-Baptiste)
- updated SoB chain in patch 4

Andy Shevchenko (13):
  iio: magnetometer: bmc150: Drop dead code from the driver
  iio: adc: pac1934: Replace strange way of checking type of enumeration
  iio: imu: inv_mpu6050: Replace strange way of checking type of
    enumeration
  iio: acpi: Improve iio_read_acpi_mount_matrix()
  iio: acpi: Add iio_get_acpi_device_name_and_data() helper function
  iio: accel: mma9551: Replace custom implementation of
    iio_get_acpi_device_name()
  iio: accel: mma9553: Replace custom implementation of
    iio_get_acpi_device_name()
  iio: gyro: bmg160: Replace custom implementation of
    iio_get_acpi_device_name()
  iio: light: isl29018: Replace a variant of
    iio_get_acpi_device_name_and_data()
  iio: light: isl29018: drop ACPI_PTR() and CONFIG_ACPI guards
  iio: light: ltr501: Drop most likely fake ACPI IDs
  iio: light: ltr501: Add LTER0303 to the supported devices
  iio: light: ltr501: Replace a variant of
    iio_get_acpi_device_name_and_data()

 drivers/iio/accel/mma9551.c                | 19 ++-------
 drivers/iio/accel/mma9553.c                | 19 ++-------
 drivers/iio/adc/pac1934.c                  |  2 +-
 drivers/iio/gyro/bmg160_core.c             | 15 --------
 drivers/iio/gyro/bmg160_i2c.c              |  4 +-
 drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c |  5 +--
 drivers/iio/industrialio-acpi.c            | 45 ++++++++++++++++++++--
 drivers/iio/light/isl29018.c               | 38 ++++++------------
 drivers/iio/light/ltr501.c                 | 29 +++++---------
 drivers/iio/magnetometer/bmc150_magn.c     | 15 --------
 include/linux/iio/iio.h                    | 10 +++++
 11 files changed, 86 insertions(+), 115 deletions(-)

-- 
2.43.0.rc1.1336.g36b5255a03ac
Re: [PATCH v2 00/13] iio: Clean up acpi_match_device() use cases
Posted by Andy Shevchenko 1 month ago
On Thu, Oct 24, 2024 at 03:36:05PM +0300, Andy Shevchenko wrote:
> There are current uses of acpi_match_device():
> - as strange way of checking if the device was enumerated via ACPI
> - as a way to get IIO device name as ACPI device instance name
> - as above with accompanying driver data
> 
> Deduplicate its use by providing two new helper functions in IIO ACPI
> library and update the rest accordingly.
> 
> This also includes a rework of previously sent ltr501 patch.

> In v2:
> - collected tags (Hans, Jean-Baptiste)
> - updated SoB chain in patch 4

Heck, I forgot to address the main comment by Jonathan! Sorry for the noise,
I will send a v3 later today.

-- 
With Best Regards,
Andy Shevchenko