Commit f5cc14e4 authored by Ricardo Ribalda's avatar Ricardo Ribalda Committed by Mauro Carvalho Chehab
Browse files

media: sun4i-csi: Do not zero reserved fields 



Core code already clears reserved fields of struct
v4l2_pix_format_mplane, check commit 4e1e0eb0 ("media: v4l2-ioctl: Zero
v4l2_plane_pix_format reserved fields").

Cc: Maxime Ripard <mripard@kernel.org>
Cc: Chen-Yu Tsai <wens@csie.org>
Reviewed-by: default avatarKieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Signed-off-by: default avatarRicardo Ribalda <ribalda@chromium.org>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+huawei@kernel.org>
parent ed2fb2f5

drivers/media/platform/sunxi/sun4i-csi/sun4i_v4l2.c

+0 −4
Original line number Diff line number Diff line
@@ -113,8 +113,6 @@ static void _sun4i_csi_try_fmt(struct sun4i_csi *csi, 
	pix->num_planes = _fmt->num_planes;

	pix->pixelformat = _fmt->fourcc;



	memset(pix->reserved, 0, sizeof(pix->reserved));



	/* Align the width and height on the subsampling */

	width = ALIGN(pix->width, _fmt->hsub);

	height = ALIGN(pix->height, _fmt->vsub);
@@ -131,8 +129,6 @@ static void _sun4i_csi_try_fmt(struct sun4i_csi *csi, 
		bpl = pix->width / hsub * _fmt->bpp[i] / 8;

		pix->plane_fmt[i].bytesperline = bpl;

		pix->plane_fmt[i].sizeimage = bpl * pix->height / vsub;

		memset(pix->plane_fmt[i].reserved, 0,

		       sizeof(pix->plane_fmt[i].reserved));

	}

}