Commit f8a2645e authored by Christian König's avatar Christian König

drm/radeon: improve PLL params if we don't match exactly v2

Otherwise we might be quite off on older chipsets.

v2: keep ref_div minimum
Signed-off-by: default avatarChristian König <christian.koenig@amd.com>
parent 74073c9d
...@@ -865,7 +865,7 @@ void radeon_compute_pll_avivo(struct radeon_pll *pll, ...@@ -865,7 +865,7 @@ void radeon_compute_pll_avivo(struct radeon_pll *pll,
unsigned post_div_min, post_div_max, post_div; unsigned post_div_min, post_div_max, post_div;
unsigned ref_div_min, ref_div_max, ref_div; unsigned ref_div_min, ref_div_max, ref_div;
unsigned post_div_best, diff_best; unsigned post_div_best, diff_best;
unsigned nom, den, tmp; unsigned nom, den;
/* determine allowed feedback divider range */ /* determine allowed feedback divider range */
fb_div_min = pll->min_feedback_div; fb_div_min = pll->min_feedback_div;
...@@ -941,22 +941,23 @@ void radeon_compute_pll_avivo(struct radeon_pll *pll, ...@@ -941,22 +941,23 @@ void radeon_compute_pll_avivo(struct radeon_pll *pll,
ref_div_max = min(210 / post_div, ref_div_max); ref_div_max = min(210 / post_div, ref_div_max);
/* get matching reference and feedback divider */ /* get matching reference and feedback divider */
ref_div = max(den / post_div, 1u); ref_div = max(DIV_ROUND_CLOSEST(den, post_div), 1u);
fb_div = nom; fb_div = DIV_ROUND_CLOSEST(nom * ref_div * post_div, den);
/* we're almost done, but reference and feedback /* we're almost done, but reference and feedback
divider might be to large now */ divider might be to large now */
tmp = ref_div; nom = fb_div;
den = ref_div;
if (fb_div > fb_div_max) { if (fb_div > fb_div_max) {
ref_div = ref_div * fb_div_max / fb_div; ref_div = DIV_ROUND_CLOSEST(den * fb_div_max, nom);
fb_div = fb_div_max; fb_div = fb_div_max;
} }
if (ref_div > ref_div_max) { if (ref_div > ref_div_max) {
ref_div = ref_div_max; ref_div = ref_div_max;
fb_div = nom * ref_div_max / tmp; fb_div = DIV_ROUND_CLOSEST(nom * ref_div_max, den);
} }
/* reduce the numbers to a simpler ratio once more */ /* reduce the numbers to a simpler ratio once more */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment