From 722dccd8b5ac9fb9667b14288ae554b08a2a02b3 Mon Sep 17 00:00:00 2001 From: Asutosh Das Date: Wed, 29 Apr 2020 16:36:40 -0700 Subject: [PATCH] scsi: ufshcd-qti: Keep the correct device and link state If the device and link are marked as off, suspend routine wouldn't turn-off the device and link. But the regulators would be put to lpm mode. However, the ufs device is still drawing active mode current. Thus this leads to a OCP in PMIC. Hence, don't mark the device and link as off in case of error, if it doesn't reflect the correct state. Change-Id: I8d5a738a92a1f339c0500a617f55cb77009356b2 Signed-off-by: Asutosh Das --- drivers/scsi/ufs/ufshcd-qti.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/drivers/scsi/ufs/ufshcd-qti.c b/drivers/scsi/ufs/ufshcd-qti.c index 2cb416634709..4cb71940b793 100644 --- a/drivers/scsi/ufs/ufshcd-qti.c +++ b/drivers/scsi/ufs/ufshcd-qti.c @@ -8355,11 +8355,6 @@ static int ufshcd_probe_hba(struct ufs_hba *hba) ufshcd_auto_hibern8_enable(hba); out: - if (ret) { - ufshcd_set_ufs_dev_poweroff(hba); - ufshcd_set_link_off(hba); - } - /* * If we failed to initialize the device or the device is not * present, turn off the power/clocks etc.