On Thu, Jan 16, 2025 at 01:08:28PM +0530, Ranu Maurya wrote:
> Add Wa_22010465259 which points to an existing WA, but
> was missing from the comment list. While at it, update
> the other WAs and their applicable platforms as well.
> 
> Signed-off-by: Ranu Maurya <ranu.mau...@intel.com>
> ---
>  drivers/gpu/drm/i915/gt/intel_workarounds.c | 13 +++++++------
>  1 file changed, 7 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c 
> b/drivers/gpu/drm/i915/gt/intel_workarounds.c
> index 570c91878189..4cf4fa1f1a92 100644
> --- a/drivers/gpu/drm/i915/gt/intel_workarounds.c
> +++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c
> @@ -691,16 +691,17 @@ static void gen12_ctx_workarounds_init(struct 
> intel_engine_cs *engine,
>       struct drm_i915_private *i915 = engine->i915;
>  
>       /*
> -      * Wa_1409142259:tgl,dg1,adl-p
> +      * Wa_1409142259:tgl,dg1,adl-p,adl-n
>        * Wa_1409347922:tgl,dg1,adl-p
>        * Wa_1409252684:tgl,dg1,adl-p
>        * Wa_1409217633:tgl,dg1,adl-p
>        * Wa_1409207793:tgl,dg1,adl-p
> -      * Wa_1409178076:tgl,dg1,adl-p
> -      * Wa_1408979724:tgl,dg1,adl-p
> -      * Wa_14010443199:tgl,rkl,dg1,adl-p
> -      * Wa_14010698770:tgl,rkl,dg1,adl-s,adl-p
> -      * Wa_1409342910:tgl,rkl,dg1,adl-s,adl-p
> +      * Wa_1409178076:tgl,dg1,adl-p,adl-n
> +      * Wa_1408979724:tgl,dg1,adl-p,adl-n
> +      * Wa_14010443199:tgl,rkl,dg1,dg2,adl-p,adl-n
> +      * Wa_14010698770:tgl,rkl,dg1,dg2,adl-s,adl-p,adl-n
> +      * Wa_1409342910:tgl,rkl,dg1,adl-s,adl-p,adl-n
> +      * Wa_22010465259:tgl,rkl,dg1,adl-s,adl-p,adl-n

Just to be clear on the conventions, are we using platform_names[] from
intel_device_info.c? Because if we are, I don't see adl-n there.

Raag

Reply via email to