[PATCH 1/3] drm/xe/pxp: Use the correct define in the set_property_funcs array

John Harrison john.c.harrison at intel.com
Wed May 21 20:40:23 UTC 2025


On 5/19/2025 4:32 PM, Daniele Ceraolo Spurio wrote:
> The define of the extension type was accidentally used instead of the
> one of the property itself. They're both zero, so no functional issue,
> but we should use the correct define for code correctness.
>
> Fixes: 41a97c4a1294 ("drm/xe/pxp/uapi: Add API to mark a BO as using PXP")
> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> Cc: John Harrison <John.C.Harrison at Intel.com>
Reviewed-by: John Harrison <John.C.Harrison at Intel.com>

> ---
>   drivers/gpu/drm/xe/xe_bo.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_bo.c b/drivers/gpu/drm/xe/xe_bo.c
> index d99d91fe8aa9..99e309642a1c 100644
> --- a/drivers/gpu/drm/xe/xe_bo.c
> +++ b/drivers/gpu/drm/xe/xe_bo.c
> @@ -2557,7 +2557,7 @@ typedef int (*xe_gem_create_set_property_fn)(struct xe_device *xe,
>   					     u64 value);
>   
>   static const xe_gem_create_set_property_fn gem_create_set_property_funcs[] = {
> -	[DRM_XE_GEM_CREATE_EXTENSION_SET_PROPERTY] = gem_create_set_pxp_type,
> +	[DRM_XE_GEM_CREATE_SET_PROPERTY_PXP_TYPE] = gem_create_set_pxp_type,
>   };
>   
>   static int gem_create_user_ext_set_property(struct xe_device *xe,



More information about the Intel-xe mailing list