summaryrefslogtreecommitdiffstats
path: root/include/sound
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2019-02-13 15:21:09 +0000
committerChris Wilson <chris@chris-wilson.co.uk>2019-02-14 21:04:47 +0000
commitd31c85fc864269ab8e6fb6ed36c87f2fc33a9430 (patch)
tree12bbfb4655f15035c96ed37c6fde897a386f49c7 /include/sound
parent290248c27c93ad70262b8112595b95ad9d867929 (diff)
downloadlinux-d31c85fc864269ab8e6fb6ed36c87f2fc33a9430.tar.gz
linux-d31c85fc864269ab8e6fb6ed36c87f2fc33a9430.tar.bz2
linux-d31c85fc864269ab8e6fb6ed36c87f2fc33a9430.zip
snd/hda, drm/i915: Track the display_power_status using a cookie
drm/i915 is tracking all wakeref owners with a cookie in order to identify leaks. To that end, each rpm acquisition ops->get_power is assigned a cookie which should be passed to ops->put_power to signify its release (and removal from the list of wakeref owners). As snd/hda is already using a bool to track current status of display_power extending that to an unsigned long to hold the boolean cookie is a trivial extension, and will quell all doubt that snd/hda is the cause of the device runtime pm leaks. v2: Keep using the power abstraction for local wakeref tracking. v3: BUILD_BUG_ON impedance mismatch Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Takashi Iwai <tiwai@suse.de> Cc: Jani Nikula <jani.nikula@intel.com> Acked-by: Takashi Iwai <tiwai@suse.de> Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20190213152109.16997-1-chris@chris-wilson.co.uk
Diffstat (limited to 'include/sound')
-rw-r--r--include/sound/hdaudio.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/include/sound/hdaudio.h b/include/sound/hdaudio.h
index b4fa1c775251..a438ec8e535b 100644
--- a/include/sound/hdaudio.h
+++ b/include/sound/hdaudio.h
@@ -367,7 +367,7 @@ struct hdac_bus {
/* DRM component interface */
struct drm_audio_component *audio_component;
long display_power_status;
- bool display_power_active;
+ unsigned long display_power_active;
/* parameters required for enhanced capabilities */
int num_streams;