mbox series

[00/18] hwmon: Move to new PM macros reducing driver complexity

Message ID 20220925172759.3573439-1-jic23@kernel.org (mailing list archive)
Headers show
Series hwmon: Move to new PM macros reducing driver complexity | expand

Message

Jonathan Cameron Sept. 25, 2022, 5:27 p.m. UTC
From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

Recently a solution was added to avoiding the need to either guard
pm functions with #ifdef magic, or mark the __maybe_unused.
https://lore.kernel.org/all/20220107181723.54392-1-paul@crapouillou.net/

This series switches hwmon over to the new macros that are intended to replace
SIMPLE_DEV_PM_OPS and similar.

There are a few drivers in hwmon that might be able fine using
DEFINE_SIMPLE_PM_OPS() but currently do not define as many of the
callbacks as that macro does. As such I haven't touched them in this set.

This is part of general effort to get rid of examples of the older macros
that might get copied in new drivers.

Jonathan Cameron (18):
  hwmon: (abitguru) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (abitguru3) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (acpi_power_meter) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (adt7x10) Switch to EXPORT_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (gpio-fan) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (it87) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and pm_sleep_ptr()
  hwmon: (lm90) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and pm_sleep_ptr()
  hwmon: (ltc2947) Switch to EXPORT_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (max31722) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (max31730) witch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (max6639) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (nct6775) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (pwm-fan) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (tmp102) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (tmp103) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (tmp108) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (w83627ehf) Switch to DEFINE_SIMPLE_DEV_PM_OPS() and
    pm_sleep_ptr()
  hwmon: (ina3221) Use DEFINE_RUNTIME_DEV_PM_OPS() and pm_ptr()

 drivers/hwmon/abituguru.c        |  9 ++-------
 drivers/hwmon/abituguru3.c       |  9 ++-------
 drivers/hwmon/acpi_power_meter.c |  9 +++------
 drivers/hwmon/adt7310.c          |  2 +-
 drivers/hwmon/adt7410.c          |  2 +-
 drivers/hwmon/adt7x10.c          |  7 +------
 drivers/hwmon/adt7x10.h          |  5 -----
 drivers/hwmon/gpio-fan.c         |  9 ++-------
 drivers/hwmon/ina3221.c          | 13 +++++--------
 drivers/hwmon/it87.c             |  8 ++++----
 drivers/hwmon/lm90.c             |  8 ++++----
 drivers/hwmon/ltc2947-core.c     |  7 +++----
 drivers/hwmon/ltc2947-i2c.c      |  2 +-
 drivers/hwmon/ltc2947-spi.c      |  2 +-
 drivers/hwmon/max31722.c         |  8 ++++----
 drivers/hwmon/max31730.c         |  8 ++++----
 drivers/hwmon/max6639.c          |  6 ++----
 drivers/hwmon/nct6775-platform.c |  8 ++++----
 drivers/hwmon/pwm-fan.c          |  6 ++----
 drivers/hwmon/tmp102.c           |  6 ++----
 drivers/hwmon/tmp103.c           |  8 ++++----
 drivers/hwmon/tmp108.c           |  8 ++++----
 drivers/hwmon/w83627ehf.c        |  8 ++++----
 23 files changed, 60 insertions(+), 98 deletions(-)

Comments

Guenter Roeck Sept. 25, 2022, 5:47 p.m. UTC | #1
On 9/25/22 10:27, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> Recently a solution was added to avoiding the need to either guard
> pm functions with #ifdef magic, or mark the __maybe_unused.
> https://lore.kernel.org/all/20220107181723.54392-1-paul@crapouillou.net/
> 
> This series switches hwmon over to the new macros that are intended to replace
> SIMPLE_DEV_PM_OPS and similar.
> 
> There are a few drivers in hwmon that might be able fine using
> DEFINE_SIMPLE_PM_OPS() but currently do not define as many of the
> callbacks as that macro does. As such I haven't touched them in this set.
> 
> This is part of general effort to get rid of examples of the older macros
> that might get copied in new drivers.

Series applied to hwmon-next.

Thanks,
Guenter