cpufreq-dt: Drop unnecessary check before cpufreq_cooling_unregister() invocation
authorMarkus Elfring <elfring@users.sourceforge.net>
Tue, 3 Feb 2015 18:21:21 +0000 (19:21 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 3 Feb 2015 22:28:37 +0000 (23:28 +0100)
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>
drivers/cpufreq/cpufreq-dt.c

index fde97d6e31d6d9749698aaf91bfae821f1a72f9f..bab67db54b7eb4fbed633ef0d65eece0637edcaa 100644 (file)
@@ -320,8 +320,7 @@ static int cpufreq_exit(struct cpufreq_policy *policy)
 {
        struct private_data *priv = policy->driver_data;
 
-       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);
        of_free_opp_table(priv->cpu_dev);
        clk_put(policy->clk);