media: rcar-vin: Fix image alignment for setting pre clipping

In Video Pixel/Line Pre-Clip Register, the setting value can be
set in 1 line unit, but it can only be specified as a multiple of
4 by v4l_bound_align_image function().
So correct that it can be specified in 1 line unit with this patch.

Signed-off-by: Koji Matsuoka <koji.matsuoka.xm@renesas.com>
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Acked-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Signed-off-by: Hans Verkuil <hansverk@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>

diff --git a/drivers/media/platform/rcar-vin/rcar-v4l2.c b/drivers/media/platform/rcar-vin/rcar-v4l2.c
index b479b882da12f62d1d14ad5bed14897baafb91fd..a3c2daca1bf32df8097776c7a4ca5d5dab0e9cea 100644 (file)
--- a/drivers/media/platform/rcar-vin/rcar-v4l2.c
+++ b/drivers/media/platform/rcar-vin/rcar-v4l2.c
@@ -405,8 +405,8 @@ static int rvin_s_selection(struct file *file, void *fh,
                max_rect.height = vin->source.height;
                v4l2_rect_map_inside(&r, &max_rect);
 
-               v4l_bound_align_image(&r.width, 2, vin->source.width, 1,
-                                     &r.height, 4, vin->source.height, 2, 0);
+               v4l_bound_align_image(&r.width, 6, vin->source.width, 0,
+                                     &r.height, 2, vin->source.height, 0, 0);
 
                r.top  = clamp_t(s32, r.top, 0, vin->source.height - r.height);
                r.left = clamp_t(s32, r.left, 0, vin->source.width - r.width);