Commit 939785d3 authored by Bean Huo's avatar Bean Huo Committed by Martin K. Petersen
Browse files

scsi: ufs: Remove unnecessary if condition in ufshcd_suspend() 

In the case that auto_bkops_enable is false, which means auto bkops has
been disabled, there is no need to call ufshcd_disable_auto_bkops().

Link: https://lore.kernel.org/r/20201125185300.3394-1-huobean@gmail.com


Reviewed-by: default avatarStanley Chu <stanley.chu@mediatek.com>
Reviewed-by: default avatarCan Guo <cang@codeaurora.org>
Signed-off-by: default avatarBean Huo <beanhuo@micron.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 623a4b6d

drivers/scsi/ufs/ufshcd.c

+1 −3
Original line number Diff line number Diff line
@@ -8609,11 +8609,9 @@ static int ufshcd_suspend(struct ufs_hba *hba, enum ufs_pm_op pm_op) 
	}



	if (req_dev_pwr_mode != hba->curr_dev_pwr_mode) {

		if ((ufshcd_is_runtime_pm(pm_op) && !hba->auto_bkops_enabled) ||

		    !ufshcd_is_runtime_pm(pm_op)) {

		if (!ufshcd_is_runtime_pm(pm_op))

			/* ensure that bkops is disabled */

			ufshcd_disable_auto_bkops(hba);

		}



		if (!hba->dev_info.b_rpm_dev_flush_capable) {

			ret = ufshcd_set_dev_pwr_mode(hba, req_dev_pwr_mode);