<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#c32">Comment # 32</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>On my TURKS chip (HD 6750M) the radeon_pm_print_states() output shows that I
should be using POWER_STATE_TYPE_POWERSAVE instead of POWER_STATE_TYPE_BATTERY.
I don't know if this is true for all TURKS chips or what other chips are
affected, but the following patch against 3.9-rc5 fixes power profiles on my
MacBookPro8,2.
diff -ruN linux-3.9-rc5.orig/drivers/gpu/drm/radeon/evergreen.c
linux-3.9-rc5/drivers/gpu/drm/radeon/evergreen.c
--- linux-3.9-rc5.orig/drivers/gpu/drm/radeon/evergreen.c 2013-03-31
15:12:43.000000000 -0700
+++ linux-3.9-rc5/drivers/gpu/drm/radeon/evergreen.c 2013-04-05
17:45:04.179779729 -0700
@@ -343,7 +343,9 @@
* 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_POWERSAVE, 0);
+ else if (rdev->flags & RADEON_IS_MOBILITY)
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);</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>