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 dw_hdmi_connector_status_update() Date: Sat, 16 May 2026 16:21:44 +1000 Message-ID: In-Reply-To: <20260510124111.1226584-13-jonas@kwiboo.se> References: <20260510124111.1226584-1-jonas@kwiboo.se> <20260510124111.1226584-13-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 Calls the helper from both `detect()` and `force()`, and removes the EDID read from `get_modes()`. The helper now handles disconnect separately: ```c + if (status == connector_status_disconnected) { + drm_edid_connector_update(connector, NULL); + cec_notifier_phys_addr_invalidate(hdmi->cec_notifier); + return; + } ``` This ensures the connector always has up-to-date EDID after detect, matching bridge connector behavior. The `get_modes` function simplifies to just `drm_edid_connector_add_modes()`. Clean. --- Generated by Claude Code Patch Reviewer