On Fri, 2019-03-22 at 22:49 +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrj...@linux.intel.com>
> 
> If I'm reading the spec right AML 0x87CA is a Y SKU, so it
> should be marked as ULX in our old style terminology.

You are right, only the CPU is based on CFL.

Reviewed-by: José Roberto de Souza <jose.so...@intel.com>

> 
> Cc: sta...@vger.kernel.org
> Cc: José Roberto de Souza <jose.so...@intel.com>
> Cc: Rodrigo Vivi <rodrigo.v...@intel.com>
> Cc: Tvrtko Ursulin <tvrtko.ursu...@intel.com>
> Fixes: c0c46ca461f1 ("drm/i915/aml: Add new Amber Lake PCI ID")
> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
> ---
>  drivers/gpu/drm/i915/i915_drv.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_drv.h
> b/drivers/gpu/drm/i915/i915_drv.h
> index f723b15527f8..428279fd88a0 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -2361,7 +2361,8 @@ static inline unsigned int
> i915_sg_segment_size(void)
>                                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)
> +                              INTEL_DEVID(dev_priv) == 0x87C0 || \
> +                              INTEL_DEVID(dev_priv) == 0x87CA)
>  #define IS_SKL_GT2(dev_priv) (IS_SKYLAKE(dev_priv) && \
>                                INTEL_INFO(dev_priv)->gt == 2)
>  #define IS_SKL_GT3(dev_priv) (IS_SKYLAKE(dev_priv) && \

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to