media: s5p-mfc: Amend initial min, max values of HEVC hierarchical coding QP controls
authorSylwester Nawrocki <s.nawrocki@samsung.com>
Mon, 19 Mar 2018 14:29:58 +0000 (10:29 -0400)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Thu, 22 Mar 2018 11:21:29 +0000 (07:21 -0400)
Valid range for those controls is specified in documentation as [0, 51],
so initialize the controls to such range rather than [INT_MIN, INT_MAX].

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/platform/s5p-mfc/s5p_mfc_enc.c

index 810dabe..7382b41 100644 (file)
@@ -856,56 +856,56 @@ static struct mfc_control controls[] = {
        {
                .id = V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L0_QP,
                .type = V4L2_CTRL_TYPE_INTEGER,
-               .minimum = INT_MIN,
-               .maximum = INT_MAX,
+               .minimum = 0,
+               .maximum = 51,
                .step = 1,
                .default_value = 0,
        },
        {
                .id = V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L1_QP,
                .type = V4L2_CTRL_TYPE_INTEGER,
-               .minimum = INT_MIN,
-               .maximum = INT_MAX,
+               .minimum = 0,
+               .maximum = 51,
                .step = 1,
                .default_value = 0,
        },
        {
                .id = V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L2_QP,
                .type = V4L2_CTRL_TYPE_INTEGER,
-               .minimum = INT_MIN,
-               .maximum = INT_MAX,
+               .minimum = 0,
+               .maximum = 51,
                .step = 1,
                .default_value = 0,
        },
        {
                .id = V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L3_QP,
                .type = V4L2_CTRL_TYPE_INTEGER,
-               .minimum = INT_MIN,
-               .maximum = INT_MAX,
+               .minimum = 0,
+               .maximum = 51,
                .step = 1,
                .default_value = 0,
        },
        {
                .id = V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L4_QP,
                .type = V4L2_CTRL_TYPE_INTEGER,
-               .minimum = INT_MIN,
-               .maximum = INT_MAX,
+               .minimum = 0,
+               .maximum = 51,
                .step = 1,
                .default_value = 0,
        },
        {
                .id = V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L5_QP,
                .type = V4L2_CTRL_TYPE_INTEGER,
-               .minimum = INT_MIN,
-               .maximum = INT_MAX,
+               .minimum = 0,
+               .maximum = 51,
                .step = 1,
                .default_value = 0,
        },
        {
                .id = V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L6_QP,
                .type = V4L2_CTRL_TYPE_INTEGER,
-               .minimum = INT_MIN,
-               .maximum = INT_MAX,
+               .minimum = 0,
+               .maximum = 51,
                .step = 1,
                .default_value = 0,
        },