Commit 089f5b64 authored Apr 13, 2020 by Asutosh Das Committed by Martin K. Petersen Apr 17, 2020

scsi: ufs: full reinit upon resume if link was off

During suspend, if the link is put to off, it would require a full
initialization during resume. This patch resets and restores both the host
and the card during initialization, otherwise host-only reset and restore
would fail occasionally.

Link: https://lore.kernel.org/r/1586844892-22720-1-git-send-email-cang@codeaurora.org


Reviewed-by: Bean Huo <beanhuo@micron.com>
Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com>
Acked-by: Stanley Chu <stanley.chu@mediatek.com>
Acked-by: Avri Altman <Avri.Altman@wdc.com>
Signed-off-by: Asutosh Das <asutoshd@codeaurora.org>
Signed-off-by: Can Guo <cang@codeaurora.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

parent 9ae58380

drivers/scsi/ufs/ufshcd.c

+6 −2

Original line number	Diff line number	Diff line
		@@ -8055,9 +8055,13 @@ static int ufshcd_resume(struct ufs_hba *hba, enum ufs_pm_op pm_op)
		else
		goto vendor_suspend;
		} else if (ufshcd_is_link_off(hba)) {
		ret = ufshcd_host_reset_and_restore(hba);
		/*
		* ufshcd_host_reset_and_restore() should have already
		* A full initialization of the host and the device is
		* required since the link was put to off during suspend.
		*/
		ret = ufshcd_reset_and_restore(hba);
		/*
		* ufshcd_reset_and_restore() should have already
		* set the link state as active
		*/
		if (ret \|\| !ufshcd_is_link_active(hba))