All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Vivekanandan, Balasubramani" <balasubramani.vivekanandan@intel.com>
To: Matt Roper <matthew.d.roper@intel.com>, <intel-xe@lists.freedesktop.org>
Subject: Re: [PATCH v2 01/23] drm/xe/mtl: Drop pre-prod workarounds Wa_14015795083 & Wa_14014475959
Date: Mon, 23 Feb 2026 13:27:40 +0530	[thread overview]
Message-ID: <aZwIdBHR-ARudmBy@bvivekan-mob1> (raw)
In-Reply-To: <20260220-forupstream-wa_cleanup-v2-1-b12005a05af6@intel.com>

On 20.02.2026 09:27, Matt Roper wrote:
> Wa_14015795083 and Wa_14014475959 only apply to early steppings of
> Xe_LPG that appeared only in pre-production hardware (in fact
> Wa_14014475959 wasn't supposed to apply to _any_ steppings of version
> 12.71).  Xe1 platforms already aren't officially supported by the Xe
> driver, but pre-production steppings are especially out of scope (and
> 'has_pre_prod_wa' is not set in the device descriptor).  Drop both
> workarounds.
> 
> Bspec: 55420
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> ---
>  drivers/gpu/drm/xe/xe_wa.c         | 4 ----
>  drivers/gpu/drm/xe/xe_wa_oob.rules | 3 +--
>  2 files changed, 1 insertion(+), 6 deletions(-)

LGTM,

Reviewed-by: Balasubramani Vivekanandan <balasubramani.vivekanandan@intel.com>

Regards,
Bala

