From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/i915/dp: Add YCBCR444 handling for sink formats Date: Mon, 25 May 2026 17:26:10 +1000 Message-ID: In-Reply-To: <20260523-color-format-v16-12-24340c5e4732@collabora.com> References: <20260523-color-format-v16-0-24340c5e4732@collabora.com> <20260523-color-format-v16-12-24340c5e4732@collabora.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review The `intel_dp_can_ycbcr444()` excludes branch devices (`drm_dp_is_branch`) = which is correct =E2=80=94 DP-to-HDMI dongles would need protocol conversio= n for YCbCr444. The check for `intel_dp->dfp.min_tmds_clock` combined with = `!intel_dp_has_hdmi_sink()` in `intel_dp_sink_format_valid()` for YCBCR444 = handles the DFP conversion case correctly. No issues. --- Generated by Claude Code Patch Reviewer