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/hdmi: Implement "color format" DRM property Date: Mon, 25 May 2026 18:48:56 +1000 Message-ID: In-Reply-To: <20260522-color-format-v15-13-21fb136c9df2@collabora.com> References: <20260522-color-format-v15-0-21fb136c9df2@collabora.com> <20260522-color-format-v15-13-21fb136c9df2@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 Clean refactoring of `intel_hdmi_compute_formats` into `_auto` and explicit= variants. The property registration in `intel_hdmi_attach_color_format_pro= perty` correctly gates YCbCr420 on `ycbcr_420_allowed` and YCbCr444 on `int= el_hdmi_can_ycbcr444()`. Note: YCbCr422 is not offered. The commit message says "RGB, YCbCr420, YCbC= r444" which matches. This is fine =E2=80=94 i915 HDMI apparently doesn't su= pport 422 output. However, this inherits the potential issue from patch 11 regarding `intel_h= dmi_can_ycbcr444`. --- Generated by Claude Code Patch Reviewer