> 
> diff --git a/drivers/gpu/drm/xe/xe_wa.c b/drivers/gpu/drm/xe/xe_wa.c
> index 76fa6d510aadb91169170ba35f65390c477a569e..aa5755bcdc04a3ec2569ab9676a9dd95516f1711 100644
> --- a/drivers/gpu/drm/xe/xe_wa.c
> +++ b/drivers/gpu/drm/xe/xe_wa.c
> @@ -191,10 +191,6 @@ static const struct xe_rtp_entry_sr gt_was[] = {
>  
>  	/* Xe_LPG */
>  
> -	{ XE_RTP_NAME("14015795083"),
> -	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(1270, 1271), GRAPHICS_STEP(A0, B0)),
> -	  XE_RTP_ACTIONS(CLR(MISCCPCTL, DOP_CLOCK_GATE_RENDER_ENABLE))
> -	},
>  	{ XE_RTP_NAME("14018575942"),
>  	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(1270, 1274)),
>  	  XE_RTP_ACTIONS(SET(COMP_MOD_CTRL, FORCE_MISS_FTLB))
> diff --git a/drivers/gpu/drm/xe/xe_wa_oob.rules b/drivers/gpu/drm/xe/xe_wa_oob.rules
> index ac08f94f90a142629545c4964ca0fd3f9cad6aa9..cc988f2a18d33365dc09d32f4e85ced01b823877 100644
> --- a/drivers/gpu/drm/xe/xe_wa_oob.rules
> +++ b/drivers/gpu/drm/xe/xe_wa_oob.rules
> @@ -2,8 +2,7 @@
>  16010904313	GRAPHICS_VERSION_RANGE(1200, 1210)
>  18022495364	GRAPHICS_VERSION_RANGE(1200, 1210)
>  22012773006	GRAPHICS_VERSION_RANGE(1200, 1250)
> -14014475959	GRAPHICS_VERSION_RANGE(1270, 1271), GRAPHICS_STEP(A0, B0)
> -		PLATFORM(DG2)
> +14014475959	PLATFORM(DG2)
>  22011391025	PLATFORM(DG2)
>  22012727170	SUBPLATFORM(DG2, G11)
>  22012727685	SUBPLATFORM(DG2, G11)
> 
> -- 
> 2.53.0
> 

  reply	other threads:[~2026-02-23  7:58 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-20 17:27 [PATCH v2 00/23] Workaround cleanup & simplification Matt Roper
2026-02-20 17:27 ` [PATCH v2 01/23] drm/xe/mtl: Drop pre-prod workarounds Wa_14015795083 & Wa_14014475959 Matt Roper
2026-02-23  7:57   ` Vivekanandan, Balasubramani [this message]
2026-02-20 17:27 ` [PATCH v2 02/23] drm/xe/pvc: Drop pre-prod workarounds Matt Roper
2026-02-23  8:27   ` Vivekanandan, Balasubramani
2026-02-20 17:27 ` [PATCH v2 03/23] drm/xe/wa: Document new policy regarding workaround IP ranges Matt Roper
2026-02-20 17:27 ` [PATCH v2 04/23] drm/xe: Consolidate workaround entries for Wa_16021867713 Matt Roper
2026-02-23  9:57   ` Vivekanandan, Balasubramani
2026-02-20 17:27 ` [PATCH v2 05/23] drm/xe: Consolidate workaround entries for Wa_14019449301 Matt Roper
2026-02-23 10:00   ` Vivekanandan, Balasubramani
2026-02-20 17:27 ` [PATCH v2 06/23] drm/xe: Consolidate workaround entries for Wa_16028005424 Matt Roper
2026-02-23 14:03   ` Vivekanandan, Balasubramani
2026-02-20 17:27 ` [PATCH v2 07/23] drm/xe: Consolidate workaround entries for Wa_16021865536 Matt Roper
2026-02-23 14:08   ` Vivekanandan, Balasubramani
2026-02-20 17:27 ` [PATCH v2 08/23] drm/xe: Consolidate workaround entries for Wa_18032247524 Matt Roper
2026-02-23 14:12   ` Vivekanandan, Balasubramani
2026-02-20 17:27 ` [PATCH v2 09/23] drm/xe: Consolidate workaround entries for Wa_16018712365 Matt Roper
2026-02-23 14:16   ` Vivekanandan, Balasubramani
2026-02-20 17:27 ` [PATCH v2 10/23] drm/xe: Consolidate workaround entries for Wa_14020338487 Matt Roper
2026-02-23 14:19   ` Vivekanandan, Balasubramani
2026-02-20 17:27 ` [PATCH v2 11/23] drm/xe: Consolidate workaround entries for Wa_14018471104 Matt Roper
2026-02-23 14:21   ` Vivekanandan, Balasubramani
2026-02-20 17:27 ` [PATCH v2 12/23] drm/xe: Consolidate workaround entries for Wa_16021639441 Matt Roper
2026-02-23 15:04   ` Vivekanandan, Balasubramani
2026-02-20 17:27 ` [PATCH v2 13/23] drm/xe: Consolidate workaround entries for Wa_14021402888 Matt Roper
2026-02-23 22:25   ` Lin, Shuicheng
2026-02-20 17:27 ` [PATCH v2 14/23] drm/xe: Consolidate workaround entries for Wa_13012615864 Matt Roper
2026-02-23 21:53   ` Lin, Shuicheng
2026-02-20 17:27 ` [PATCH v2 15/23] drm/xe: Consolidate workaround entries for Wa_18041344222 Matt Roper
2026-02-23 21:47   ` Lin, Shuicheng
2026-02-23 21:51     ` Matt Roper
2026-02-20 17:27 ` [PATCH v2 16/23] drm/xe: Consolidate workaround entries for Wa_14019877138 Matt Roper
2026-02-23 21:33   ` Lin, Shuicheng
2026-02-23 21:41     ` Matt Roper
2026-02-23 22:35       ` Lin, Shuicheng
2026-02-20 17:27 ` [PATCH v2 17/23] drm/xe: Consolidate workaround entries for Wa_14019386621 Matt Roper
2026-02-23 19:11   ` Lin, Shuicheng
2026-02-20 17:27 ` [PATCH v2 18/23] drm/xe: Consolidate workaround entries for Wa_14019988906 Matt Roper
2026-02-23 21:56   ` Lin, Shuicheng
2026-02-20 17:27 ` [PATCH v2 19/23] drm/xe: Consolidate workaround entries for Wa_18033852989 Matt Roper
2026-02-23 19:14   ` Lin, Shuicheng
2026-02-20 17:27 ` [PATCH v2 20/23] drm/xe: Consolidate workaround entries for Wa_15016589081 Matt Roper
2026-02-23 19:06   ` Lin, Shuicheng
2026-02-20 17:27 ` [PATCH v2 21/23] drm/xe/xe2_hpg: Drop invalid workaround Wa_15010599737 Matt Roper
2026-02-24  9:40   ` Vivekanandan, Balasubramani
2026-02-20 17:27 ` [PATCH v2 22/23] drm/xe/wa_oob: Consolidate some OOB rules Matt Roper
2026-02-23 22:44   ` Lin, Shuicheng
2026-02-20 17:27 ` [PATCH v2 23/23] drm/xe: Consolidate workaround entries for Wa_14023061436 Matt Roper
2026-02-23 22:47   ` Lin, Shuicheng
2026-02-20 19:05 ` ✓ CI.KUnit: success for Workaround cleanup & simplification (rev2) Patchwork
2026-02-20 19:46 ` ✓ Xe.CI.BAT: " Patchwork
2026-02-23  7:23 ` ✗ Xe.CI.FULL: failure " Patchwork
2026-02-24  0:07   ` Matt Roper

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aZwIdBHR-ARudmBy@bvivekan-mob1 \
    --to=balasubramani.vivekanandan@intel.com \
    --cc=intel-xe@lists.freedesktop.org \
    --cc=matthew.d.roper@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.