[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