WARN_ON() already contains an unlikely(), so it's not necessary to wrap it
into another.

Signed-off-by: Igor Stoppa <igor.sto...@huawei.com>
Cc: "Rafael J. Wysocki" <r...@rjwysocki.net>
Cc: Srivatsa S. Bhat <srivatsa.b...@linux.vnet.ibm.com>
---
 drivers/cpufreq/cpufreq.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
index f53fb41efb7b..7aa3dcad2175 100644
--- a/drivers/cpufreq/cpufreq.c
+++ b/drivers/cpufreq/cpufreq.c
@@ -403,7 +403,7 @@ EXPORT_SYMBOL_GPL(cpufreq_freq_transition_begin);
 void cpufreq_freq_transition_end(struct cpufreq_policy *policy,
                struct cpufreq_freqs *freqs, int transition_failed)
 {
-       if (unlikely(WARN_ON(!policy->transition_ongoing)))
+       if (WARN_ON(!policy->transition_ongoing))
                return;
 
        cpufreq_notify_post_transition(policy, freqs, transition_failed);
-- 
2.17.1

Reply via email to