Re: [PATCH v3 09/13] drm/xe/pat: Report missing PAT table using structured xe_log API

["Mallesh, Koujalagi" <mallesh.koujalagi@intel.com>]

On 08-09-2026 04:15 pm, Dnyaneshwar Bhadane wrote:
> Replace drm_err() with xe_log_err() using the PAT component tag
> when a platform has no matching PAT table entry.
>
> v2:
> - correction in the error code.(shashiko)

Move to v2: portion under --- like patch:

https://patchwork.freedesktop.org/patch/746476/?series=171022&rev=5

> Signed-off-by: Dnyaneshwar Bhadane <dnyaneshwar.bhadane@intel.com>
With that
Reviewed-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
> ---
>   drivers/gpu/drm/xe/xe_pat.c | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_pat.c b/drivers/gpu/drm/xe/xe_pat.c
> index 8af92e3c4c17..1deecfd80f30 100644
> --- a/drivers/gpu/drm/xe/xe_pat.c
> +++ b/drivers/gpu/drm/xe/xe_pat.c
> @@ -16,6 +16,7 @@
>   #include "xe_force_wake.h"
>   #include "xe_gt.h"
>   #include "xe_gt_mcr.h"
> +#include "xe_log.h"
>   #include "xe_mmio.h"
>   #include "xe_sriov.h"
>   #include "xe_wa.h"
> @@ -678,8 +679,9 @@ int xe_pat_init_early(struct xe_device *xe)
>   		xe->pat.idx[XE_CACHE_WT] = 2;
>   		xe->pat.idx[XE_CACHE_WB] = 0;
>   	} else {
> -		drm_err(&xe->drm, "Missing PAT table for platform with graphics version %d.%02d!\n",
> -			GRAPHICS_VER(xe), GRAPHICS_VERx100(xe) % 100);
> +		xe_log_err(xe, PAT, -ENODATA,
> +			   "Missing PAT table for platform with graphics version %d.%02d!\n",
> +			   GRAPHICS_VER(xe), GRAPHICS_VERx100(xe) % 100);
>   		return -ENODATA;
>   
>   	}