Hi, Gentle ping on this one - it has had no review since July.
I re-checked it against the tree today rather than assuming it still stands. On drm-misc-next (8ce26b09f652) all five hunks still apply at their original line numbers, with no offset and no fuzz, and nothing merged since exposes these properties. So the gap the patch describes is unchanged: every YUV plane is still converted with the BT.709 limited range matrix, which is wrong for BT.601 (SD) and BT.2020 content. One thing worth flagging, since it decides the merge order rather than the content. Cristian's "drm/rockchip: vop2: Switch to enum vop_csc_format" (10/14 of the VOP2 series) touches the same two lines of vop2_setup_csc_mode() that this patch touches. The two changes are complementary rather than conflicting: that patch tightens the type of csc_mode and replaces the "csc_mode = false" assignment, while this one replaces where the value comes from - the hardcoded input_csc is dropped in favour of the plane's COLOR_ENCODING and COLOR_RANGE. Whichever lands second needs a small rebase. I am happy to do that rebase in either direction. If you would rather have this on top of the VOP2 series, say so and I will send a v2 once it lands, with vop2_convert_color_encoding() returning enum vop_csc_format so the two fit together properly. Thanks, Igor
