[PATCH 00/24] IIO: Clean up ACPI_PTR() usage.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>

Many of these patches touch ancient drivers so the authors have long
moved on to other things. As such I'd appreciate it if anyone has time
to take a look!

The initial part of this series deals with cases where unused variable
warnings are generated with !CONFIG_ACPI but they most date back long
enough to predate this being reported by lkp in general and the lkp folk
have gotten push back not to push out such reports due to the high level
of noise. I think some of these were reported to me directly on my
Huawei.com account but aren't on lore.kernel.org. If I can easily
track down appropriate closes tags I will add them when applying.

The last few patches are for drivers that did have the right protections
but where the usage was so trivial I'm not convinced they took the best
approach. In the interests of not leaving cases of what I consider an
undesirable pattern around for people to copy I have dropped the ACPI_PTR()
usage and config guards from those as well.

There are 3 remaining cases of ACPI_PTR() in IIO. In each of those there
is a significant chunk of ACPI specific handling so the relevant guards
are there anyway and the complexity is more easily justified. I have
not intent to touch those at this time.

Jonathan Cameron (24):
  iio: accel: da280: Stop using ACPI_PTR()
  iio: accel: kxcjk-1013: Move acpi_device_id table under ifdef
    CONFIG_ACPI
  iio: accel: mma9551: Drop ACPI_PTR() usage
  iio: accel: mma9553: Drop ACPI_PTR() usage
  iio: accel: mxc4005: Drop ACPI_PTR() usage
  iio: accel: mxc6255: Drop ACPI_PTR() usage
  iio: accel: stk8ba50: Drop ACPI_PTR() usage
  iio: accel: bmc150: Drop ACPI_PTR()
  iio: gyro: bmg160: Drop ACPI_PTR() usage
  iio: humidity: hts221: Drop ACPI_PTR() usage
  iio: imu: fxos8700: Drop ACPI_PTR() usage
  iio: imu: kmx61: Drop ACPI_PTR() usage
  iio: light: jsa1212: Drop ACPI_PTR() usage
  iio: light: ltr501: Drop ACPI_PTR() usage
  iio: light: rpr0521: Drop ACPI_PTR() usage
  iio: light: stk3310: Drop ACPI_PTR() usage
  iio: light: us5182d: Drop ACPI_PTR() usage
  iio: magnetometer: bmc150: Drop ACPI_PTR() usage
  iio: magnetometer: mmc35240: Drop ACPI_PTR() usage
  iio: potentiometer: max5487: Drop ACPI_PTR() usage
  iio: st_sensors: drop ACPI_PTR() and CONFIG_ACPI guards
  iio: pressure: hp206c: drop ACPI_PTR() and CONFIG_ACPI guards
  iio: light: max44000: drop ACPI_PTR() and CONFIG_ACPI guards
  iio: adc: ti-adc109s102: drop ACPI_PTR() and CONFIG_ACPI guards

 drivers/iio/accel/bmc150-accel-i2c.c       |  2 +-
 drivers/iio/accel/bmc150-accel-spi.c       |  3 +-
 drivers/iio/accel/da280.c                  |  2 +-
 drivers/iio/accel/kxcjk-1013.c             | 33 +++++++++++-----------
 drivers/iio/accel/mma9551.c                |  4 +--
 drivers/iio/accel/mma9553.c                |  4 +--
 drivers/iio/accel/mxc4005.c                |  4 +--
 drivers/iio/accel/mxc6255.c                |  4 +--
 drivers/iio/accel/st_accel_i2c.c           |  5 +---
 drivers/iio/accel/stk8ba50.c               |  4 +--
 drivers/iio/adc/ti-adc108s102.c            |  4 +--
 drivers/iio/gyro/bmg160_i2c.c              |  4 +--
 drivers/iio/humidity/hts221_i2c.c          |  4 +--
 drivers/iio/imu/fxos8700_i2c.c             |  3 +-
 drivers/iio/imu/fxos8700_spi.c             |  3 +-
 drivers/iio/imu/kmx61.c                    |  2 +-
 drivers/iio/light/jsa1212.c                |  4 +--
 drivers/iio/light/ltr501.c                 |  3 +-
 drivers/iio/light/max44000.c               |  6 ++--
 drivers/iio/light/rpr0521.c                |  4 +--
 drivers/iio/light/stk3310.c                |  4 +--
 drivers/iio/light/us5182d.c                |  4 +--
 drivers/iio/magnetometer/bmc150_magn_i2c.c |  3 +-
 drivers/iio/magnetometer/bmc150_magn_spi.c |  3 +-
 drivers/iio/magnetometer/mmc35240.c        |  4 +--
 drivers/iio/potentiometer/max5487.c        |  4 +--
 drivers/iio/pressure/hp206c.c              |  6 ++--
 drivers/iio/pressure/st_pressure_i2c.c     |  5 +---
 28 files changed, 60 insertions(+), 75 deletions(-)

-- 
2.43.0





[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux