Commit a508253d authored by Gilad Broner's avatar Gilad Broner Committed by Martin K. Petersen

scsi: ufs: suspend clock scaling for failed runtime_resume

During runtime resume operation, clock scaling may get indirectly
resumed via call to ufshcd_set_dev_pwr_mode(): Start/Stop Unit command
times out and SCSI error handling ultimately calls the host reset
handler to recover, during which clock scaling is resumed.  Error case
exit path of runtime resume will disable clocks.  As clock scaling was
already resumed, it will get scheduled later on and try to access UFS
registers while clocks are disabled, resulting in unclocked register
access.
Signed-off-by: default avatarGilad Broner <gbroner@codeaurora.org>
Signed-off-by: default avatarSubhash Jadavani <subhashj@codeaurora.org>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent dcea0bfb
...@@ -600,6 +600,20 @@ static bool ufshcd_is_unipro_pa_params_tuning_req(struct ufs_hba *hba) ...@@ -600,6 +600,20 @@ static bool ufshcd_is_unipro_pa_params_tuning_req(struct ufs_hba *hba)
return false; return false;
} }
static void ufshcd_suspend_clkscaling(struct ufs_hba *hba)
{
if (ufshcd_is_clkscaling_enabled(hba)) {
devfreq_suspend_device(hba->devfreq);
hba->clk_scaling.window_start_t = 0;
}
}
static void ufshcd_resume_clkscaling(struct ufs_hba *hba)
{
if (ufshcd_is_clkscaling_enabled(hba))
devfreq_resume_device(hba->devfreq);
}
static void ufshcd_ungate_work(struct work_struct *work) static void ufshcd_ungate_work(struct work_struct *work)
{ {
int ret; int ret;
...@@ -633,8 +647,7 @@ static void ufshcd_ungate_work(struct work_struct *work) ...@@ -633,8 +647,7 @@ static void ufshcd_ungate_work(struct work_struct *work)
hba->clk_gating.is_suspended = false; hba->clk_gating.is_suspended = false;
} }
unblock_reqs: unblock_reqs:
if (ufshcd_is_clkscaling_enabled(hba)) ufshcd_resume_clkscaling(hba);
devfreq_resume_device(hba->devfreq);
scsi_unblock_requests(hba->host); scsi_unblock_requests(hba->host);
} }
...@@ -733,10 +746,7 @@ static void ufshcd_gate_work(struct work_struct *work) ...@@ -733,10 +746,7 @@ static void ufshcd_gate_work(struct work_struct *work)
ufshcd_set_link_hibern8(hba); ufshcd_set_link_hibern8(hba);
} }
if (ufshcd_is_clkscaling_enabled(hba)) { ufshcd_suspend_clkscaling(hba);
devfreq_suspend_device(hba->devfreq);
hba->clk_scaling.window_start_t = 0;
}
if (!ufshcd_is_link_active(hba)) if (!ufshcd_is_link_active(hba))
ufshcd_setup_clocks(hba, false); ufshcd_setup_clocks(hba, false);
...@@ -5076,8 +5086,7 @@ static int ufshcd_probe_hba(struct ufs_hba *hba) ...@@ -5076,8 +5086,7 @@ static int ufshcd_probe_hba(struct ufs_hba *hba)
hba->is_init_prefetch = true; hba->is_init_prefetch = true;
/* Resume devfreq after UFS device is detected */ /* Resume devfreq after UFS device is detected */
if (ufshcd_is_clkscaling_enabled(hba)) ufshcd_resume_clkscaling(hba);
devfreq_resume_device(hba->devfreq);
out: out:
/* /*
...@@ -5583,6 +5592,7 @@ static void ufshcd_hba_exit(struct ufs_hba *hba) ...@@ -5583,6 +5592,7 @@ static void ufshcd_hba_exit(struct ufs_hba *hba)
if (hba->is_powered) { if (hba->is_powered) {
ufshcd_variant_hba_exit(hba); ufshcd_variant_hba_exit(hba);
ufshcd_setup_vreg(hba, false); ufshcd_setup_vreg(hba, false);
ufshcd_suspend_clkscaling(hba);
ufshcd_setup_clocks(hba, false); ufshcd_setup_clocks(hba, false);
ufshcd_setup_hba_vreg(hba, false); ufshcd_setup_hba_vreg(hba, false);
hba->is_powered = false; hba->is_powered = false;
...@@ -5911,10 +5921,8 @@ static int ufshcd_suspend(struct ufs_hba *hba, enum ufs_pm_op pm_op) ...@@ -5911,10 +5921,8 @@ static int ufshcd_suspend(struct ufs_hba *hba, enum ufs_pm_op pm_op)
* for pending clock scaling work to be done before clocks are * for pending clock scaling work to be done before clocks are
* turned off. * turned off.
*/ */
if (ufshcd_is_clkscaling_enabled(hba)) { ufshcd_suspend_clkscaling(hba);
devfreq_suspend_device(hba->devfreq);
hba->clk_scaling.window_start_t = 0;
}
/* /*
* Call vendor specific suspend callback. As these callbacks may access * Call vendor specific suspend callback. As these callbacks may access
* vendor specific host controller register space call them before the * vendor specific host controller register space call them before the
...@@ -5941,6 +5949,7 @@ static int ufshcd_suspend(struct ufs_hba *hba, enum ufs_pm_op pm_op) ...@@ -5941,6 +5949,7 @@ static int ufshcd_suspend(struct ufs_hba *hba, enum ufs_pm_op pm_op)
goto out; goto out;
set_link_active: set_link_active:
ufshcd_resume_clkscaling(hba);
ufshcd_vreg_set_hpm(hba); ufshcd_vreg_set_hpm(hba);
if (ufshcd_is_link_hibern8(hba) && !ufshcd_uic_hibern8_exit(hba)) if (ufshcd_is_link_hibern8(hba) && !ufshcd_uic_hibern8_exit(hba))
ufshcd_set_link_active(hba); ufshcd_set_link_active(hba);
...@@ -6028,8 +6037,7 @@ static int ufshcd_resume(struct ufs_hba *hba, enum ufs_pm_op pm_op) ...@@ -6028,8 +6037,7 @@ static int ufshcd_resume(struct ufs_hba *hba, enum ufs_pm_op pm_op)
ufshcd_urgent_bkops(hba); ufshcd_urgent_bkops(hba);
hba->clk_gating.is_suspended = false; hba->clk_gating.is_suspended = false;
if (ufshcd_is_clkscaling_enabled(hba)) ufshcd_resume_clkscaling(hba);
devfreq_resume_device(hba->devfreq);
/* Schedule clock gating in case of no access to UFS device yet */ /* Schedule clock gating in case of no access to UFS device yet */
ufshcd_release(hba); ufshcd_release(hba);
...@@ -6043,6 +6051,7 @@ static int ufshcd_resume(struct ufs_hba *hba, enum ufs_pm_op pm_op) ...@@ -6043,6 +6051,7 @@ static int ufshcd_resume(struct ufs_hba *hba, enum ufs_pm_op pm_op)
ufshcd_vreg_set_lpm(hba); ufshcd_vreg_set_lpm(hba);
disable_irq_and_vops_clks: disable_irq_and_vops_clks:
ufshcd_disable_irq(hba); ufshcd_disable_irq(hba);
ufshcd_suspend_clkscaling(hba);
ufshcd_setup_clocks(hba, false); ufshcd_setup_clocks(hba, false);
out: out:
hba->pm_op_in_progress = 0; hba->pm_op_in_progress = 0;
...@@ -6529,8 +6538,7 @@ int ufshcd_init(struct ufs_hba *hba, void __iomem *mmio_base, unsigned int irq) ...@@ -6529,8 +6538,7 @@ int ufshcd_init(struct ufs_hba *hba, void __iomem *mmio_base, unsigned int irq)
goto out_remove_scsi_host; goto out_remove_scsi_host;
} }
/* Suspend devfreq until the UFS device is detected */ /* Suspend devfreq until the UFS device is detected */
devfreq_suspend_device(hba->devfreq); ufshcd_suspend_clkscaling(hba);
hba->clk_scaling.window_start_t = 0;
} }
/* Hold auto suspend until async scan completes */ /* Hold auto suspend until async scan completes */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment