linux-media.vger.kernel.org archive mirror
 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).