According to patch "drm/i915/aml: Introducing Amber Lake platform" (e364672477a1). Amber Lake uses the same gen graphics as Kaby Lake. And it is the member of KBL ULX series. So far, IS_KBL_ULX macro did not include AML platform. It may caused driver load DDI translation table for KBL H/S series for AML system. The change will lead driver to apply translation table for KBL Y for AML device to avoid unexpected eDP/DP signal quality issue. 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 | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index 808204a7ca7c..0388de4d64a2 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++ b/drivers/gpu/drm/i915/i915_drv.h @@ -2460,7 +2460,9 @@ intel_info(const struct drm_i915_private *dev_priv) INTEL_DEVID(dev_priv) == 0x5926) #define IS_KBL_ULX(dev_priv) (INTEL_DEVID(dev_priv) == 0x590E || \ INTEL_DEVID(dev_priv) == 0x5915 || \ - INTEL_DEVID(dev_priv) == 0x591E) + INTEL_DEVID(dev_priv) == 0x591E || \ + 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) && \ -- 2.7.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx