diff options
author | Paulo Zanoni <paulo.r.zanoni@intel.com> | 2014-07-09 13:31:57 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-07-10 02:30:06 -0400 |
commit | 03872064f77f4beb61d1afc22adf62da47cf087b (patch) | |
tree | 2cb026394ae1f2b9409fb86746267fcedf7ff479 | |
parent | 9ccd5aeb2901aaaabf77f185e2e5ea3a0b577e86 (diff) |
drm/i915: get/put runtime PM at i915_semaphore_status
Otherwise we will print some WARNs when we read registers and the
machine is suspended.
Testcase: igt/pm_rpm/debugfs-read
Cc: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Reviewed-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
-rw-r--r-- | drivers/gpu/drm/i915/i915_debugfs.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c index 394519182510..952f19949be1 100644 --- a/drivers/gpu/drm/i915/i915_debugfs.c +++ b/drivers/gpu/drm/i915/i915_debugfs.c | |||
@@ -2314,6 +2314,7 @@ static int i915_semaphore_status(struct seq_file *m, void *unused) | |||
2314 | ret = mutex_lock_interruptible(&dev->struct_mutex); | 2314 | ret = mutex_lock_interruptible(&dev->struct_mutex); |
2315 | if (ret) | 2315 | if (ret) |
2316 | return ret; | 2316 | return ret; |
2317 | intel_runtime_pm_get(dev_priv); | ||
2317 | 2318 | ||
2318 | if (IS_BROADWELL(dev)) { | 2319 | if (IS_BROADWELL(dev)) { |
2319 | struct page *page; | 2320 | struct page *page; |
@@ -2363,6 +2364,7 @@ static int i915_semaphore_status(struct seq_file *m, void *unused) | |||
2363 | } | 2364 | } |
2364 | seq_putc(m, '\n'); | 2365 | seq_putc(m, '\n'); |
2365 | 2366 | ||
2367 | intel_runtime_pm_put(dev_priv); | ||
2366 | mutex_unlock(&dev->struct_mutex); | 2368 | mutex_unlock(&dev->struct_mutex); |
2367 | return 0; | 2369 | return 0; |
2368 | } | 2370 | } |