diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2015-02-03 13:21:21 -0500 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2015-02-03 17:28:37 -0500 |
commit | 17ad13ba842bd5c197e20c17e107788aa0498ba9 (patch) | |
tree | 2617498a9cdf0c818713439744d86fbdf0412c97 | |
parent | f7b2706117bc4459cd748352cbe4ae69a41c8ed6 (diff) |
cpufreq-dt: Drop unnecessary check before cpufreq_cooling_unregister() invocation
The cpufreq_cooling_unregister() function tests whether its argument is NULL
and then returns immediately. Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
[ rjw: Subject ]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r-- | drivers/cpufreq/cpufreq-dt.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/cpufreq/cpufreq-dt.c b/drivers/cpufreq/cpufreq-dt.c index fde97d6e31d6..bab67db54b7e 100644 --- a/drivers/cpufreq/cpufreq-dt.c +++ b/drivers/cpufreq/cpufreq-dt.c | |||
@@ -320,8 +320,7 @@ static int cpufreq_exit(struct cpufreq_policy *policy) | |||
320 | { | 320 | { |
321 | struct private_data *priv = policy->driver_data; | 321 | struct private_data *priv = policy->driver_data; |
322 | 322 | ||
323 | if (priv->cdev) | 323 | cpufreq_cooling_unregister(priv->cdev); |
324 | cpufreq_cooling_unregister(priv->cdev); | ||
325 | dev_pm_opp_free_cpufreq_table(priv->cpu_dev, &policy->freq_table); | 324 | dev_pm_opp_free_cpufreq_table(priv->cpu_dev, &policy->freq_table); |
326 | of_free_opp_table(priv->cpu_dev); | 325 | of_free_opp_table(priv->cpu_dev); |
327 | clk_put(policy->clk); | 326 | clk_put(policy->clk); |