media: vimc: cap: fix step width/height in enum framesize
authorHelen Fornazier <helen.koike@collabora.com>
Wed, 6 Mar 2019 22:42:39 +0000 (17:42 -0500)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Tue, 19 Mar 2019 19:28:26 +0000 (15:28 -0400)
The type V4L2_FRMSIZE_TYPE_CONTINUOUS expects a step of 1.
This fixes v4l2-compliance test error:

        fail: v4l2-test-formats.cpp(184): invalid step_width/height for continuous framesize
test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: FAIL

Signed-off-by: Helen Koike <helen.koike@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/platform/vimc/vimc-capture.c

index 54fda5b29dd0c5eba4db9899d61e2513a37a29ef..290b08ebb1b64af1aefdf04c3e182197618dd31f 100644 (file)
@@ -187,8 +187,8 @@ static int vimc_cap_enum_framesizes(struct file *file, void *fh,
        fsize->stepwise.max_width = VIMC_FRAME_MAX_WIDTH;
        fsize->stepwise.min_height = VIMC_FRAME_MIN_HEIGHT;
        fsize->stepwise.max_height = VIMC_FRAME_MAX_HEIGHT;
-       fsize->stepwise.step_width = 2;
-       fsize->stepwise.step_height = 2;
+       fsize->stepwise.step_width = 1;
+       fsize->stepwise.step_height = 1;
 
        return 0;
 }