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); |
