From: Koji Matsuoka <koji.matsuoka...@renesas.com>

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...@renesas.com>
Signed-off-by: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Acked-by: Niklas Söderlund <niklas.soderlund+rene...@ragnatech.se>
---
 drivers/media/platform/rcar-vin/rcar-v4l2.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/platform/rcar-vin/rcar-v4l2.c 
b/drivers/media/platform/rcar-vin/rcar-v4l2.c
index 22a6ecc..71ae65b 100644
--- a/drivers/media/platform/rcar-vin/rcar-v4l2.c
+++ b/drivers/media/platform/rcar-vin/rcar-v4l2.c
@@ -409,8 +409,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);
-- 
1.9.1

Reply via email to