From: Jeonghee Kim <jhhhh.kim@samsung.com>
Date: Fri, 23 Mar 2018 09:10:06 +0000 (+0900)
Subject: media: mfc: fix build error(clang)
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=c0163105b995950cbaa2b24d3f7b7a4ba193eb72;p=GitHub%2FLineageOS%2Fandroid_kernel_motorola_exynos9610.git

media: mfc: fix build error(clang)

Change-Id: Ib036206d2f0374d80065e8f8552f9749868839fd
Signed-off-by: Jeonghee Kim <jhhhh.kim@samsung.com>
---

diff --git a/drivers/media/platform/exynos/mfc/s5p_mfc_dec_ops.c b/drivers/media/platform/exynos/mfc/s5p_mfc_dec_ops.c
index 0029d4956cae..b794812b05dd 100644
--- a/drivers/media/platform/exynos/mfc/s5p_mfc_dec_ops.c
+++ b/drivers/media/platform/exynos/mfc/s5p_mfc_dec_ops.c
@@ -901,7 +901,7 @@ static int s5p_mfc_dec_get_buf_update_val(struct s5p_mfc_ctx *ctx,
 	struct s5p_mfc_buf_ctrl *buf_ctrl;
 
 	list_for_each_entry(buf_ctrl, head, list) {
-		if ((buf_ctrl->id == id)) {
+		if (buf_ctrl->id == id) {
 			buf_ctrl->val = value;
 			mfc_debug(5, "++id: 0x%08x val: %d\n",
 					buf_ctrl->id, buf_ctrl->val);
diff --git a/drivers/media/platform/exynos/mfc/s5p_mfc_enc_internal.h b/drivers/media/platform/exynos/mfc/s5p_mfc_enc_internal.h
index 5ca05276426a..cecafe200bc3 100644
--- a/drivers/media/platform/exynos/mfc/s5p_mfc_enc_internal.h
+++ b/drivers/media/platform/exynos/mfc/s5p_mfc_enc_internal.h
@@ -1674,8 +1674,8 @@ static struct v4l2_queryctrl controls[] = {
 		.id = V4L2_CID_MPEG_MFC_GET_EXTRA_BUFFER_SIZE,
 		.type = V4L2_CTRL_TYPE_INTEGER,
 		.name = "Get extra buffer size",
-		.minimum = 0,
-		.maximum = (2 << 31) - 1,
+		.minimum = INT_MIN,
+		.maximum = INT_MAX,
 		.step = 1,
 		.default_value = 0,
 	},
diff --git a/drivers/media/platform/exynos/mfc/s5p_mfc_enc_ops.c b/drivers/media/platform/exynos/mfc/s5p_mfc_enc_ops.c
index 62a748abcb1e..ab548a400424 100644
--- a/drivers/media/platform/exynos/mfc/s5p_mfc_enc_ops.c
+++ b/drivers/media/platform/exynos/mfc/s5p_mfc_enc_ops.c
@@ -691,7 +691,7 @@ static int s5p_mfc_enc_get_buf_update_val(struct s5p_mfc_ctx *ctx,
 	struct s5p_mfc_buf_ctrl *buf_ctrl;
 
 	list_for_each_entry(buf_ctrl, head, list) {
-		if ((buf_ctrl->id == id)) {
+		if (buf_ctrl->id == id) {
 			buf_ctrl->val = value;
 			mfc_debug(5, "++id: 0x%08x val: %d\n",
 					buf_ctrl->id, buf_ctrl->val);
@@ -1132,7 +1132,7 @@ static void mfc_enc_set_buf_ctrls_exception(struct s5p_mfc_ctx *ctx,
 		MFC_WRITEL(value, S5P_FIMV_E_H264_NAL_CONTROL);
 	}
 
-	if ((buf_ctrl->id == V4L2_CID_MPEG_MFC51_VIDEO_I_PERIOD_CH)) {
+	if (buf_ctrl->id == V4L2_CID_MPEG_MFC51_VIDEO_I_PERIOD_CH) {
 		value = MFC_READL(S5P_FIMV_E_GOP_CONFIG2);
 		buf_ctrl->old_val |= (value << 16) & 0x3FFF0000;
 		value &= ~(0x3FFF);