Re: [PATCH V12 48/49] drm/colorop: Add DRM_COLOROP_1D_CURVE_GAMMA22 to 1D Curve

[Harry Wentland <harry.wentland@amd.com>]

On 2025-10-29 23:42, Alex Hung wrote:
> Add "DRM_COLOROP_1D_CURVE_GAMMA22" and DRM_COLOROP_1D_CURVE_GAMMA22_INV
> subtypes to drm_colorop of DRM_COLOROP_1D_CURVE.
> 
> Signed-off-by: Alex Hung <alex.hung@amd.com>

Reviewed-by: Harry Wentland <harry.wentland@amd.com>

Harry

> ---
> v12:
>  - Add gamma 2.2 (Sebastian Wick & Xaver Hugl & Pekka)
> 
>  drivers/gpu/drm/drm_colorop.c |  2 ++
>  include/drm/drm_colorop.h     | 18 ++++++++++++++++++
>  2 files changed, 20 insertions(+)
> 
> diff --git a/drivers/gpu/drm/drm_colorop.c b/drivers/gpu/drm/drm_colorop.c
> index e1b2b446faf2..4e54bd46be7a 100644
> --- a/drivers/gpu/drm/drm_colorop.c
> +++ b/drivers/gpu/drm/drm_colorop.c
> @@ -77,6 +77,8 @@ static const char * const colorop_curve_1d_type_names[] = {
>  	[DRM_COLOROP_1D_CURVE_PQ_125_INV_EOTF] = "PQ 125 Inverse EOTF",
>  	[DRM_COLOROP_1D_CURVE_BT2020_INV_OETF] = "BT.2020 Inverse OETF",
>  	[DRM_COLOROP_1D_CURVE_BT2020_OETF] = "BT.2020 OETF",
> +	[DRM_COLOROP_1D_CURVE_GAMMA22] = "Gamma 2.2",
> +	[DRM_COLOROP_1D_CURVE_GAMMA22_INV] = "Gamma 2.2 Inverse",
>  };
>  
>  static const struct drm_prop_enum_list drm_colorop_lut1d_interpolation_list[] = {
> diff --git a/include/drm/drm_colorop.h b/include/drm/drm_colorop.h
> index 3e70f66940e0..01e6774c482e 100644
> --- a/include/drm/drm_colorop.h
> +++ b/include/drm/drm_colorop.h
> @@ -108,6 +108,24 @@ enum drm_colorop_curve_1d_type {
>  	 */
>  	DRM_COLOROP_1D_CURVE_BT2020_OETF,
>  
> +	/**
> +	 * @DRM_COLOROP_1D_CURVE_GAMMA22:
> +	 *
> +	 * enum string "Gamma 2.2"
> +	 *
> +	 * A gamma 2.2 power function. This applies a power curve with
> +	 * gamma value of 2.2 to the input values.
> +	 */
> +	DRM_COLOROP_1D_CURVE_GAMMA22,
> +
> +	/**
> +	 * @DRM_COLOROP_1D_CURVE_GAMMA22_INV:
> +	 *
> +	 * enum string "Gamma 2.2 Inverse"
> +	 *
> +	 * The inverse of &DRM_COLOROP_1D_CURVE_GAMMA22
> +	 */
> +	DRM_COLOROP_1D_CURVE_GAMMA22_INV,
>  	/**
>  	 * @DRM_COLOROP_1D_CURVE_COUNT:
>  	 *