On Tue, 2018-09-25 at 21:18 -0700, Souza, Jose wrote: >> According to patch "drm/i915/aml: Introducing Amber Lake platform" >> (e364672477a1). Add a new marco for AML ULX GT2 devices. > >Just to confirm, there is a newly added Amber lake id( >https://patchwork.freedesktop.org/series/50037/) 0x87CA that is based in WHL/CFL, I guess this one will don't need to be added to >IS_AML_ULX() too. >If you agree with that: > >Reviewed-by: Jose Roberto de Souza <jose.souza@xxxxxxxxx> > Yes, I think so too. We can add more which is based in WHL/CFL in the future. >> >> Cc: Jani Nikula <jani.nikula@xxxxxxxxx> >> Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> >> Cc: Jose Roberto de Souza <jose.souza@xxxxxxxxx> >> Signed-off-by: Lee, Shawn C <shawn.c.lee@xxxxxxxxx> >> --- >> drivers/gpu/drm/i915/i915_drv.h | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/i915_drv.h >> b/drivers/gpu/drm/i915/i915_drv.h index 808204a7ca7c..0c3de2d14db0 >> 100644 >> --- a/drivers/gpu/drm/i915/i915_drv.h >> +++ b/drivers/gpu/drm/i915/i915_drv.h >> @@ -2461,6 +2461,8 @@ intel_info(const struct drm_i915_private >> *dev_priv) >> #define IS_KBL_ULX(dev_priv) (INTEL_DEVID(dev_priv) == 0x590E || \ >> INTEL_DEVID(dev_priv) == 0x5915 || \ >> INTEL_DEVID(dev_priv) == 0x591E) >> +#define IS_AML_ULX(dev_priv) (INTEL_DEVID(dev_priv) == 0x591C || \ >> + INTEL_DEVID(dev_priv) == 0x87C0) >> #define IS_SKL_GT2(dev_priv) (IS_SKYLAKE(dev_priv) && \ >> (dev_priv)->info.gt == 2) >> #define IS_SKL_GT3(dev_priv) (IS_SKYLAKE(dev_priv) && \ _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
