Skip to content

Commit

Permalink
cpufreq: qcom-cpufreq-hw: Clear dcvs interrupts
Browse files Browse the repository at this point in the history
[ Upstream commit e4e6448 ]

It's noted that dcvs interrupts are not self-clearing, thus an interrupt
handler runs constantly, which leads to a severe regression in runtime.
To fix the problem an explicit write to clear interrupt register is
required, note that on OSM platforms the register may not be present.

Fixes: 275157b ("cpufreq: qcom-cpufreq-hw: Add dcvs interrupt support")
Signed-off-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
  • Loading branch information
Vladimir Zapolskiy authored and gregkh committed May 9, 2022
1 parent 7356c1f commit abff341
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions drivers/cpufreq/qcom-cpufreq-hw.c
Expand Up @@ -24,13 +24,16 @@
#define CLK_HW_DIV 2
#define LUT_TURBO_IND 1

#define GT_IRQ_STATUS BIT(2)

#define HZ_PER_KHZ 1000

struct qcom_cpufreq_soc_data {
u32 reg_enable;
u32 reg_domain_state;
u32 reg_freq_lut;
u32 reg_volt_lut;
u32 reg_intr_clr;
u32 reg_current_vote;
u32 reg_perf_state;
u8 lut_row_size;
Expand Down Expand Up @@ -349,6 +352,10 @@ static irqreturn_t qcom_lmh_dcvs_handle_irq(int irq, void *data)
disable_irq_nosync(c_data->throttle_irq);
schedule_delayed_work(&c_data->throttle_work, 0);

if (c_data->soc_data->reg_intr_clr)
writel_relaxed(GT_IRQ_STATUS,
c_data->base + c_data->soc_data->reg_intr_clr);

return IRQ_HANDLED;
}

Expand All @@ -366,6 +373,7 @@ static const struct qcom_cpufreq_soc_data epss_soc_data = {
.reg_domain_state = 0x20,
.reg_freq_lut = 0x100,
.reg_volt_lut = 0x200,
.reg_intr_clr = 0x308,
.reg_perf_state = 0x320,
.lut_row_size = 4,
};
Expand Down

0 comments on commit abff341

Please sign in to comment.