Commit 37fa1d2a authored by Tomi Valkeinen's avatar Tomi Valkeinen Committed by Mauro Carvalho Chehab
Browse files

media: ti-vpe: cal: set field always to V4L2_FIELD_NONE 



cal_camerarx_sd_set_fmt() accepts any value for the format field, but
there should be no reason to have any other value accepted than
V4L2_FIELD_NONE. So set the field always to V4L2_FIELD_NONE.

Signed-off-by: default avatarTomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+huawei@kernel.org>
parent a6c9aad9

drivers/media/platform/ti-vpe/cal-camerarx.c

+2 −4
Original line number Diff line number Diff line
@@ -705,10 +705,7 @@ static int cal_camerarx_sd_set_fmt(struct v4l2_subdev *sd, 
	if (!fmtinfo)

		fmtinfo = &cal_formats[0];



	/*

	 * Clamp the size, update the code. The field and colorspace are

	 * accepted as-is.

	 */

	/* Clamp the size, update the code. The colorspace is accepted as-is. */

	bpp = ALIGN(fmtinfo->bpp, 8);



	format->format.width = clamp_t(unsigned int, format->format.width,
@@ -718,6 +715,7 @@ static int cal_camerarx_sd_set_fmt(struct v4l2_subdev *sd, 
					CAL_MIN_HEIGHT_LINES,

					CAL_MAX_HEIGHT_LINES);

	format->format.code = fmtinfo->code;

	format->format.field = V4L2_FIELD_NONE;



	/* Store the format and propagate it to the source pad. */

	fmt = cal_camerarx_get_pad_format(phy, sd_state,