Attachment 131451 Details for Bug 96868 – possible fix 2/4

[patch] possible fix 2/4

0002-drm-radeon-ci-disable-mclk-switching-for-high-refres.patch (text/plain), 1.15 KB, created by Alex Deucher on 2017-05-23 21:24:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Deucher

Created: 2017-05-23 21:24:55 UTC

Size: 1.15 KB

patch

obsolete

>From cbdf98c05644edec4b7a648b3c83d3a1c257ac47 Mon Sep 17 00:00:00 2001
>From: Alex Deucher <alexander.deucher@amd.com>
>Date: Thu, 11 May 2017 13:14:14 -0400
>Subject: [PATCH 2/4] drm/radeon/ci: disable mclk switching for high refresh
> rates (v2)
>
>Even if the vblank period would allow it, it still seems to
>be problematic on some cards.
>
>v2: fix logic inversion (Nils)
>
>bug: https://bugs.freedesktop.org/show_bug.cgi?id=96868
>
>Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
>---
> drivers/gpu/drm/radeon/ci_dpm.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
>diff --git a/drivers/gpu/drm/radeon/ci_dpm.c b/drivers/gpu/drm/radeon/ci_dpm.c
>index 7ba4508..ea36dc4 100644
>--- a/drivers/gpu/drm/radeon/ci_dpm.c
>+++ b/drivers/gpu/drm/radeon/ci_dpm.c
>@@ -776,6 +776,12 @@ bool ci_dpm_vblank_too_short(struct radeon_device *rdev)
> 	u32 vblank_time = r600_dpm_get_vblank_time(rdev);
> 	u32 switch_limit = pi->mem_gddr5 ? 450 : 300;
> 
>+	/* disable mclk switching if the refresh is >120Hz, even if the
>+        * blanking period would allow it
>+        */
>+	if (r600_dpm_get_vrefresh(rdev) > 120)
>+		return true;
>+
> 	if (vblank_time < switch_limit)
> 		return true;
> 	else
>-- 
>2.5.5
>

Actions: View | Splinter Review

Attachments on bug 96868: 125006 | 125007 | 125011 | 131319 | 131320 | 131321 | 131322 | 131450 | 131451 | 131452 | 131453 | 142660

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.