Re: [PATCH v13 00/27] Add new general DRM property "color format"

Nicolas Frattaroli
2026-04-13T10:27:27Z

On Monday, 13 April 2026 12:07:14 Central European Summer Time Nicolas Frattaroli wrote:
> Hello,
>
> [snip mucho texto]
>
> ---
> Nicolas Frattaroli (26):
> drm/display: hdmi-state-helper: Use default case for unsupported formats
> drm: Add new general DRM property "color format"
> drm/connector: Let connectors have a say in their color format
> drm/display: bridge_connector: Use HDMI color format for HDMI conns
> drm/bridge: Act on the DRM color format property
> drm/atomic-helper: Add HDMI bridge output bus formats helper
> drm/display: hdmi-state-helper: Act on color format DRM property
> drm/display: hdmi-state-helper: Try subsampling in mode_valid
> drm/amdgpu: Implement "color format" DRM property
> drm/i915/hdmi: Add YCBCR444 handling for sink formats
> drm/i915/dp: Add YCBCR444 handling for sink formats
> drm/i915: Implement the "color format" DRM property
> drm/rockchip: Add YUV422 output mode constants for VOP2
> drm/rockchip: vop2: Add RK3576 to the RG swap special case
> drm/rockchip: vop2: Recognise 10-bit YUV422 as YUV format
> drm/rockchip: vop2: Set correct output format for RK3576 YUV422
> drm/bridge: dw-hdmi-qp: Use common HDMI output bus fmts helper
> drm/rockchip: dw_hdmi_qp: Implement "color format" DRM property
> drm/rockchip: dw_hdmi_qp: Set supported_formats platdata
> drm/connector: Register color format property on HDMI connectors
> drm/tests: hdmi: Add tests for the color_format property
> drm/tests: hdmi: Add tests for HDMI helper's mode_valid
> drm/tests: bridge: Add KUnit tests for bridge chain format selection
> drm/tests: bridge: Add test for HDMI output bus formats helper
> drm/bridge: Document bridge chain format selection
> drm/connector: Update docs of "colorspace" for color format prop
>
> Werner Sembach (1):
> drm/amd/display: Remove unnecessary SIGNAL_TYPE_HDMI_TYPE_A check
>
> Documentation/gpu/drm-kms-helpers.rst | 6 +
> Documentation/gpu/drm-kms.rst | 6 +
> drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 91 +-
> drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c | 1 +
> drivers/gpu/drm/display/drm_bridge_connector.c | 24 +
> drivers/gpu/drm/display/drm_hdmi_state_helper.c | 53 +-
> drivers/gpu/drm/drm_atomic_helper.c | 86 ++
> drivers/gpu/drm/drm_atomic_uapi.c | 11 +
> drivers/gpu/drm/drm_bridge.c | 104 ++-
> drivers/gpu/drm/drm_connector.c | 178 +++-
> drivers/gpu/drm/i915/display/intel_connector.c | 10 +
> drivers/gpu/drm/i915/display/intel_connector.h | 1 +
> drivers/gpu/drm/i915/display/intel_dp.c | 43 +-
> drivers/gpu/drm/i915/display/intel_hdmi.c | 47 +-
> drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c | 111 ++-
> drivers/gpu/drm/rockchip/rockchip_drm_drv.h | 4 +
> drivers/gpu/drm/rockchip/rockchip_drm_vop2.c | 21 +-
> drivers/gpu/drm/tests/drm_bridge_test.c | 971 +++++++++++++++++++++
> drivers/gpu/drm/tests/drm_hdmi_state_helper_test.c | 345 ++++++++
> include/drm/drm_atomic_helper.h | 7 +
> include/drm/drm_connector.h | 120 +++
> 21 files changed, 2203 insertions(+), 37 deletions(-)
> ---
> base-commit: e3dc6072028a594d5e9bfc6476165842602d5307
> change-id: 20251028-color-format-49fd202b7183
> prerequisite-message-id: <20260409101539.22032-1-ville.syrjala@linux.intel.com>
> prerequisite-patch-id: f382aeb5da5f2b8f6e2cb22b88eb47f490f2c724
> prerequisite-patch-id: 20570aeb28e3c31353e6f697b193b23d8b47c47c
> prerequisite-patch-id: 1b7e24034883b22cd82be025a1cf82ae77170fd0
> prerequisite-patch-id: 6ec2dc2c05a75391b67cb12d93168f5e8da8ec55
> prerequisite-patch-id: 32e84581998ef4eef05e1681c7ec36b90f2a6bb7
> prerequisite-patch-id: 99c2187a846b0c9ac2ea1a892c17483120cb7da1
> prerequisite-patch-id: fb41b4668a3b7c8c375c67ffd6b178fa3273e86a
> prerequisite-patch-id: 4c115a36eea0d5f80643dc34310690894ac80e0e
> prerequisite-patch-id: 6fdec0832cd6062de3cc5c2f363c5d624d8a00f9
>
> Best regards,
> --
> Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
>
>

Oops, forgot to apply the review trailers from last revision. (I did
actually remember, but didn't remember to explicitly confirm applying
them, and it defaults to not doing so.)

So these two:
- drm/atomic-helper: Add HDMI bridge output bus formats helper
- drm/connector: Register color format property on HDMI connectors

Lack Dmitry Baryshkov's R-b, but I assume it'll get picked up
on the next `b4 trailers -u` anyway. Just mentioning it though
to ruin the surprise for any reviewer who feels gaslit otherwise.

Kind regards,
Nicolas Frattaroli