From be37fcf79c4b8facf33f75f1d67178daa089b52a Mon Sep 17 00:00:00 2001 From: Jeonghee Kim Date: Fri, 23 Mar 2018 18:10:06 +0900 Subject: [PATCH] media: mfc: fix build error(clang) Change-Id: Ib036206d2f0374d80065e8f8552f9749868839fd Signed-off-by: Jeonghee Kim --- drivers/media/platform/exynos/mfc/s5p_mfc_dec_ops.c | 2 +- drivers/media/platform/exynos/mfc/s5p_mfc_enc_internal.h | 4 ++-- drivers/media/platform/exynos/mfc/s5p_mfc_enc_ops.c | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) 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); -- 2.20.1