<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW --- - On HD6850, Power Profile doesn't change if 2 screen is attached."
href="https://bugs.freedesktop.org/show_bug.cgi?id=49981#c33">Comment # 33</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW --- - On HD6850, Power Profile doesn't change if 2 screen is attached."
href="https://bugs.freedesktop.org/show_bug.cgi?id=49981">bug 49981</a>
from <span class="vcard"><a class="email" href="mailto:ben@b1c1l1.com" title="Benjamin Lee <ben@b1c1l1.com>"> <span class="fn">Benjamin Lee</span></a>
</span></b>
<pre>Sorry, my previous patch was bogus -- TURKS mobility actually needs
POWER_STATE_TYPE_PERFORMANCE. radeon_pm_get_type_index() currently fails to
find the requested state (POWER_STATE_TYPE_BATTERY).
Here is a trivial patch against linux-3.8.8:
diff -ruN linux-3.8.8.orig/drivers/gpu/drm/radeon/evergreen.c
linux-3.8.8/drivers/gpu/drm/radeon/evergreen.c
--- linux-3.8.8.orig/drivers/gpu/drm/radeon/evergreen.c 2013-04-16
22:11:28.000000000 -0700
+++ linux-3.8.8/drivers/gpu/drm/radeon/evergreen.c 2013-04-17
16:51:04.858752564 -0700
@@ -343,7 +343,7 @@
* MH and SH. Difference is that we always use the high clock index
for
* mclk.
*/
- if (rdev->flags & RADEON_IS_MOBILITY)
+ if ((rdev->flags & RADEON_IS_MOBILITY) && (rdev->family != CHIP_TURKS))
idx = radeon_pm_get_type_index(rdev, POWER_STATE_TYPE_BATTERY,
0);
else
idx = radeon_pm_get_type_index(rdev,
POWER_STATE_TYPE_PERFORMANCE, 0);
Running with the patch in multi-head shows reduced engine clock and voltage,
but high memory clock as expected:
blee@supra ~ $ cat /sys/class/drm/card0/device/power_method
profile
blee@supra ~ $ cat /sys/class/drm/card0/device/power_profile
low
blee@supra ~ $ sudo cat /sys/kernel/debug/dri/0/radeon_pm_info
default engine clock: 600000 kHz
current engine clock: 99990 kHz
default memory clock: 793750 kHz
current memory clock: 793750 kHz
voltage: 900 mV</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>