PM / devfreq: exynos: dev_pm_opp_put_*() accepts NULL argument
authorViresh Kumar <viresh.kumar@linaro.org>
Fri, 6 Nov 2020 06:48:39 +0000 (12:18 +0530)
committerViresh Kumar <viresh.kumar@linaro.org>
Wed, 9 Dec 2020 05:51:12 +0000 (11:21 +0530)
The dev_pm_opp_put_*() APIs now accepts a NULL opp_table pointer and so
there is no need for us to carry the extra check. Drop them.

Acked-by: Chanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
drivers/devfreq/exynos-bus.c

index 1e684a4..143fd58 100644 (file)
@@ -158,10 +158,8 @@ static void exynos_bus_exit(struct device *dev)
 
        dev_pm_opp_of_remove_table(dev);
        clk_disable_unprepare(bus->clk);
-       if (bus->opp_table) {
-               dev_pm_opp_put_regulators(bus->opp_table);
-               bus->opp_table = NULL;
-       }
+       dev_pm_opp_put_regulators(bus->opp_table);
+       bus->opp_table = NULL;
 }
 
 static void exynos_bus_passive_exit(struct device *dev)
@@ -444,10 +442,8 @@ err:
        dev_pm_opp_of_remove_table(dev);
        clk_disable_unprepare(bus->clk);
 err_reg:
-       if (!passive) {
-               dev_pm_opp_put_regulators(bus->opp_table);
-               bus->opp_table = NULL;
-       }
+       dev_pm_opp_put_regulators(bus->opp_table);
+       bus->opp_table = NULL;
 
        return ret;
 }