When precedence rules are applied, the condition always evaluates
to be false which was not the intention. Adding the missing braces
for correct evaluation of the expression and subsequent functionality.

Signed-off-by: Sachin Kamat <sachin.ka...@linaro.org>
---
 drivers/media/platform/s5p-fimc/fimc-lite.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/media/platform/s5p-fimc/fimc-lite.c 
b/drivers/media/platform/s5p-fimc/fimc-lite.c
index 9289008..20e5e24 100644
--- a/drivers/media/platform/s5p-fimc/fimc-lite.c
+++ b/drivers/media/platform/s5p-fimc/fimc-lite.c
@@ -825,7 +825,7 @@ static int fimc_lite_reqbufs(struct file *file, void *priv,
 
        reqbufs->count = max_t(u32, FLITE_REQ_BUFS_MIN, reqbufs->count);
        ret = vb2_reqbufs(&fimc->vb_queue, reqbufs);
-       if (!ret < 0)
+       if (!(ret < 0))
                fimc->reqbufs_count = reqbufs->count;
 
        return ret;
-- 
1.7.4.1

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to