[Bug 49981] On HD6850, Power Profile doesn't change if 2 screen is attached.

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Fri Apr 5 17:51:55 PDT 2013


https://bugs.freedesktop.org/show_bug.cgi?id=49981

--- Comment #32 from Benjamin Lee <ben at b1c1l1.com> ---
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);

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130406/2db48b4e/attachment.html>


More information about the dri-devel mailing list