From: Niklas Söderlund Date: Wed, 24 May 2017 00:15:25 +0000 (-0300) Subject: [media] rcar-vin: use rvin_reset_format() in S_DV_TIMINGS X-Git-Tag: v4.13-rc4~5^2~309 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b655741f2149296bb1b9efc1161d7125718108d7;p=platform%2Fkernel%2Flinux-exynos.git [media] rcar-vin: use rvin_reset_format() in S_DV_TIMINGS Use rvin_reset_format() in rvin_s_dv_timings() instead of just resetting a few fields. This fixes an issue where the field format was not properly set after S_DV_TIMINGS. Signed-off-by: Niklas Söderlund Signed-off-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/platform/rcar-vin/rcar-v4l2.c b/drivers/media/platform/rcar-vin/rcar-v4l2.c index 69bc4cf..7ca2759 100644 --- a/drivers/media/platform/rcar-vin/rcar-v4l2.c +++ b/drivers/media/platform/rcar-vin/rcar-v4l2.c @@ -573,12 +573,8 @@ static int rvin_s_dv_timings(struct file *file, void *priv_fh, if (ret) return ret; - vin->source.width = timings->bt.width; - vin->source.height = timings->bt.height; - vin->format.width = timings->bt.width; - vin->format.height = timings->bt.height; - - return 0; + /* Changing the timings will change the width/height */ + return rvin_reset_format(vin); } static int rvin_g_dv_timings(struct file *file, void *priv_fh,