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/amd/display: Remove unnecessary SIGNAL_TYPE_HDMI_TYPE_A check Date: Mon, 25 May 2026 18:48:53 +1000 Message-ID: In-Reply-To: <20260522-color-format-v15-1-21fb136c9df2@collabora.com> References: <20260522-color-format-v15-0-21fb136c9df2@collabora.com> <20260522-color-format-v15-1-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 simplification. Removing the HDMI-only guard on `drm_mode_is_420_only= ()` makes sense =E2=80=94 if the EDID says 420-only, the signal type should= n't matter. The condition rewrite from `||` of two separate branches into o= ne combined `if` is correct. No issues. --- Generated by Claude Code Patch Reviewer