Commit 603fbdc1 authored by Laurent Pinchart's avatar Laurent Pinchart Committed by Hans Verkuil
Browse files

media: rkisp1: resizer: Use v4l2_area instead of v4l2_rect to store size 



The rkisp1_rsz_config() and rkisp1_rsz_config_regs() functions use a
v4l2_rect to pass frame sizes, leaving the top and left members unused
and uninitialized. Use v4l2_area instead.

Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
parent 1a6ae627

drivers/media/platform/rockchip/rkisp1/rkisp1-resizer.c

+4 −3
Original line number Diff line number Diff line
@@ -208,8 +208,8 @@ static void rkisp1_rsz_disable(struct rkisp1_resizer *rsz, 
static void rkisp1_rsz_config_regs(struct rkisp1_resizer *rsz,

				   const struct v4l2_rect *sink_y,

				   const struct v4l2_rect *sink_c,

				   const struct v4l2_rect *src_y,

				   const struct v4l2_rect *src_c,

				   const struct v4l2_area *src_y,

				   const struct v4l2_area *src_c,

				   enum rkisp1_shadow_regs_when when)

{

	u32 ratio, rsz_ctrl = 0;
@@ -272,7 +272,8 @@ static void rkisp1_rsz_config(struct rkisp1_resizer *rsz, 
	const struct rkisp1_rsz_yuv_mbus_info *sink_yuv_info, *src_yuv_info;

	const struct v4l2_mbus_framefmt *src_fmt, *sink_fmt;

	const struct v4l2_rect *sink_crop;

	struct v4l2_rect sink_y, sink_c, src_y, src_c;

	struct v4l2_rect sink_y, sink_c;

	struct v4l2_area src_y, src_c;



	sink_fmt = v4l2_subdev_get_pad_format(&rsz->sd, sd_state,

					      RKISP1_RSZ_PAD_SINK);