aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorViresh Kumar <viresh.kumar@linaro.org>2014-12-09 23:15:33 -0500
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2014-12-10 16:18:34 -0500
commit29df0ee1b14ab5cdc83c225258f42600825f45b2 (patch)
tree403f3b100350e2b167ad3cbb1d827e949013b861
parent86453b473b1f68c238a6901b26158b4ca3b369bc (diff)
PM / OPP: reuse find_device_opp() instead of duplicating code
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>
-rw-r--r--drivers/base/power/opp.c9
1 files changed, 2 insertions, 7 deletions
diff --git a/drivers/base/power/opp.c b/drivers/base/power/opp.c
index fa065d6e1731..525ffb202d77 100644
--- a/drivers/base/power/opp.c
+++ b/drivers/base/power/opp.c
@@ -597,7 +597,7 @@ EXPORT_SYMBOL_GPL(dev_pm_opp_remove);
597static int opp_set_availability(struct device *dev, unsigned long freq, 597static int opp_set_availability(struct device *dev, unsigned long freq,
598 bool availability_req) 598 bool availability_req)
599{ 599{
600 struct device_opp *tmp_dev_opp, *dev_opp = ERR_PTR(-ENODEV); 600 struct device_opp *dev_opp;
601 struct dev_pm_opp *new_opp, *tmp_opp, *opp = ERR_PTR(-ENODEV); 601 struct dev_pm_opp *new_opp, *tmp_opp, *opp = ERR_PTR(-ENODEV);
602 int r = 0; 602 int r = 0;
603 603
@@ -611,12 +611,7 @@ static int opp_set_availability(struct device *dev, unsigned long freq,
611 mutex_lock(&dev_opp_list_lock); 611 mutex_lock(&dev_opp_list_lock);
612 612
613 /* Find the device_opp */ 613 /* Find the device_opp */
614 list_for_each_entry(tmp_dev_opp, &dev_opp_list, node) { 614 dev_opp = find_device_opp(dev);
615 if (dev == tmp_dev_opp->dev) {
616 dev_opp = tmp_dev_opp;
617 break;
618 }
619 }
620 if (IS_ERR(dev_opp)) { 615 if (IS_ERR(dev_opp)) {
621 r = PTR_ERR(dev_opp); 616 r = PTR_ERR(dev_opp);
622 dev_warn(dev, "%s: Device OPP not found (%d)\n", __func__, r); 617 dev_warn(dev, "%s: Device OPP not found (%d)\n", __func__, r);