On Thu, Nov 16, 2017 at 09:22:23AM +0000, Jani Nikula wrote:
> On Wed, 15 Nov 2017, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote:
> > Quoting David Weinehall (2017-11-15 18:01:41)
> >> On Tue, Nov 14, 2017 at 01:51:16PM +0000, Chris Wilson wrote:
> >> > WaEnablePooledEuFor2x6 only applies to preproduction models, unsupported
> >> > since commit 0102ba1fd8af ("drm/i915: Add early BXT sdv to the list of
> >> > preproduction machines").
> >> >
> >> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> >> > Cc: Jani Nikula <jani.nikula@xxxxxxxxx>
> >>
> >> Reviewed-by: David Weinehall <david.weinehall@xxxxxxxxxxxxxxx>
> >
> > Ta, hope your sdp is ok in the bin ;)
> >
> > Any volunteers for IS_GLK_REVID, IS_KBL_REVID ?
> >
> > I see one IS_SKL_REVID that can easily be removed.
>
> Need to know what the first shipped rev was. For example with bxt we
> apparently can't remove some of the rev checks.
>
> Seems like we should add more checks to intel_detect_preproduction_hw()
> too.
With the IS_*_REVID in place I'd advocate to leave the workarounds around.
I understand that codes get cleaner if we remove them from here,
but usually previous pre-prod workarounds are useful reference for
new platforms power-on...
Thanks,
Rodrigo.
>
> BR,
> Jani.
>
> --
> Jani Nikula, Intel Open Source Technology Center
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
