* [PATCH 1/2] drm/i915/vlv: split CCK and DDR freq usage
@ 2013-11-05 0:06 Jesse Barnes
2013-11-05 0:07 ` [PATCH 2/2] drm/i915/vlv: fixup DDR freq detection per Punit spec Jesse Barnes
2013-11-05 11:24 ` [PATCH 1/2] drm/i915/vlv: split CCK and DDR freq usage Ville Syrjälä
0 siblings, 2 replies; 4+ messages in thread
From: Jesse Barnes @ 2013-11-05 0:06 UTC (permalink / raw)
To: intel-gfx
It's possible that the CCK clock could run at a different rate than the
DDR clock, so use the same method to get CCK as the GMBUS code does when
calculating the new CDclk divider in the VLV display code.
Reported-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
---
drivers/gpu/drm/i915/intel_display.c | 23 ++++++++---------------
drivers/gpu/drm/i915/intel_drv.h | 2 +-
drivers/gpu/drm/i915/intel_i2c.c | 11 +++--------
3 files changed, 12 insertions(+), 24 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index 7e0af61..bd0804a 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -3894,24 +3894,17 @@ static void i9xx_pfit_enable(struct intel_crtc *crtc)
I915_WRITE(BCLRPAT(crtc->pipe), 0);
}
-static int valleyview_get_vco(struct drm_i915_private *dev_priv)
+int valleyview_get_vco(struct drm_i915_private *dev_priv)
{
- int vco;
+ int hpll_freq, vco_freq[] = { 800, 1600, 2000, 2400 };
- switch (dev_priv->mem_freq) {
- default:
- case 800:
- vco = 800;
- break;
- case 1066:
- vco = 1600;
- break;
- case 1333:
- vco = 2000;
- break;
- }
+ /* Obtain SKU information */
+ mutex_lock(&dev_priv->dpio_lock);
+ hpll_freq = vlv_cck_read(dev_priv, CCK_FUSE_REG) &
+ CCK_FUSE_HPLL_FREQ_MASK;
+ mutex_unlock(&dev_priv->dpio_lock);
- return vco;
+ return vco_freq[hpll_freq];
}
/* Adjust CDclk dividers to allow high res or save power if possible */
diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
index 1e49aa8..1876ea1 100644
--- a/drivers/gpu/drm/i915/intel_drv.h
+++ b/drivers/gpu/drm/i915/intel_drv.h
@@ -694,7 +694,7 @@ void i915_disable_vga_mem(struct drm_device *dev);
void hsw_enable_ips(struct intel_crtc *crtc);
void hsw_disable_ips(struct intel_crtc *crtc);
void intel_display_set_init_power(struct drm_device *dev, bool enable);
-
+int valleyview_get_vco(struct drm_i915_private *dev_priv);
/* intel_dp.c */
void intel_dp_init(struct drm_device *dev, int output_reg, enum port port);
diff --git a/drivers/gpu/drm/i915/intel_i2c.c b/drivers/gpu/drm/i915/intel_i2c.c
index 1263409..b1dc33f 100644
--- a/drivers/gpu/drm/i915/intel_i2c.c
+++ b/drivers/gpu/drm/i915/intel_i2c.c
@@ -82,16 +82,11 @@ static int get_disp_clk_div(struct drm_i915_private *dev_priv,
static void gmbus_set_freq(struct drm_i915_private *dev_priv)
{
- int vco_freq[] = { 800, 1600, 2000, 2400 };
- int gmbus_freq = 0, cdclk_div, hpll_freq;
+ int vco, gmbus_freq = 0, cdclk_div;
BUG_ON(!IS_VALLEYVIEW(dev_priv->dev));
- /* Obtain SKU information */
- mutex_lock(&dev_priv->dpio_lock);
- hpll_freq =
- vlv_cck_read(dev_priv, CCK_FUSE_REG) & CCK_FUSE_HPLL_FREQ_MASK;
- mutex_unlock(&dev_priv->dpio_lock);
+ vco = valleyview_get_vco(dev_priv);
/* Get the CDCLK divide ratio */
cdclk_div = get_disp_clk_div(dev_priv, CDCLK);
@@ -102,7 +97,7 @@ static void gmbus_set_freq(struct drm_i915_private *dev_priv)
* in fact 1MHz is the correct frequency.
*/
if (cdclk_div)
- gmbus_freq = (vco_freq[hpll_freq] << 1) / cdclk_div;
+ gmbus_freq = (vco << 1) / cdclk_div;
if (WARN_ON(gmbus_freq == 0))
return;
--
1.8.3.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH 2/2] drm/i915/vlv: fixup DDR freq detection per Punit spec
2013-11-05 0:06 [PATCH 1/2] drm/i915/vlv: split CCK and DDR freq usage Jesse Barnes
@ 2013-11-05 0:07 ` Jesse Barnes
2013-11-05 11:24 ` [PATCH 1/2] drm/i915/vlv: split CCK and DDR freq usage Ville Syrjälä
1 sibling, 0 replies; 4+ messages in thread
From: Jesse Barnes @ 2013-11-05 0:07 UTC (permalink / raw)
To: intel-gfx
Either the docs were wrong or the values have changed since the old days
before we had wheels.
Reported-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
---
drivers/gpu/drm/i915/intel_pm.c | 13 ++++++++++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 2e7072e..71da695 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -5319,15 +5319,22 @@ static void valleyview_init_clock_gating(struct drm_device *dev)
mutex_unlock(&dev_priv->rps.hw_lock);
switch ((val >> 6) & 3) {
case 0:
- case 1:
dev_priv->mem_freq = 800;
break;
- case 2:
+ case 1:
dev_priv->mem_freq = 1066;
break;
- case 3:
+ case 2:
dev_priv->mem_freq = 1333;
break;
+ case 3:
+ /*
+ * Probably a BIOS/Punit bug, or a new platform we don't
+ * support yet.
+ */
+ WARN(1, "invalid DDR freq detected, assuming 800MHz\n");
+ dev_priv->mem_freq = 800;
+ break;
}
DRM_DEBUG_DRIVER("DDR speed: %d MHz", dev_priv->mem_freq);
--
1.8.3.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH 1/2] drm/i915/vlv: split CCK and DDR freq usage
2013-11-05 0:06 [PATCH 1/2] drm/i915/vlv: split CCK and DDR freq usage Jesse Barnes
2013-11-05 0:07 ` [PATCH 2/2] drm/i915/vlv: fixup DDR freq detection per Punit spec Jesse Barnes
@ 2013-11-05 11:24 ` Ville Syrjälä
2013-11-05 18:31 ` Daniel Vetter
1 sibling, 1 reply; 4+ messages in thread
From: Ville Syrjälä @ 2013-11-05 11:24 UTC (permalink / raw)
To: Jesse Barnes; +Cc: intel-gfx
On Mon, Nov 04, 2013 at 04:06:59PM -0800, Jesse Barnes wrote:
> It's possible that the CCK clock could run at a different rate than the
> DDR clock, so use the same method to get CCK as the GMBUS code does when
> calculating the new CDclk divider in the VLV display code.
>
> Reported-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
For the series:
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> drivers/gpu/drm/i915/intel_display.c | 23 ++++++++---------------
> drivers/gpu/drm/i915/intel_drv.h | 2 +-
> drivers/gpu/drm/i915/intel_i2c.c | 11 +++--------
> 3 files changed, 12 insertions(+), 24 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index 7e0af61..bd0804a 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -3894,24 +3894,17 @@ static void i9xx_pfit_enable(struct intel_crtc *crtc)
> I915_WRITE(BCLRPAT(crtc->pipe), 0);
> }
>
> -static int valleyview_get_vco(struct drm_i915_private *dev_priv)
> +int valleyview_get_vco(struct drm_i915_private *dev_priv)
> {
> - int vco;
> + int hpll_freq, vco_freq[] = { 800, 1600, 2000, 2400 };
>
> - switch (dev_priv->mem_freq) {
> - default:
> - case 800:
> - vco = 800;
> - break;
> - case 1066:
> - vco = 1600;
> - break;
> - case 1333:
> - vco = 2000;
> - break;
> - }
> + /* Obtain SKU information */
> + mutex_lock(&dev_priv->dpio_lock);
> + hpll_freq = vlv_cck_read(dev_priv, CCK_FUSE_REG) &
> + CCK_FUSE_HPLL_FREQ_MASK;
> + mutex_unlock(&dev_priv->dpio_lock);
>
> - return vco;
> + return vco_freq[hpll_freq];
> }
>
> /* Adjust CDclk dividers to allow high res or save power if possible */
> diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
> index 1e49aa8..1876ea1 100644
> --- a/drivers/gpu/drm/i915/intel_drv.h
> +++ b/drivers/gpu/drm/i915/intel_drv.h
> @@ -694,7 +694,7 @@ void i915_disable_vga_mem(struct drm_device *dev);
> void hsw_enable_ips(struct intel_crtc *crtc);
> void hsw_disable_ips(struct intel_crtc *crtc);
> void intel_display_set_init_power(struct drm_device *dev, bool enable);
> -
> +int valleyview_get_vco(struct drm_i915_private *dev_priv);
>
> /* intel_dp.c */
> void intel_dp_init(struct drm_device *dev, int output_reg, enum port port);
> diff --git a/drivers/gpu/drm/i915/intel_i2c.c b/drivers/gpu/drm/i915/intel_i2c.c
> index 1263409..b1dc33f 100644
> --- a/drivers/gpu/drm/i915/intel_i2c.c
> +++ b/drivers/gpu/drm/i915/intel_i2c.c
> @@ -82,16 +82,11 @@ static int get_disp_clk_div(struct drm_i915_private *dev_priv,
>
> static void gmbus_set_freq(struct drm_i915_private *dev_priv)
> {
> - int vco_freq[] = { 800, 1600, 2000, 2400 };
> - int gmbus_freq = 0, cdclk_div, hpll_freq;
> + int vco, gmbus_freq = 0, cdclk_div;
>
> BUG_ON(!IS_VALLEYVIEW(dev_priv->dev));
>
> - /* Obtain SKU information */
> - mutex_lock(&dev_priv->dpio_lock);
> - hpll_freq =
> - vlv_cck_read(dev_priv, CCK_FUSE_REG) & CCK_FUSE_HPLL_FREQ_MASK;
> - mutex_unlock(&dev_priv->dpio_lock);
> + vco = valleyview_get_vco(dev_priv);
>
> /* Get the CDCLK divide ratio */
> cdclk_div = get_disp_clk_div(dev_priv, CDCLK);
> @@ -102,7 +97,7 @@ static void gmbus_set_freq(struct drm_i915_private *dev_priv)
> * in fact 1MHz is the correct frequency.
> */
> if (cdclk_div)
> - gmbus_freq = (vco_freq[hpll_freq] << 1) / cdclk_div;
> + gmbus_freq = (vco << 1) / cdclk_div;
>
> if (WARN_ON(gmbus_freq == 0))
> return;
> --
> 1.8.3.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Ville Syrjälä
Intel OTC
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH 1/2] drm/i915/vlv: split CCK and DDR freq usage
2013-11-05 11:24 ` [PATCH 1/2] drm/i915/vlv: split CCK and DDR freq usage Ville Syrjälä
@ 2013-11-05 18:31 ` Daniel Vetter
0 siblings, 0 replies; 4+ messages in thread
From: Daniel Vetter @ 2013-11-05 18:31 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
On Tue, Nov 05, 2013 at 01:24:49PM +0200, Ville Syrjälä wrote:
> On Mon, Nov 04, 2013 at 04:06:59PM -0800, Jesse Barnes wrote:
> > It's possible that the CCK clock could run at a different rate than the
> > DDR clock, so use the same method to get CCK as the GMBUS code does when
> > calculating the new CDclk divider in the VLV display code.
> >
> > Reported-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
>
> For the series:
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Both series merged, thanks for patches and review.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-11-05 18:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-05 0:06 [PATCH 1/2] drm/i915/vlv: split CCK and DDR freq usage Jesse Barnes
2013-11-05 0:07 ` [PATCH 2/2] drm/i915/vlv: fixup DDR freq detection per Punit spec Jesse Barnes
2013-11-05 11:24 ` [PATCH 1/2] drm/i915/vlv: split CCK and DDR freq usage Ville Syrjälä
2013-11-05 18:31 ` Daniel Vetter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox