[PATCH] drm/xe/xe_pat: Adding the graphics version for BMG-G31

[Matthew Auld]

On 28/07/2025 09:16, nakshtra.goyal at intel.com wrote:
> From: Nakshtra Goyal <nakshtra.goyal at intel.com>
> 
> Adding the corresponding graphics version for bmg_g31 i.e 20.02
> Bspec 68090
> 
> Signed-off-by: Nakshtra Goyal <nakshtra.goyal at intel.com>
> ---
>   drivers/gpu/drm/xe/xe_pat.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_pat.c b/drivers/gpu/drm/xe/xe_pat.c
> index 2e7cb99ae87a..a0872f84dc29 100644
> --- a/drivers/gpu/drm/xe/xe_pat.c
> +++ b/drivers/gpu/drm/xe/xe_pat.c
> @@ -385,7 +385,7 @@ void xe_pat_init_early(struct xe_device *xe)
>   			xe->pat.pat_pta = &xe2_pat_pta;
>   
>   		/* Wa_16023588340. XXX: Should use XE_WA */
> -		if (GRAPHICS_VERx100(xe) == 2001)
> +		if (GRAPHICS_VERx100(xe) == 2001 || GRAPHICS_VERx100(xe) == 2002)

This reason we need this is tied to Wa_16023588340, since CLOS3 is 
hard-wired to controlling the behaviour of the caching of VRAM from host 
side when that WA is turned on, and so we don't want UMD interacting 
with CLOS3 on that platform. But AFAICT we don't currently enable this 
WA on BMG 2002, see xe_wa_oob.rules, so not sure why we need this 
change? Also I'm fairly sure that the WA is not meant to be needed on 
BMG g31. Do you have some more information?

>   			xe->pat.n_entries = 28; /* Disable CLOS3 */
>   		else
>   			xe->pat.n_entries = ARRAY_SIZE(xe2_pat_table);