On Thu, Mar 23, 2017 at 09:27:08PM +0200, [email protected] wrote: > From: Ville Syrjälä <[email protected]> > > Share the code to compute the primary plane control register value > between the i9xx and ilk codepaths as the differences are minimal. > Actually there are no differences between g4x and ilk, so the > current split doesn't really make any sense. > > Cc: Chris Wilson <[email protected]> > Signed-off-by: Ville Syrjälä <[email protected]> Reviewed-by: Chris Wilson <[email protected]> > --- > drivers/gpu/drm/i915/intel_display.c | 58 > ++++-------------------------------- > 1 file changed, 6 insertions(+), 52 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c > b/drivers/gpu/drm/i915/intel_display.c > index aa2c85b2bf78..4f57ce982a72 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -2974,9 +2974,13 @@ static u32 i9xx_plane_ctl(const struct > intel_crtc_state *crtc_state, > > dspcntr = DISPLAY_PLANE_ENABLE | DISPPLANE_GAMMA_ENABLE; > > - if (IS_G4X(dev_priv)) > + if (IS_G4X(dev_priv) || IS_GEN5(dev_priv) || > + IS_GEN6(dev_priv) || IS_IVYBRIDGE(dev_priv)) > dspcntr |= DISPPLANE_TRICKLE_FEED_DISABLE;
Fortunately I remembered the explanation last time about vlv and chv, and yes this does make that much more obvious. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
