From: Cho KyongHo <pullip.cho@samsung.com>
Date: Tue, 27 Dec 2016 11:29:11 +0000 (+0900)
Subject: [COMMON] media: smfc: compare the conditions clearly
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=a9d711fa7ef1f056bc468229cf7a072604c22b9c;p=GitHub%2FLineageOS%2Fandroid_kernel_motorola_exynos9610.git

[COMMON] media: smfc: compare the conditions clearly

The recent versions of the compilers(e.g. GCC 5.3) warn that a unary
operator may be used without a care. Therefore, we should wrap the
operator and its operands with parenthesis to show clear precedence.

Change-Id: Ibc7e758dd67e62182a120bc705a561586a521c70
Signed-off-by: Cho KyongHo <pullip.cho@samsung.com>
---

diff --git a/drivers/media/platform/exynos/smfc/smfc.h b/drivers/media/platform/exynos/smfc/smfc.h
index 9b3ff8fa1df3..0d84eef15641 100644
--- a/drivers/media/platform/exynos/smfc/smfc.h
+++ b/drivers/media/platform/exynos/smfc/smfc.h
@@ -216,7 +216,7 @@ static inline u32 smfc_config_ctxflag(struct smfc_ctx *ctx, u32 flag, bool set)
 
 static inline bool smfc_is_compressed_type(struct smfc_ctx *ctx, __u32 type)
 {
-	return !(ctx->flags & SMFC_CTX_COMPRESS) == V4L2_TYPE_IS_OUTPUT(type);
+	return (!(ctx->flags & SMFC_CTX_COMPRESS)) == V4L2_TYPE_IS_OUTPUT(type);
 }
 
 int smfc_init_controls(struct smfc_dev *smfc, struct v4l2_ctrl_handler *hdlr);