On Tue, Sep 25, 2018 at 08:08:59PM +0000, Souza, Jose wrote:
> On Mon, 2018-09-24 at 16:43 -0700, Rodrigo Vivi wrote:
> > commit 'b9be78531d27 ("drm/i915/whl: Introducing
> > Whiskey Lake platform")' introduced WHL by moving some
> > of CFL IDs here and using the Spec information of "U43" for
> > most of IDs what appeared to be GT3.
> >
> > However when propagating the change to Mesa, Lionel noticed
> > that based on number of execution unities the classification
> > here seems at least strange.
> >
> > So, let's move for now with the information we trust more:
> > the number of EUs. So we are able to propagate this change
> > across the stack without getting stuck forever.
> >
> > Reference: https://patchwork.freedesktop.org/patch/246695/
> > Fixes: b9be78531d27 ("drm/i915/whl: Introducing Whiskey Lake
> > platform")
> > Cc: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>
>
> By the EU number this looks right.
> Reviewed-by: José Roberto de Souza <jose.souza@xxxxxxxxx>
pushed to dinq, thanks
>
> > Cc: José Roberto de Souza <jose.souza@xxxxxxxxx>
> > Cc: David Airlie <airlied@xxxxxxxx>
> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> > ---
> > include/drm/i915_pciids.h | 10 +++++-----
> > 1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/include/drm/i915_pciids.h b/include/drm/i915_pciids.h
> > index fd965ffbb92e..c8d3d541ad01 100644
> > --- a/include/drm/i915_pciids.h
> > +++ b/include/drm/i915_pciids.h
> > @@ -407,17 +407,17 @@
> >
> > /* WHL/CFL U GT1 */
> > #define INTEL_WHL_U_GT1_IDS(info) \
> > - INTEL_VGA_DEVICE(0x3EA1, info)
> > + INTEL_VGA_DEVICE(0x3EA1, info), \
> > + INTEL_VGA_DEVICE(0x3EA4, info)
> >
> > /* WHL/CFL U GT2 */
> > #define INTEL_WHL_U_GT2_IDS(info) \
> > - INTEL_VGA_DEVICE(0x3EA0, info)
> > + INTEL_VGA_DEVICE(0x3EA0, info), \
> > + INTEL_VGA_DEVICE(0x3EA3, info)
> >
> > /* WHL/CFL U GT3 */
> > #define INTEL_WHL_U_GT3_IDS(info) \
> > - INTEL_VGA_DEVICE(0x3EA2, info), \
> > - INTEL_VGA_DEVICE(0x3EA3, info), \
> > - INTEL_VGA_DEVICE(0x3EA4, info)
> > + INTEL_VGA_DEVICE(0x3EA2, info)
> >
> > #define INTEL_CFL_IDS(info) \
> > INTEL_CFL_S_GT1_IDS(info), \
> _______________________________________________
> 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
