> -----Original Message-----
> From: Borah, Chaitanya Kumar <[email protected]>
> Sent: Monday, May 11, 2026 11:02 AM
> To: [email protected]; [email protected]
> Cc: [email protected]; Shankar, Uma <[email protected]>;
> Borah, Chaitanya Kumar <[email protected]>; Samala, Pranay
> <[email protected]>; [email protected]; Srinivas, Vidya
> <[email protected]>
> Subject: [PATCH v3 1/4] drm/i915/display: Copy color pipeline from plane in
> the
> primary joiner pipe
>
> When copying plane color state in a joiner configuration, use the plane in the
> primary joiner pipe since it carries the pipeline number selected by the user-
> space.
>
> This assumes that all pipes in the joiner are symmetric in their plane color
> capabilities.
Looks Good to me.
Reviewed-by: Uma Shankar <[email protected]>
> Cc: [email protected] # v6.19+
> Fixes: a78f1b6baf4d ("drm/i915/color: Add framework to program CSC")
> Tested-by: Vidya Srinivas <[email protected]>
> Signed-off-by: Chaitanya Kumar Borah <[email protected]>
> ---
> drivers/gpu/drm/i915/display/intel_plane.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_plane.c
> b/drivers/gpu/drm/i915/display/intel_plane.c
> index c181a7d063ec..e403fe4a8a20 100644
> --- a/drivers/gpu/drm/i915/display/intel_plane.c
> +++ b/drivers/gpu/drm/i915/display/intel_plane.c
> @@ -396,7 +396,7 @@ intel_plane_color_copy_uapi_to_hw_state(struct
> intel_plane_state *plane_state,
> bool changed = false;
> int i = 0;
>
> - iter_colorop = plane_state->uapi.color_pipeline;
> + iter_colorop = from_plane_state->uapi.color_pipeline;
>
> while (iter_colorop) {
> for_each_new_colorop_in_state(state, colorop,
> new_colorop_state, i) {
> --
> 2.25.1
