diff mbox

[6/7] drm/i915: Drop WaDoubleCursorLP3Latency:ivb

Message ID 20180130203807.13721-6-ville.syrjala@linux.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Ville Syrjälä Jan. 30, 2018, 8:38 p.m. UTC 
From: Ville Syrjälä <ville.syrjala@linux.intel.com>

WaDoubleCursorLP3Latency was meant for pre-production hardware.
Drop it.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 drivers/gpu/drm/i915/intel_pm.c | 4 ----
 1 file changed, 4 deletions(-)

Comments

Mika Kahola Jan. 31, 2018, 2:10 p.m. UTC | #1 
On Tue, 2018-01-30 at 22:38 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> 
> WaDoubleCursorLP3Latency was meant for pre-production hardware.
> Drop it.
> 
Reviewed-by: Mika Kahola <mika.kahola@intel.com>

> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
>  drivers/gpu/drm/i915/intel_pm.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_pm.c
> b/drivers/gpu/drm/i915/intel_pm.c
> index 0b92ea1dbd40..8eac3895ea9a 100644
> --- a/drivers/gpu/drm/i915/intel_pm.c
> +++ b/drivers/gpu/drm/i915/intel_pm.c
> @@ -2916,10 +2916,6 @@ static void intel_fixup_cur_wm_latency(struct
> drm_i915_private *dev_priv,
>  	/* ILK cursor LP0 latency is 1300 ns */
>  	if (IS_GEN5(dev_priv))
>  		wm[0] = 13;
> -
> -	/* WaDoubleCursorLP3Latency:ivb */
> -	if (IS_IVYBRIDGE(dev_priv))
> -		wm[3] *= 2;
>  }
>  
>  int ilk_wm_max_level(const struct drm_i915_private *dev_priv)
Ville Syrjälä Feb. 20, 2018, 7:01 p.m. UTC | #2 
On Wed, Jan 31, 2018 at 04:10:10PM +0200, Mika Kahola wrote:
> On Tue, 2018-01-30 at 22:38 +0200, Ville Syrjala wrote:
> > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > 
> > WaDoubleCursorLP3Latency was meant for pre-production hardware.
> > Drop it.
> > 
> Reviewed-by: Mika Kahola <mika.kahola@intel.com>

Patches 1,2,6 pushed to dinq. Thanks for the review.

> 
> > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > ---
> >  drivers/gpu/drm/i915/intel_pm.c | 4 ----
> >  1 file changed, 4 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/intel_pm.c
> > b/drivers/gpu/drm/i915/intel_pm.c
> > index 0b92ea1dbd40..8eac3895ea9a 100644
> > --- a/drivers/gpu/drm/i915/intel_pm.c
> > +++ b/drivers/gpu/drm/i915/intel_pm.c
> > @@ -2916,10 +2916,6 @@ static void intel_fixup_cur_wm_latency(struct
> > drm_i915_private *dev_priv,
> >  	/* ILK cursor LP0 latency is 1300 ns */
> >  	if (IS_GEN5(dev_priv))
> >  		wm[0] = 13;
> > -
> > -	/* WaDoubleCursorLP3Latency:ivb */
> > -	if (IS_IVYBRIDGE(dev_priv))
> > -		wm[3] *= 2;
> >  }
> >  
> >  int ilk_wm_max_level(const struct drm_i915_private *dev_priv)
> -- 
> Mika Kahola - Intel OTC
diff mbox

Patch

diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 0b92ea1dbd40..8eac3895ea9a 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -2916,10 +2916,6 @@  static void intel_fixup_cur_wm_latency(struct drm_i915_private *dev_priv,
 	/* ILK cursor LP0 latency is 1300 ns */
 	if (IS_GEN5(dev_priv))
 		wm[0] = 13;
-
-	/* WaDoubleCursorLP3Latency:ivb */
-	if (IS_IVYBRIDGE(dev_priv))
-		wm[3] *= 2;
 }
 
 int ilk_wm_max_level(const struct drm_i915_private *dev_priv)