Attachment 74606 Details for Bug 49981 – fix multi-head stability

[patch] fix multi-head stability

0001-drm-radeon-fix-multi-head-power-profile-stability-on.patch (text/plain), 2.13 KB, created by Alex Deucher on 2013-02-11 13:49:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Deucher

Created: 2013-02-11 13:49:16 UTC

Size: 2.13 KB

patch

obsolete

>From 0b64279e289ea0876d12ec5e5b91f9d076675552 Mon Sep 17 00:00:00 2001
>From: Alex Deucher <alexander.deucher@amd.com>
>Date: Mon, 11 Feb 2013 08:44:48 -0500
>Subject: [PATCH] drm/radeon: fix multi-head power profile stability on BTC+
> asics
>
>vddci needs to track mclk for multi-head.
>
>Fixes:
>https://bugs.freedesktop.org/show_bug.cgi?id=49981
>
>Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
>Cc: stable@vger.kernel.org
>---
> drivers/gpu/drm/radeon/evergreen.c |   13 +++++++++++++
> drivers/gpu/drm/radeon/radeon_pm.c |    2 +-
> 2 files changed, 14 insertions(+), 1 deletions(-)
>
>diff --git a/drivers/gpu/drm/radeon/evergreen.c b/drivers/gpu/drm/radeon/evergreen.c
>index fe10800..c9ee5d5 100644
>--- a/drivers/gpu/drm/radeon/evergreen.c
>+++ b/drivers/gpu/drm/radeon/evergreen.c
>@@ -700,6 +700,19 @@ void evergreen_pm_misc(struct radeon_device *rdev)
> 			rdev->pm.current_vddc = voltage->voltage;
> 			DRM_DEBUG("Setting: vddc: %d\n", voltage->voltage);
> 		}
>+
>+		/* starting with BTC, there is one state that is used for both
>+		 * MH and SH.  Difference is that we always use the high clock index for
>+		 * mclk and vddci.
>+		 */
>+		if ((rdev->pm.pm_method == PM_METHOD_PROFILE) &&
>+		    (rdev->family >= CHIP_BARTS) &&
>+		    rdev->pm.active_crtc_count &&
>+		    ((rdev->pm.profile_index == PM_PROFILE_MID_MH_IDX) ||
>+		     (rdev->pm.profile_index == PM_PROFILE_LOW_MH_IDX)))
>+			voltage = &rdev->pm.power_state[req_ps_idx].
>+				clock_info[rdev->pm.profiles[PM_PROFILE_HIGH_MH_IDX].dpms_on_cm_idx].voltage;
>+
> 		/* 0xff01 is a flag rather then an actual voltage */
> 		if (voltage->vddci == 0xff01)
> 			return;
>diff --git a/drivers/gpu/drm/radeon/radeon_pm.c b/drivers/gpu/drm/radeon/radeon_pm.c
>index 4182814..5058968 100644
>--- a/drivers/gpu/drm/radeon/radeon_pm.c
>+++ b/drivers/gpu/drm/radeon/radeon_pm.c
>@@ -169,7 +169,7 @@ static void radeon_set_power_state(struct radeon_device *rdev)
> 
> 		/* starting with BTC, there is one state that is used for both
> 		 * MH and SH.  Difference is that we always use the high clock index for
>-		 * mclk.
>+		 * mclk and vddci.
> 		 */
> 		if ((rdev->pm.pm_method == PM_METHOD_PROFILE) &&
> 		    (rdev->family >= CHIP_BARTS) &&
>-- 
>1.7.7.5
>

Actions: View | Splinter Review

Attachments on bug 49981: 62305 | 67892 | 67893 | 67903 | 67937 | 67953 | 74554 | 74606

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.