Commit 5ebbdd7a authored by Laurent Pinchart's avatar Laurent Pinchart Committed by Hans Verkuil
Browse files

media: i2c: imx219: Drop IMX219_VTS_* macros 



The IMX219_VTS_* macros define default VTS values for the modes
supported by the driver. They are used in a single place, and hinder
readability compared to using the value directly as a decimal number.
Drop them.

Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: default avatarDave Stevenson <dave.stevenson@raspberrypi.com>
Reviewed-by: default avatarJacopo Mondi <jacopo.mondi@ideasonboard.com>
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
parent e3e5d172

drivers/media/i2c/imx219.c

+4 −8
Original line number Diff line number Diff line
@@ -71,10 +71,6 @@ 


/* V_TIMING internal */

#define IMX219_REG_VTS			CCI_REG16(0x0160)

#define IMX219_VTS_15FPS		0x0dc6

#define IMX219_VTS_30FPS_1080P		0x06e3

#define IMX219_VTS_30FPS_BINNED		0x06e3

#define IMX219_VTS_30FPS_640x480	0x06e3

#define IMX219_VTS_MAX			0xffff



#define IMX219_VBLANK_MIN		4
@@ -301,7 +297,7 @@ static const struct imx219_mode supported_modes[] = { 
			.width = 3280,

			.height = 2464

		},

		.vts_def = IMX219_VTS_15FPS,

		.vts_def = 3526,

	},

	{

		/* 1080P 30fps cropped */
@@ -313,7 +309,7 @@ static const struct imx219_mode supported_modes[] = { 
			.width = 1920,

			.height = 1080

		},

		.vts_def = IMX219_VTS_30FPS_1080P,

		.vts_def = 1763,

	},

	{

		/* 2x2 binned 30fps mode */
@@ -325,7 +321,7 @@ static const struct imx219_mode supported_modes[] = { 
			.width = 3280,

			.height = 2464

		},

		.vts_def = IMX219_VTS_30FPS_BINNED,

		.vts_def = 1763,

	},

	{

		/* 640x480 30fps mode */
@@ -337,7 +333,7 @@ static const struct imx219_mode supported_modes[] = { 
			.width = 1280,

			.height = 960

		},

		.vts_def = IMX219_VTS_30FPS_640x480,

		.vts_def = 1763,

	},

};