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/msm/dp: re-arrange dp_display_disable() into functional parts Date: Thu, 04 Jun 2026 13:01:24 +1000 Message-ID: In-Reply-To: <20260602-dp_mstclean-v6-9-2c17ff40a9b2@oss.qualcomm.com> References: <20260602-dp_mstclean-v6-0-2c17ff40a9b2@oss.qualcomm.com> <20260602-dp_mstclean-v6-9-2c17ff40a9b2@oss.qualcomm.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Replaces `msm_dp_ctrl_off_link_stream()` with calls to `msm_dp_ctrl_off()` + `msm_dp_ctrl_reinit_phy()`. The old function combined these with pixel clock disable, link disable, and phy reinit. Note that `msm_dp_panel_disable_vsc_sdp()` is moved from `msm_dp_ctrl_off_link_stream()` up into `msm_dp_display_disable()`. This means it's now called in BOTH the sink_count==0 and unplug paths, matching the existing `msm_dp_ctrl_off()` behavior. Correct. **No issues.** --- Generated by Claude Code Patch Reviewer