Re: [PATCH v3 02/12] lib/igt_kms: Add IGT_COLOROP_FIXED_MATRIX property

[Alex Hung <alex.hung@amd.com>]

On 6/23/26 11:57, Harry Wentland wrote:
> Add support for the Fixed Matrix colorop property:
> - Add IGT_COLOROP_FIXED_MATRIX property to the colorop property enum
>    and names array in igt_kms
> 
> Assisted-by: Claude:claude-opus-4-6
> Signed-off-by: Harry Wentland <harry.wentland@amd.com>
> ---
>   lib/igt_kms.c | 1 +
>   lib/igt_kms.h | 1 +
>   2 files changed, 2 insertions(+)
> 
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index e82d32130666..9003b2b9e670 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -725,6 +725,7 @@ const char * const igt_colorop_prop_names[IGT_NUM_COLOROP_PROPS] = {
>   	[IGT_COLOROP_MULTIPLIER] = "MULTIPLIER",
>   	[IGT_COLOROP_LUT3D_INTERPOLATION] = "LUT3D_INTERPOLATION",
>   	[IGT_COLOROP_NEXT] = "NEXT",
> +	[IGT_COLOROP_FIXED_MATRIX] = "FIXED_MATRIX_TYPE",
Should we move IGT_COLOROP_FIXED_MATRIX before NEXT?
>   };
>   
>   const char * const igt_crtc_prop_names[IGT_NUM_CRTC_PROPS] = {
> diff --git a/lib/igt_kms.h b/lib/igt_kms.h
> index 3df3ba86630e..1be58288e119 100644
> --- a/lib/igt_kms.h
> +++ b/lib/igt_kms.h
> @@ -363,6 +363,7 @@ enum igt_atomic_colorop_properties {
>   	IGT_COLOROP_MULTIPLIER,
>   	IGT_COLOROP_LUT3D_INTERPOLATION,
>   	IGT_COLOROP_NEXT,
> +	IGT_COLOROP_FIXED_MATRIX,
Should we move IGT_COLOROP_FIXED_MATRIX before NEXT?
>   	IGT_NUM_COLOROP_PROPS
>   };
>