Re: [PATCH 0/3] scsi: ufs: qcom: Suspend fixes

[Bart Van Assche]

On 12/11/24 9:40 AM, Manivannan Sadhasivam via B4 Relay wrote:
> This series fixes the several suspend issues on Qcom platforms. Patch 1 fixes
> the resume failure with spm_lvl=5 suspend on most of the Qcom platforms. For
> this patch, I couldn't figure out the exact commit that caused the issue. So I
> used the commit that introduced reinit support as a placeholder.
>
> Patch 3 fixes the suspend issue on SM8550 and SM8650 platforms where UFS
> PHY retention is not supported. Hence the default spm_lvl=3 suspend fails. So
> this patch configures spm_lvl=5 as the default suspend level to force UFSHC/
> device powerdown during suspend. This supersedes the previous series [1] that
> tried to fix the issue in clock drivers.

If Avri's comment is addressed, feel free to add my reviewed-by to this
series.

Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>