drm/i915/eDP: compute the panel power clock divisor from the pch rawclock
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Sat, 20 Oct 2012 18:57:43 +0000 (20:57 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 24 Oct 2012 15:54:36 +0000 (17:54 +0200)
We need this when the bios forgets even to set that bit up. Most seem
to do that, even when they don't set up anything else in the panel
power sequencer.

Note that on IBX the rawclk is variable according to Bspec, but
everyone is using 125MHz. The rawclk is fixed to 125MHz on CPT, but
luckily we still have the same register available. On hsw, different
variants have different clocks, hence we need to check the register.

Since other pieces are driven by the rawclock, too, keep the little
helper in a central place.

Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Tested-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_display.c
drivers/gpu/drm/i915/intel_dp.c
drivers/gpu/drm/i915/intel_drv.h

index 338570c8e0dc5030f6fb7d73e4e13712a3152351..0fb55429bb952d38b361dc6b2e853af30890599a 100644 (file)
@@ -80,6 +80,16 @@ struct intel_limit {
 /* FDI */
 #define IRONLAKE_FDI_FREQ              2700000 /* in kHz for mode->clock */
 
+int
+intel_pch_rawclk(struct drm_device *dev)
+{
+       struct drm_i915_private *dev_priv = dev->dev_private;
+
+       WARN_ON(!HAS_PCH_SPLIT(dev));
+
+       return I915_READ(PCH_RAWCLK_FREQ) & RAWCLK_FREQ_MASK;
+}
+
 static bool
 intel_find_best_PLL(const intel_limit_t *limit, struct drm_crtc *crtc,
                    int target, int refclk, intel_clock_t *match_clock,
index da526c9a9ceff268741daccce01a12351c5257e4..adfb98cb7ba93b5bc5b62148792b4f651a07ffd9 100644 (file)
@@ -2711,8 +2711,12 @@ intel_dp_init(struct drm_device *dev, int output_reg, enum port port)
                        (final.t8 << PANEL_LIGHT_ON_DELAY_SHIFT);
                pp_off = (final.t9 << PANEL_LIGHT_OFF_DELAY_SHIFT) |
                         (final.t10 << PANEL_POWER_DOWN_DELAY_SHIFT);
-               pp_div = (pp_div & PP_REFERENCE_DIVIDER_MASK) |
-                        (DIV_ROUND_UP(final.t11_t12, 1000) << PANEL_POWER_CYCLE_DELAY_SHIFT);
+               /* Compute the divisor for the pp clock, simply match the Bspec
+                * formula. */
+               pp_div = ((100 * intel_pch_rawclk(dev))/2 - 1)
+                               << PP_REFERENCE_DIVIDER_SHIFT;
+               pp_div |= (DIV_ROUND_UP(final.t11_t12, 1000)
+                               << PANEL_POWER_CYCLE_DELAY_SHIFT);
 
                /* Haswell doesn't have any port selection bits for the panel
                 * power sequence any more. */
index 3b590baad96f4b662be38d2b6af84ebcd58372cf..c2e439b0d98340ef8b279870ce64e8d799656ec9 100644 (file)
@@ -405,6 +405,8 @@ struct intel_fbc_work {
        int interval;
 };
 
+int intel_pch_rawclk(struct drm_device *dev);
+
 int intel_connector_update_modes(struct drm_connector *connector,
                                struct edid *edid);
 int intel_ddc_get_modes(struct drm_connector *c, struct i2c_adapter *adapter);