[v2,0/7] Small Runtime PM API changes

Message

Sakari Ailus Nov. 17, 2023, 11:14 a.m. UTC

Hi folks,

This small set happily mixes Runtime PM and media patches.

The set does two main things Runtime PM API-wise. Firstly,
pm_runtime_get_if_active() is made more user-friendly by removing the
ign_use_count argument so the users no longer need to call it with that
set to true. Secondly, pm_runtime_put_mark_busy_autosusp() helper is added
to avoid drivers having to call pm_runtime_mark_last_busy() only to be
followed by pm_runtime_autosuspend().

The vast majority of the users of pm_runtime_autosuspend() would probably
have been fine with making pm_runtime_autosuspend() do the last busy
stamping, too, but given the sheer number of users it's hard to tell if
there could be problems here and there. On the other hand, there are
probably a sizable proportion of call sites where the missing
pm_runtime_mark_last_busy() call is simply a bug.

The three last patches are addressing Runtime PM issues in a few sensor
drivers.

Comments would be welcome.

since v1:

- Fix a compilation issue when CONFIG_PM is disabled in the first patch.

- Improve the documentation patch, assume the use of autosuspend (this
  generally makes sense for camera sensor drivers).

- Keep using pm_runtime_get_if_in_use() in imx319 and imx219 drivers (they
  don't use autosuspend).

- Add a patch to document acpi_dev_state_d0() in conjunction of non-D0
  probe.

Sakari Ailus (7):
  pm: runtime: Simplify pm_runtime_get_if_active() usage
  pm: runtime: Add pm_runtime_put_mark_busy_autosusp() helper
  ACPI: Documentation: Document acpi_dev_state_d0()
  media: Documentation: Improve camera sensor runtime PM documentation
  media: ov8858: Use pm_runtime_get_if_active(), put usage_count
    correctly
  media: imx319: Put usage_count correctly in s_ctrl callback
  media: imx219: Put usage_count correctly in s_ctrl callback

 .../driver-api/media/camera-sensor.rst        | 76 +++++++++++++------
 .../firmware-guide/acpi/non-d0-probe.rst      | 10 +++
 Documentation/power/runtime_pm.rst            |  5 +-
 drivers/base/power/runtime.c                  |  9 ++-
 drivers/gpu/drm/i915/intel_runtime_pm.c       |  2 +-
 drivers/media/i2c/ccs/ccs-core.c              |  2 +-
 drivers/media/i2c/imx219.c                    |  8 +-
 drivers/media/i2c/imx319.c                    |  8 +-
 drivers/media/i2c/ov8858.c                    |  8 +-
 drivers/net/ipa/ipa_smp2p.c                   |  2 +-
 drivers/pci/pci.c                             |  2 +-
 include/linux/pm_runtime.h                    | 49 ++++++++++--
 sound/hda/hdac_device.c                       |  2 +-
 13 files changed, 133 insertions(+), 50 deletions(-)


base-commit: 3e238417254bfdcc23fe207780b59cbb08656762