[PATCH 2/9] drm/xe/xe3: Add support for graphics IP version 30.03

[Bhadane, Dnyaneshwar]

> -----Original Message-----
> From: Vivekanandan, Balasubramani
> <balasubramani.vivekanandan at intel.com>
> Sent: Monday, June 9, 2025 12:30 PM
> To: Bhadane, Dnyaneshwar <dnyaneshwar.bhadane at intel.com>; intel-
> xe at lists.freedesktop.org
> Cc: Roper, Matthew D <matthew.d.roper at intel.com>
> Subject: Re: [PATCH 2/9] drm/xe/xe3: Add support for graphics IP version 30.03
> 
> On 07.06.2025 01:20, Dnyaneshwar Bhadane wrote:
> > From: Matt Roper <matthew.d.roper at intel.com>
> >
> > Graphics version 30.03 should be treated the same as other Xe3 IP, but
> > will have a slightly different set of workarounds.
> >
> > Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
> > Signed-off-by: Dnyaneshwar Bhadane <dnyaneshwar.bhadane at intel.com>
> > ---
> >  drivers/gpu/drm/xe/xe_pci.c        | 1 +
> >  drivers/gpu/drm/xe/xe_wa.c         | 4 ++++
> >  drivers/gpu/drm/xe/xe_wa_oob.rules | 5 ++++-
> >  3 files changed, 9 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> > index b5559800db7a..8824a8016b1c 100644
> > --- a/drivers/gpu/drm/xe/xe_pci.c
> > +++ b/drivers/gpu/drm/xe/xe_pci.c
> > @@ -183,6 +183,7 @@ static const struct xe_ip graphics_ips[] = {
> >  	{ 2004, "Xe2_LPG", &graphics_xe2 },
> >  	{ 3000, "Xe3_LPG", &graphics_xe2 },
> >  	{ 3001, "Xe3_LPG", &graphics_xe2 },
> > +	{ 3003, "Xe3_LPG", &graphics_xe2 },
> >  };
> >
> >  /* Pre-GMDID Media IPs */
> > diff --git a/drivers/gpu/drm/xe/xe_wa.c b/drivers/gpu/drm/xe/xe_wa.c
> > index 67196baa4249..5f750054a9c4 100644
> > --- a/drivers/gpu/drm/xe/xe_wa.c
> > +++ b/drivers/gpu/drm/xe/xe_wa.c
> > @@ -640,6 +640,10 @@ static const struct xe_rtp_entry_sr engine_was[] = {
> >  	  XE_RTP_ACTIONS(SET(RING_PSMI_CTL(0),
> RC_SEMA_IDLE_MSG_DISABLE,
> >  			     XE_RTP_ACTION_FLAG(ENGINE_BASE)))
> >  	},
> > +	{ XE_RTP_NAME("14021402888"),
> > +	  XE_RTP_RULES(GRAPHICS_VERSION(3003),
> FUNC(xe_rtp_match_first_render_or_compute)),
> > +	  XE_RTP_ACTIONS(SET(HALF_SLICE_CHICKEN7,
> > +CLEAR_OPTIMIZATION_DISABLE))
> Instead of adding a new entry, can you extend the range of the already existing
> entry -> GRAPHICS_VERSION_RANGE(3000, 3001) ?
> 
Thanks, I have updated in the new revision.

Dnyaneshwar,

> Regards,
> Bala
> > +	},
> >  };
> >
> >  static const struct xe_rtp_entry_sr lrc_was[] = { diff --git
> > a/drivers/gpu/drm/xe/xe_wa_oob.rules
> > b/drivers/gpu/drm/xe/xe_wa_oob.rules
> > index 9efc5accd43d..557d23c1bb9f 100644
> > --- a/drivers/gpu/drm/xe/xe_wa_oob.rules
> > +++ b/drivers/gpu/drm/xe/xe_wa_oob.rules
> > @@ -33,9 +33,11 @@
> >  14022293748	GRAPHICS_VERSION(2001)
> >  		GRAPHICS_VERSION(2004)
> >  		GRAPHICS_VERSION_RANGE(3000, 3001)
> > +		GRAPHICS_VERSION(3003)
> >  22019794406	GRAPHICS_VERSION(2001)
> >  		GRAPHICS_VERSION(2004)
> >  		GRAPHICS_VERSION_RANGE(3000, 3001)
> > +		GRAPHICS_VERSION(3003)
> >  22019338487	MEDIA_VERSION(2000)
> >  		GRAPHICS_VERSION(2001)
> >  		MEDIA_VERSION(3000), MEDIA_STEP(A0, B0),
> FUNC(xe_rtp_match_not_sriov_vf)
> > @@ -57,5 +59,6 @@ no_media_l3	MEDIA_VERSION(3000)
> >  		GRAPHICS_VERSION(1260), GRAPHICS_STEP(A0, B0)
> >  16023105232	GRAPHICS_VERSION_RANGE(2001, 3001)
> >  		MEDIA_VERSION_RANGE(1301, 3000)
> > -16026508708	GRAPHICS_VERSION_RANGE(1200, 3001)
> > +		GRAPHICS_VERSION(3003)
> > +16026508708	GRAPHICS_VERSION_RANGE(1200, 3003)
> >  		MEDIA_VERSION_RANGE(1300, 3000)
> > --
> > 2.34.1
> >