drm/radeon/kms: remove some pll algo flags
authorAlex Deucher <alexdeucher@gmail.com>
Wed, 29 Sep 2010 15:37:40 +0000 (11:37 -0400)
committerDave Airlie <airlied@redhat.com>
Wed, 6 Oct 2010 01:46:24 +0000 (11:46 +1000)
These shouldn't be needed with the post div changes
in the last patch.

Signed-off-by: Alex Deucher <alexdeucher@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/radeon/atombios_crtc.c
drivers/gpu/drm/radeon/radeon_display.c
drivers/gpu/drm/radeon/radeon_legacy_crtc.c
drivers/gpu/drm/radeon/radeon_mode.h

index cd0290f946cff51e8aa8c702dda5e650ec5f9af0..ca04a1bdb75b3121a01bcab69c1f4bba94213b82 100644 (file)
@@ -501,21 +501,9 @@ static u32 atombios_adjust_pll(struct drm_crtc *crtc,
                    (rdev->family == CHIP_RS740))
                        pll->flags |= (/*RADEON_PLL_USE_FRAC_FB_DIV |*/
                                       RADEON_PLL_PREFER_CLOSEST_LOWER);
-
-               if (ASIC_IS_DCE32(rdev) && mode->clock > 200000)        /* range limits??? */
-                       pll->flags |= RADEON_PLL_PREFER_HIGH_FB_DIV;
-               else
-                       pll->flags |= RADEON_PLL_PREFER_LOW_REF_DIV;
-       } else {
+       } else
                pll->flags |= RADEON_PLL_LEGACY;
 
-               if (mode->clock > 200000)       /* range limits??? */
-                       pll->flags |= RADEON_PLL_PREFER_HIGH_FB_DIV;
-               else
-                       pll->flags |= RADEON_PLL_PREFER_LOW_REF_DIV;
-
-       }
-
        list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
                if (encoder->crtc == crtc) {
                        radeon_encoder = to_radeon_encoder(encoder);
index d276d6d8e2b000c7cd0a3de64c4d54da37c2fa80..20464659d3fad458dba90cc308af62f5d2fed989 100644 (file)
@@ -579,7 +579,8 @@ static void radeon_compute_pll_legacy(struct radeon_pll *pll,
                                        if ((best_vco == 0 && error < best_error) ||
                                            (best_vco != 0 &&
                                             ((best_error > 100 && error < best_error - 100) ||
-                                             (abs(error - best_error) < 100 && vco_diff < best_vco_diff)))) {
+                                             (abs(error - best_error) < 100 &&
+                                              vco_diff < best_vco_diff)))) {
                                                best_post_div = post_div;
                                                best_ref_div = ref_div;
                                                best_feedback_div = feedback_div;
@@ -587,29 +588,6 @@ static void radeon_compute_pll_legacy(struct radeon_pll *pll,
                                                best_freq = current_freq;
                                                best_error = error;
                                                best_vco_diff = vco_diff;
-                                       } else if (current_freq == freq) {
-                                               if (best_freq == -1) {
-                                                       best_post_div = post_div;
-                                                       best_ref_div = ref_div;
-                                                       best_feedback_div = feedback_div;
-                                                       best_frac_feedback_div = frac_feedback_div;
-                                                       best_freq = current_freq;
-                                                       best_error = error;
-                                                       best_vco_diff = vco_diff;
-                                               } else if (((pll->flags & RADEON_PLL_PREFER_LOW_REF_DIV) && (ref_div < best_ref_div)) ||
-                                                          ((pll->flags & RADEON_PLL_PREFER_HIGH_REF_DIV) && (ref_div > best_ref_div)) ||
-                                                          ((pll->flags & RADEON_PLL_PREFER_LOW_FB_DIV) && (feedback_div < best_feedback_div)) ||
-                                                          ((pll->flags & RADEON_PLL_PREFER_HIGH_FB_DIV) && (feedback_div > best_feedback_div)) ||
-                                                          ((pll->flags & RADEON_PLL_PREFER_LOW_POST_DIV) && (post_div < best_post_div)) ||
-                                                          ((pll->flags & RADEON_PLL_PREFER_HIGH_POST_DIV) && (post_div > best_post_div))) {
-                                                       best_post_div = post_div;
-                                                       best_ref_div = ref_div;
-                                                       best_feedback_div = feedback_div;
-                                                       best_frac_feedback_div = frac_feedback_div;
-                                                       best_freq = current_freq;
-                                                       best_error = error;
-                                                       best_vco_diff = vco_diff;
-                                               }
                                        }
                                        if (current_freq < freq)
                                                min_frac_feed_div = frac_feedback_div + 1;
index 305049afde15235e558dc22f3992543508f362e6..d60b319828451c315324897474e20fa1697af182 100644 (file)
@@ -722,11 +722,6 @@ static void radeon_set_pll(struct drm_crtc *crtc, struct drm_display_mode *mode)
        else
                pll->algo = PLL_ALGO_LEGACY;
 
-       if (mode->clock > 200000) /* range limits??? */
-               pll->flags |= RADEON_PLL_PREFER_HIGH_FB_DIV;
-       else
-               pll->flags |= RADEON_PLL_PREFER_LOW_REF_DIV;
-
        list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
                if (encoder->crtc == crtc) {
                        struct radeon_encoder *radeon_encoder = to_radeon_encoder(encoder);
index 8e071bf5e250dc6e2cb472a3429dab828515ecf2..8707cd61e58b61345e0bb609f542a5d6f930bd38 100644 (file)
@@ -139,16 +139,10 @@ struct radeon_tmds_pll {
 #define RADEON_PLL_NO_ODD_POST_DIV      (1 << 1)
 #define RADEON_PLL_USE_REF_DIV          (1 << 2)
 #define RADEON_PLL_LEGACY               (1 << 3)
-#define RADEON_PLL_PREFER_LOW_REF_DIV   (1 << 4)
-#define RADEON_PLL_PREFER_HIGH_REF_DIV  (1 << 5)
-#define RADEON_PLL_PREFER_LOW_FB_DIV    (1 << 6)
-#define RADEON_PLL_PREFER_HIGH_FB_DIV   (1 << 7)
-#define RADEON_PLL_PREFER_LOW_POST_DIV  (1 << 8)
-#define RADEON_PLL_PREFER_HIGH_POST_DIV (1 << 9)
-#define RADEON_PLL_USE_FRAC_FB_DIV      (1 << 10)
-#define RADEON_PLL_PREFER_CLOSEST_LOWER (1 << 11)
-#define RADEON_PLL_USE_POST_DIV         (1 << 12)
-#define RADEON_PLL_IS_LCD               (1 << 13)
+#define RADEON_PLL_USE_FRAC_FB_DIV      (1 << 4)
+#define RADEON_PLL_PREFER_CLOSEST_LOWER (1 << 5)
+#define RADEON_PLL_USE_POST_DIV         (1 << 6)
+#define RADEON_PLL_IS_LCD               (1 << 7)
 
 /* pll algo */
 enum radeon_pll_algo {