From: Dave Stevenson Date: Mon, 5 Sep 2022 10:50:30 +0000 (+0100) Subject: Revert "media: i2c: imx290: Explicitly set v&h blank on mode change" X-Git-Tag: accepted/tizen/unified/20230118.172025~430 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b181de45cfde3853537cef2e5e45f00d0674350a;p=platform%2Fkernel%2Flinux-rpi.git Revert "media: i2c: imx290: Explicitly set v&h blank on mode change" This reverts commit 090d29f5854aa7b6511b0ae8a926929130fd3978. Whilst the behaviour of not resetting the frame rate to the "correct"/default value on mode change is useful, it is not correct under the V4L2 API. Revert the patch so that blanking values persist. Signed-off-by: Dave Stevenson --- diff --git a/drivers/media/i2c/imx290.c b/drivers/media/i2c/imx290.c index 2c0c0b9..da7d0d9 100644 --- a/drivers/media/i2c/imx290.c +++ b/drivers/media/i2c/imx290.c @@ -916,23 +916,17 @@ static int imx290_set_fmt(struct v4l2_subdev *sd, __v4l2_ctrl_s_ctrl_int64(imx290->pixel_rate, imx290_calc_pixel_rate(imx290)); - if (imx290->hblank) { + if (imx290->hblank) __v4l2_ctrl_modify_range(imx290->hblank, imx290->hmax_min - mode->width, IMX290_HMAX_MAX - mode->width, 1, mode->hmax - mode->width); - __v4l2_ctrl_s_ctrl(imx290->hblank, - mode->hmax - mode->width); - } - if (imx290->vblank) { + if (imx290->vblank) __v4l2_ctrl_modify_range(imx290->vblank, mode->vmax - mode->height, IMX290_VMAX_MAX - mode->height, 1, mode->vmax - mode->height); - __v4l2_ctrl_s_ctrl(imx290->vblank, - mode->vmax - mode->height); - } if (imx290->exposure) __v4l2_ctrl_modify_range(imx290->exposure, IMX290_EXPOSURE_MIN,