Commit 2f0f609f authored by Viresh Kumar's avatar Viresh Kumar Committed by Rafael J. Wysocki

cpufreq-dt: free OPP table created during ->init()

OPP layer now supports freeing of OPPs and we should free them once they aren't
useful anymore.
Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Tested-by: default avatarStefan Wahren <stefan.wahren@i2se.com>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 2af3411f
...@@ -214,7 +214,7 @@ static int cpufreq_init(struct cpufreq_policy *policy) ...@@ -214,7 +214,7 @@ static int cpufreq_init(struct cpufreq_policy *policy)
priv = kzalloc(sizeof(*priv), GFP_KERNEL); priv = kzalloc(sizeof(*priv), GFP_KERNEL);
if (!priv) { if (!priv) {
ret = -ENOMEM; ret = -ENOMEM;
goto out_put_node; goto out_free_opp;
} }
of_property_read_u32(np, "voltage-tolerance", &priv->voltage_tolerance); of_property_read_u32(np, "voltage-tolerance", &priv->voltage_tolerance);
...@@ -294,7 +294,8 @@ static int cpufreq_init(struct cpufreq_policy *policy) ...@@ -294,7 +294,8 @@ static int cpufreq_init(struct cpufreq_policy *policy)
dev_pm_opp_free_cpufreq_table(cpu_dev, &freq_table); dev_pm_opp_free_cpufreq_table(cpu_dev, &freq_table);
out_free_priv: out_free_priv:
kfree(priv); kfree(priv);
out_put_node: out_free_opp:
of_free_opp_table(cpu_dev);
of_node_put(np); of_node_put(np);
out_put_reg_clk: out_put_reg_clk:
clk_put(cpu_clk); clk_put(cpu_clk);
...@@ -311,6 +312,7 @@ static int cpufreq_exit(struct cpufreq_policy *policy) ...@@ -311,6 +312,7 @@ static int cpufreq_exit(struct cpufreq_policy *policy)
if (priv->cdev) if (priv->cdev)
cpufreq_cooling_unregister(priv->cdev); cpufreq_cooling_unregister(priv->cdev);
dev_pm_opp_free_cpufreq_table(priv->cpu_dev, &policy->freq_table); dev_pm_opp_free_cpufreq_table(priv->cpu_dev, &policy->freq_table);
of_free_opp_table(priv->cpu_dev);
clk_put(policy->clk); clk_put(policy->clk);
if (!IS_ERR(priv->cpu_reg)) if (!IS_ERR(priv->cpu_reg))
regulator_put(priv->cpu_reg); regulator_put(priv->cpu_reg);
......
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