> -----Original Message-----
> From: Shankar, Uma
> Sent: Monday, August 31, 2026 6:54 PM
> To: Borah, Chaitanya Kumar <[email protected]>; intel-
> [email protected]; [email protected]; dri-
> [email protected]
> Subject: RE: [v6 6/9] drm/i915/color: Add support for 1D LUT in SDR planes
>
>
>
> > -----Original Message-----
> > From: Borah, Chaitanya Kumar <[email protected]>
> > Sent: Tuesday, August 4, 2026 6:01 PM
> > To: [email protected]; [email protected];
> > dri- [email protected]
> > Cc: Borah, Chaitanya Kumar <[email protected]>; Shankar,
> > Uma <[email protected]>
> > Subject: [v6 6/9] drm/i915/color: Add support for 1D LUT in SDR planes
> >
> > Extend the SDR plane color pipeline to post-CSC 1D LUT block.
> >
> > v2:
> > - In preparation of a simple pipeline
> > [YUV Full/Limited -> RGB] -> [1D LUT]
> > Drop pre-CSC LUT from the pipeline as it has no use in a YUV -> RGB
> > pipeline. This makes the pipeline simple since the block lies between
> > the YUV range correct block and Fixed function CSC. It can be added back
> > when [RGB709 -> RGB2020] capability is added. Then it can be used for
> > linearization.
> >
> > Signed-off-by: Chaitanya Kumar Borah <[email protected]>
> > Reviewed-by: Uma Shankar <[email protected]> #v1
> > ---
> > drivers/gpu/drm/i915/display/intel_color_pipeline.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_color_pipeline.c
> > b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
> > index efd4375c4331..53e55ce0a5a3 100644
> > --- a/drivers/gpu/drm/i915/display/intel_color_pipeline.c
> > +++ b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
> > @@ -45,6 +45,7 @@ static const enum intel_color_block
> > hdr_plane_pipeline[] = {
> >
> > static const enum intel_color_block sdr_plane_yuv_pipeline[] = {
> > INTEL_PLANE_CB_CSC_FF,
> > + INTEL_PLANE_CB_POST_CSC_LUT,
>
> With this we will expose support for post csc lut but corresponding handling
> in
> driver is added later in the series. Would be good to move this after patch 8.
Please ignore the earlier comment, enabling of SDR planes is done in last patch
so this should be fine. We are good.
> > };
> >
> > static const u64 intel_plane_supported_csc_ff =
> > --
> > 2.50.1
