From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: scsi: ufs: qcom: keep parallel track of PHY power state Date: Sun, 22 Mar 2026 04:07:13 +1000 Message-ID: In-Reply-To: <20260319223241.1351137-11-vladimir.oltean@nxp.com> References: <20260319223241.1351137-1-vladimir.oltean@nxp.com> <20260319223241.1351137-11-vladimir.oltean@nxp.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Same pattern as patch 09 but for Qualcomm UFS. The commit message correctly notes that `phy_power_off()` errors are discarded. The tracking in `ufs_qcom_setup_clocks()` correctly updates `phy_powered_on` after both `phy_power_off()` (PRE_CHANGE) and `phy_power_on()` (POST_CHANGE) -- but only if they succeed (the existing error paths return early). Also updates `ufs_qcom_exit()`. Looks correct. --- Generated by Claude Code Patch Reviewer