All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] media: rcar-vin: Fix image alignment for setting pre clipping
       [not found]         ` <87bmepxga1.wl%kuninori.morimoto.gx@renesas.com>
@ 2018-04-13  2:18           ` Kuninori Morimoto
  0 siblings, 0 replies; only message in thread
From: Kuninori Morimoto @ 2018-04-13  2:18 UTC (permalink / raw)
  To: "Niklas Söderlund", Mauro Carvalho Chehab
  Cc: linux-media, Niklas Soderlund, koji.matsuoka.xm, Magnus, Laurent,
	shimoda, shiiba, sakato, Kihara, Fukawa, hien.dang.eb,
	khiem.nguyen.xt, kouei.abe.cp, harunobu.kurokawa.dn,
	hiroyuki.yokoyama.vx


From: Koji Matsuoka <koji.matsuoka.xm@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.xm@renesas.com>
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Acked-by: Niklas Söderlund <niklas.soderlund+renesas@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

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2018-04-13  2:18 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <redmine.issue-121985.20170421084025@dm.renesas.com>
     [not found] ` <redmine.issue-121985.20170421084025.54bb05cdb01b9ffe@dm.renesas.com>
     [not found]   ` <87tw5iuoqa.wl%kuninori.morimoto.gx@renesas.com>
     [not found]     ` <20170421130432.GT28868@bigcity.dyn.berto.se>
     [not found]       ` <877f2awv7u.wl%kuninori.morimoto.gx@renesas.com>
     [not found]         ` <87bmepxga1.wl%kuninori.morimoto.gx@renesas.com>
2018-04-13  2:18           ` [PATCH] media: rcar-vin: Fix image alignment for setting pre clipping Kuninori Morimoto

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.