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: Mon, 18 May 2026 16:41:48 +1000 Message-ID: In-Reply-To: <20260516183838.2024991-17-jonas@kwiboo.se> References: <20260516183838.2024991-1-jonas@kwiboo.se> <20260516183838.2024991-17-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 **Status:** Correct cleanup. Replaces `hdmi->sink_is_hdmi` with `display->is_hdmi` and `hdmi->sink_has_audio` with `display->has_audio`, removing the stale EDID parsing code that used deprecated `drm_detect_hdmi_monitor()` and `drm_detect_monitor_audio()`. Also removes the unused `edid[512]` array and `HDMI_EDID_LEN` define. The old code even had a FIXME comment saying this should be done. --- Generated by Claude Code Patch Reviewer