Attachment 96900 Details for Bug 77009 – Possible fix.

[patch] Possible fix.

0001-drm-radeon-apply-more-strict-limits-for-PLL-params.patch (text/plain), 1.08 KB, created by Christian König on 2014-04-04 11:59:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Christian König

Created: 2014-04-04 11:59:27 UTC

Size: 1.08 KB

patch

obsolete

>From 146b3e334cdbfa4bf5b5d3a4162f7b677356773d Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Christian=20K=C3=B6nig?= <christian.koenig@amd.com>
>Date: Fri, 4 Apr 2014 13:45:42 +0200
>Subject: [PATCH] drm/radeon: apply more strict limits for PLL params
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>Letting post and refernce divider get to big is bad for signal stability.
>
>Signed-off-by: Christian KÃ¶nig <christian.koenig@amd.com>
>---
> drivers/gpu/drm/radeon/radeon_display.c | 3 +++
> 1 file changed, 3 insertions(+)
>
>diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c
>index 386cfa4..d68863e 100644
>--- a/drivers/gpu/drm/radeon/radeon_display.c
>+++ b/drivers/gpu/drm/radeon/radeon_display.c
>@@ -937,6 +937,9 @@ void radeon_compute_pll_avivo(struct radeon_pll *pll,
> 	}
> 	post_div = post_div_best;
> 
>+	/* limit reference * post divider to a maximum */
>+	ref_div_max = min(100 / post_div, ref_div_max);
>+
> 	/* get matching reference and feedback divider */
> 	ref_div = max(den / post_div, 1u);
> 	fb_div = nom;
>-- 
>1.9.1
>

Actions: View | Splinter Review

Attachments on bug 77009: 96854 | 96900 | 96914 | 96915 | 96962 | 96963 | 96966 | 96968 | 97003 | 97004 | 97006 | 97043

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.