Commit 79bb4c7d authored by Alexander Stein's avatar Alexander Stein Committed by Hans Verkuil
Browse files

media: imx: imx7-media-csi: Remove incorrect interlacing support 



The driver doesn't currently support interlacing, but due to legacy
leftovers, it accepts values for the pixel format "field" field other
than V4L2_FIELD_NONE. Fix it by hardcoding V4L2_FIELD_NONE. Proper
interlacing support can be implemented later if desired.

Signed-off-by: default avatarAlexander Stein <alexander.stein@ew.tq-group.com>
Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
parent 6f482c47

drivers/media/platform/nxp/imx7-media-csi.c

+1 −14
Original line number Diff line number Diff line
@@ -1162,20 +1162,6 @@ __imx7_csi_video_try_fmt(struct v4l2_pix_format *pixfmt, 
		cc = imx7_csi_find_pixel_format(pixfmt->pixelformat);

	}



	/* Allow IDMAC interweave but enforce field order from source. */

	if (V4L2_FIELD_IS_INTERLACED(pixfmt->field)) {

		switch (pixfmt->field) {

		case V4L2_FIELD_SEQ_TB:

			pixfmt->field = V4L2_FIELD_INTERLACED_TB;

			break;

		case V4L2_FIELD_SEQ_BT:

			pixfmt->field = V4L2_FIELD_INTERLACED_BT;

			break;

		default:

			break;

		}

	}



	/*

	 * Round up width for minimum burst size.

	 *
@@ -1187,6 +1173,7 @@ __imx7_csi_video_try_fmt(struct v4l2_pix_format *pixfmt, 


	pixfmt->bytesperline = pixfmt->width * cc->bpp / 8;

	pixfmt->sizeimage = pixfmt->bytesperline * pixfmt->height;

	pixfmt->field = V4L2_FIELD_NONE;



	return cc;

}