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: Unregister CEC notifier during connector cleanup Date: Sat, 16 May 2026 16:21:43 +1000 Message-ID: In-Reply-To: <20260510124111.1226584-9-jonas@kwiboo.se> References: <20260510124111.1226584-1-jonas@kwiboo.se> <20260510124111.1226584-9-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 Moves CEC notifier unregistration from `dw_hdmi_bridge_detach()` to `dw_hdmi_connector_destroy()`, aligning CEC notifier lifetime with the connector. This makes sense because the notifier was registered during connector creation. The bridge detach callback is removed entirely. Note: patch 14 later replaces this with `drmm_connector_hdmi_cec_notifier_register()`, which handles cleanup automatically, so the code added here is transitional. That's fine for bisectability. --- Generated by Claude Code Patch Reviewer