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: analogix_dp: Remove bridge disabing and panel unpreparing in analogix_dp_unbind() Date: Sun, 12 Apr 2026 11:37:42 +1000 Message-ID: In-Reply-To: <20260409071908.466294-4-damon.ding@rock-chips.com> References: <20260409065301.446670-1-damon.ding@rock-chips.com> <20260409071908.466294-4-damon.ding@rock-chips.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Removes `analogix_dp_bridge_disable()` and `drm_panel_unprepare()` calls from `analogix_dp_unbind()`. The rationale is that unbind should be symmetric with bind, and bind doesn't enable/prepare. The framework should handle disable/unprepare during the normal atomic shutdown path before unbind is called. This is correct - the DRM framework ensures the display pipeline is disabled before component unbind. No issues. --- Generated by Claude Code Patch Reviewer