diff options
| author | Viresh Kumar <viresh.kumar@linaro.org> | 2014-12-09 23:15:33 -0500 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2014-12-10 16:18:34 -0500 |
| commit | 29df0ee1b14ab5cdc83c225258f42600825f45b2 (patch) | |
| tree | 403f3b100350e2b167ad3cbb1d827e949013b861 | |
| parent | 86453b473b1f68c238a6901b26158b4ca3b369bc (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.c | 9 |
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); | |||
| 597 | static int opp_set_availability(struct device *dev, unsigned long freq, | 597 | static 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); |
