mbox series

[0/6] Small Runtime PM API changes

Message ID 20231115181840.1509046-1-sakari.ailus@linux.intel.com (mailing list archive)
Headers show
Series Small Runtime PM API changes | expand

Message

Sakari Ailus Nov. 15, 2023, 6:18 p.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.


Sakari Ailus (6):
  pm: runtime: Simplify pm_runtime_get_if_active() usage
  pm: runtime: Add pm_runtime_put_mark_busy_autosusp() helper
  media: Documentation: Improve camera sensor runtime PM documentation
  media: ov8858: Use pm_runtime_get_if_active(), put usage_count
    correctly
  media: imx319: Use pm_runtime_get_if_active(), put usage_count
    correctly
  media: imx219: Use pm_runtime_get_if_active(), put usage_count
    correctly

 .../driver-api/media/camera-sensor.rst        | 28 ++++++++---
 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                    | 48 +++++++++++++++++--
 sound/hda/hdac_device.c                       |  2 +-
 12 files changed, 92 insertions(+), 32 deletions(-)


base-commit: 3e238417254bfdcc23fe207780b59cbb08656762