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: bridge: dw_hdmi: Use display_info is_hdmi and has_audio Date: Sat, 16 May 2026 16:21:44 +1000 Message-ID: In-Reply-To: <20260510124111.1226584-14-jonas@kwiboo.se> References: <20260510124111.1226584-1-jonas@kwiboo.se> <20260510124111.1226584-14-jonas@kwiboo.se> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Replaces manually-tracked `sink_is_hdmi`/`sink_has_audio` with `display->is_hdmi`/`display->has_audio` from `connector->display_info`. This is correct because `drm_edid_connector_update()` populates these fields, and patches 11-12 ensure that's called before they're needed. Also removes the stale `edid[512]` array and `HDMI_EDID_LEN` define that were completely unused. Removes the FIXME comment about this exact issue. Very clean. --- Generated by Claude Code Patch Reviewer