[COMMON] media: smfc: compare the conditions clearly
authorCho KyongHo <pullip.cho@samsung.com>
Tue, 27 Dec 2016 11:29:11 +0000 (20:29 +0900)
committerSeungchul Kim <sc377.kim@samsung.com>
Mon, 28 May 2018 05:31:22 +0000 (14:31 +0900)
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>
drivers/media/platform/exynos/smfc/smfc.h

index 9b3ff8fa1df3b5d4f4fd7fd1c934b408205c04cd..0d84eef156415a844879d3ba7b971ec2229fcf2d 100644 (file)
@@ -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);