s5p-mfc: Use WARN_ON(condition) directly
authorSachin Kamat <sachin.kamat@linaro.org>
Tue, 22 Jan 2013 04:00:06 +0000 (01:00 -0300)
committerChromeBot <chrome-bot@google.com>
Wed, 20 Mar 2013 09:05:13 +0000 (02:05 -0700)
<UPSTREAM MERGE NOT REQUIRED>

linux-media commit e82564475eab196b2e8a11572fff8e268329530e
Original commit message as follows:

  Use WARN_ON(condition) directly instead of wrapping around an if
  condition.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Change-Id: Iea1296a0452dd1b948bd1e6ba41b3e8149f579a3
Signed-off-by: John Sheu <sheu@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/45099

drivers/media/video/s5p-mfc/s5p_mfc.c

index 098fab4..a63afe5 100644 (file)
@@ -589,8 +589,7 @@ static void s5p_mfc_handle_stream_complete(struct s5p_mfc_ctx *ctx,
 
        clear_work_bit(ctx);
 
-       if (test_and_clear_bit(0, &dev->hw_lock) == 0)
-               WARN_ON(1);
+       WARN_ON(test_and_clear_bit(0, &dev->hw_lock) == 0);
 
        s5p_mfc_clock_off();
        wake_up(&ctx->queue);