PM / OPP: reuse find_device_opp() instead of duplicating code
authorViresh Kumar <viresh.kumar@linaro.org>
Wed, 10 Dec 2014 04:15:33 +0000 (09:45 +0530)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 10 Dec 2014 21:18:34 +0000 (22:18 +0100)
Reuse find_device_opp() in opp_set_availability() instead of duplicating code.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/base/power/opp.c

index fa065d6e17312ba503e1bb42c32417e1f94f0948..525ffb202d77f67961983675f2cdf461937662b0 100644 (file)
@@ -597,7 +597,7 @@ EXPORT_SYMBOL_GPL(dev_pm_opp_remove);
 static int opp_set_availability(struct device *dev, unsigned long freq,
                bool availability_req)
 {
-       struct device_opp *tmp_dev_opp, *dev_opp = ERR_PTR(-ENODEV);
+       struct device_opp *dev_opp;
        struct dev_pm_opp *new_opp, *tmp_opp, *opp = ERR_PTR(-ENODEV);
        int r = 0;
 
@@ -611,12 +611,7 @@ static int opp_set_availability(struct device *dev, unsigned long freq,
        mutex_lock(&dev_opp_list_lock);
 
        /* Find the device_opp */
-       list_for_each_entry(tmp_dev_opp, &dev_opp_list, node) {
-               if (dev == tmp_dev_opp->dev) {
-                       dev_opp = tmp_dev_opp;
-                       break;
-               }
-       }
+       dev_opp = find_device_opp(dev);
        if (IS_ERR(dev_opp)) {
                r = PTR_ERR(dev_opp);
                dev_warn(dev, "%s: Device OPP not found (%d)\n", __func__, r);