diff options
author | Can Guo <cang@codeaurora.org> | 2021-01-20 11:04:23 +0100 |
---|---|---|
committer | Martin K. Petersen <martin.petersen@oracle.com> | 2021-01-21 04:23:56 +0100 |
commit | b02d51afca002a0cd29a22200a2ed81c5e1f58de (patch) | |
tree | 34d572a7088cfedc9e734f53fbc0ab69435082a4 /drivers/scsi/ufs | |
parent | scsi: ufs: Refactor ufshcd_init/exit_clk_scaling/gating() (diff) | |
download | linux-b02d51afca002a0cd29a22200a2ed81c5e1f58de.tar.xz linux-b02d51afca002a0cd29a22200a2ed81c5e1f58de.zip |
Revert "Make sure clk scaling happens only when HBA is runtime ACTIVE"
Commit 73cc291c2702 ("scsi: ufs: Make sure clk scaling happens only
when HBA is runtime ACTIVE") is no longer needed since commit
0e9d4ca43ba8 ("scsi: ufs: Protect some contexts from unexpected clock
scaling") is a more mature fix to protect UFS LLD stability from clock
scaling invoked through sysfs nodes by users.
Link: https://lore.kernel.org/r/1611137065-14266-4-git-send-email-cang@codeaurora.org
Reviewed-by: Stanley Chu <stanley.chu@mediatek.com>
Signed-off-by: Can Guo <cang@codeaurora.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/scsi/ufs')
-rw-r--r-- | drivers/scsi/ufs/ufshcd.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c index edb2987275ee..2c3c06bfd52b 100644 --- a/drivers/scsi/ufs/ufshcd.c +++ b/drivers/scsi/ufs/ufshcd.c @@ -1352,15 +1352,8 @@ static int ufshcd_devfreq_target(struct device *dev, } spin_unlock_irqrestore(hba->host->host_lock, irq_flags); - pm_runtime_get_noresume(hba->dev); - if (!pm_runtime_active(hba->dev)) { - pm_runtime_put_noidle(hba->dev); - ret = -EAGAIN; - goto out; - } start = ktime_get(); ret = ufshcd_devfreq_scale(hba, scale_up); - pm_runtime_put(hba->dev); trace_ufshcd_profile_clk_scaling(dev_name(hba->dev), (scale_up ? "up" : "down"), |