On Thu, Oct 14, 2021 at 12:01:41AM +0300, Imre Deak wrote: > On Wed, Oct 13, 2021 at 11:14:42PM +0300, Ville Syrjälä wrote: > > On Fri, Oct 08, 2021 at 03:19:08AM +0300, Imre Deak wrote: > > > + /* Wa_22011186057 */ > > > + if (IS_ADLP_DISPLAY_STEP(i915, STEP_A0, STEP_B0)) > > > + return false; > > > + > > > + if (DISPLAY_VER(i915) >= 11) > > > + return true; > > > + > > > + if (IS_GEMINILAKE(i915)) > > > + return pipe != PIPE_C; > > > + > > > + return pipe != PIPE_C && > > > + (plane_id == PLANE_PRIMARY || > > > + plane_id == PLANE_SPRITE0); > > > +} > > > > A bit tempted to say we should chop this up into more > > platform specific variants. But that can be left for later I guess. > > You mean clarifying that last check is for SKL/BXT? Would a code comment > be ok?
I don't really enjoy comments when the code can express what we mean more clearly. So I'm thinking just a clean skl/glk/icl split could perhaps be the thing. Pretty sure we have that exact if ladder in the init function already at least once so could shove this stuff in there as well. But it's not really important for the moment. -- Ville Syrjälä Intel
