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/connector: Let connectors have a say in their color format Date: Mon, 25 May 2026 18:48:54 +1000 Message-ID: In-Reply-To: <20260522-color-format-v15-4-21fb136c9df2@collabora.com> References: <20260522-color-format-v15-0-21fb136c9df2@collabora.com> <20260522-color-format-v15-4-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 The `drm_connector_get_color_format()` indirection is a clean pattern. One = nit: the kernel-doc for this function is incomplete =E2=80=94 it's missing = the `Return:` section: ```c /** * drm_connector_get_color_format - Return connector color format of @conn_= state * @conn_state: pointer to the &struct drm_connector_state to go check * */ ``` Should document what it returns and when the override is used. --- Generated by Claude Code Patch Reviewer