diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2015-08-12 06:29:39 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2015-08-28 10:05:35 -0400 |
commit | 50a3cb04a5f9cd5323a76db9ee409a7f3004259a (patch) | |
tree | 4805a6eaf9b145d0561d4dc811582c0296be5551 | |
parent | 68fa9f0ab1b61cfc7deee699da8b5b5cb12f7a58 (diff) |
PM / OPP: Drop unlikely before IS_ERR(_OR_NULL)
IS_ERR(_OR_NULL) already contain an 'unlikely' compiler flag and there
is no need to do that again from its callers. Drop it.
Acked-by: Pavel Machek <pavel@ucw.cz>
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 | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/base/power/opp.c b/drivers/base/power/opp.c index 803d8b7ced89..bb703b5ebaff 100644 --- a/drivers/base/power/opp.c +++ b/drivers/base/power/opp.c | |||
@@ -204,7 +204,7 @@ static struct device_opp *_find_device_opp(struct device *dev) | |||
204 | { | 204 | { |
205 | struct device_opp *dev_opp; | 205 | struct device_opp *dev_opp; |
206 | 206 | ||
207 | if (unlikely(IS_ERR_OR_NULL(dev))) { | 207 | if (IS_ERR_OR_NULL(dev)) { |
208 | pr_err("%s: Invalid parameters\n", __func__); | 208 | pr_err("%s: Invalid parameters\n", __func__); |
209 | return ERR_PTR(-EINVAL); | 209 | return ERR_PTR(-EINVAL); |
210 | } | 210 | } |
@@ -239,7 +239,7 @@ unsigned long dev_pm_opp_get_voltage(struct dev_pm_opp *opp) | |||
239 | opp_rcu_lockdep_assert(); | 239 | opp_rcu_lockdep_assert(); |
240 | 240 | ||
241 | tmp_opp = rcu_dereference(opp); | 241 | tmp_opp = rcu_dereference(opp); |
242 | if (unlikely(IS_ERR_OR_NULL(tmp_opp)) || !tmp_opp->available) | 242 | if (IS_ERR_OR_NULL(tmp_opp) || !tmp_opp->available) |
243 | pr_err("%s: Invalid parameters\n", __func__); | 243 | pr_err("%s: Invalid parameters\n", __func__); |
244 | else | 244 | else |
245 | v = tmp_opp->u_volt; | 245 | v = tmp_opp->u_volt; |
@@ -271,7 +271,7 @@ unsigned long dev_pm_opp_get_freq(struct dev_pm_opp *opp) | |||
271 | opp_rcu_lockdep_assert(); | 271 | opp_rcu_lockdep_assert(); |
272 | 272 | ||
273 | tmp_opp = rcu_dereference(opp); | 273 | tmp_opp = rcu_dereference(opp); |
274 | if (unlikely(IS_ERR_OR_NULL(tmp_opp)) || !tmp_opp->available) | 274 | if (IS_ERR_OR_NULL(tmp_opp) || !tmp_opp->available) |
275 | pr_err("%s: Invalid parameters\n", __func__); | 275 | pr_err("%s: Invalid parameters\n", __func__); |
276 | else | 276 | else |
277 | f = tmp_opp->rate; | 277 | f = tmp_opp->rate; |