drm/i915/dp: Use auxch precharge value of 5 everywhere
authorAdam Jackson <ajax@redhat.com>
Tue, 26 Jul 2011 19:39:45 +0000 (15:39 -0400)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 17 Jan 2012 15:46:56 +0000 (16:46 +0100)
The default in the Sandybridge docs is 5, as on Ironlake, and I have no
reason to believe 3 would work any better.

Signed-off-by: Adam Jackson <ajax@redhat.com>
Acked-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_dp.c

index add871911a6389c75a56df83ef44ca22b6d4c981..2f4766385797787f57b909acff84b66941893f7f 100644 (file)
@@ -362,7 +362,7 @@ intel_dp_aux_ch(struct intel_dp *intel_dp,
        int recv_bytes;
        uint32_t status;
        uint32_t aux_clock_divider;
-       int try, precharge;
+       int try, precharge = 5;
 
        intel_dp_check_edp(intel_dp);
        /* The clock divider is based off the hrawclk,
@@ -382,11 +382,6 @@ intel_dp_aux_ch(struct intel_dp *intel_dp,
        else
                aux_clock_divider = intel_hrawclk(dev) / 2;
 
-       if (IS_GEN6(dev))
-               precharge = 3;
-       else
-               precharge = 5;
-
        /* Try to wait for any previous AUX channel activity */
        for (try = 0; try < 3; try++) {
                status = I915_READ(ch_ctl);