Commit 64b46a06 authored by Ville Syrjälä's avatar Ville Syrjälä

drm/i915: Make the LPT iclkip 20MHz case more generic

The reason for spcial casing 20MHz in the iclkip calculations is that
it would overflow the 7 bit divisor value. Let's rewrite the special
case to check for just that, and bump up auxdiv when needed. This makes
the code work for freqeuencies close to but not exactly 20MHz. The real
lower limit for auxdiv=0 is actually:
172800000/(0x7f+2)*64)=~20930 kHz, and below that we must resort to
auxdiv=1.

Actually this is all very theoretical since we limit the dotclock to
min 25MHz with CRT on all platforms. 25Mhz is actually the documented
limit in Bspec, so it seems we ought to never need to worry about the
auxdiv=1 case. But no harm in having it.

Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1455738073-14502-5-git-send-email-ville.syrjala@linux.intel.comReviewed-by: default avatarPaulo Zanoni <paulo.r.zanoni@intel.com>
Reviewed-by: default avatarImre Deak <imre.deak@intel.com>
parent 21a727b3
...@@ -4030,37 +4030,35 @@ static void lpt_disable_iclkip(struct drm_i915_private *dev_priv) ...@@ -4030,37 +4030,35 @@ static void lpt_disable_iclkip(struct drm_i915_private *dev_priv)
/* Program iCLKIP clock to the desired frequency */ /* Program iCLKIP clock to the desired frequency */
static void lpt_program_iclkip(struct drm_crtc *crtc) static void lpt_program_iclkip(struct drm_crtc *crtc)
{ {
struct drm_device *dev = crtc->dev; struct drm_i915_private *dev_priv = to_i915(crtc->dev);
struct drm_i915_private *dev_priv = dev->dev_private;
int clock = to_intel_crtc(crtc)->config->base.adjusted_mode.crtc_clock; int clock = to_intel_crtc(crtc)->config->base.adjusted_mode.crtc_clock;
u32 divsel, phaseinc, auxdiv, phasedir = 0; u32 divsel, phaseinc, auxdiv, phasedir = 0;
u32 temp; u32 temp;
lpt_disable_iclkip(dev_priv); lpt_disable_iclkip(dev_priv);
/* 20MHz is a corner case which is out of range for the 7-bit divisor */ /* The iCLK virtual clock root frequency is in MHz,
if (clock == 20000) { * but the adjusted_mode->crtc_clock in in KHz. To get the
auxdiv = 1; * divisors, it is necessary to divide one by another, so we
divsel = 0x41; * convert the virtual clock precision to KHz here for higher
phaseinc = 0x20; * precision.
} else { */
/* The iCLK virtual clock root frequency is in MHz, for (auxdiv = 0; auxdiv < 2; auxdiv++) {
* but the adjusted_mode->crtc_clock in in KHz. To get the
* divisors, it is necessary to divide one by another, so we
* convert the virtual clock precision to KHz here for higher
* precision.
*/
u32 iclk_virtual_root_freq = 172800 * 1000; u32 iclk_virtual_root_freq = 172800 * 1000;
u32 iclk_pi_range = 64; u32 iclk_pi_range = 64;
u32 desired_divisor, msb_divisor_value, pi_value; u32 desired_divisor;
desired_divisor = DIV_ROUND_CLOSEST(iclk_virtual_root_freq, clock); desired_divisor = DIV_ROUND_CLOSEST(iclk_virtual_root_freq,
msb_divisor_value = desired_divisor / iclk_pi_range; clock << auxdiv);
pi_value = desired_divisor % iclk_pi_range; divsel = (desired_divisor / iclk_pi_range) - 2;
phaseinc = desired_divisor % iclk_pi_range;
auxdiv = 0; /*
divsel = msb_divisor_value - 2; * Near 20MHz is a corner case which is
phaseinc = pi_value; * out of range for the 7-bit divisor
*/
if (divsel <= 0x7f)
break;
} }
/* This should not happen with any sane values */ /* This should not happen with any sane values */
......
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