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: Update EDID and CEC phys addr in bridge detect() Date: Mon, 18 May 2026 16:41:48 +1000 Message-ID: In-Reply-To: <20260516183838.2024991-15-jonas@kwiboo.se> References: <20260516183838.2024991-1-jonas@kwiboo.se> <20260516183838.2024991-15-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:** New in v6, no Reviewed-by. Looks reasonable. This patch adds EDID/CEC update to the *bridge* `detect()` op (as opposed to the *connector* `detect()` which was handled in patch 12). This is described as a "temporary change" until dw-hdmi is fully converted to a HDMI bridge. ```c + dw_hdmi_connector_status_update(hdmi, connector, status); ``` This means EDID will be read during bridge detection. When using the bridge connector, this is the right behavior. --- Generated by Claude Code Patch Reviewer