From: Daniel Mack <daniel@zonque.org>
To: linux-media@vger.kernel.org
Cc: slongerbeam@gmail.com, mchehab@kernel.org,
Daniel Mack <daniel@zonque.org>
Subject: [PATCH 1/3] media: ov5640: initialize mode data structs by name
Date: Fri, 20 Apr 2018 11:44:17 +0200 [thread overview]
Message-ID: <20180420094419.11267-1-daniel@zonque.org> (raw)
This patch initializes the members of struct ov5640_mode_info by name for
better readability. This makes later additions to this struct easier.
No functional change intended.
Signed-off-by: Daniel Mack <daniel@zonque.org>
---
drivers/media/i2c/ov5640.c | 207 +++++++++++++++++++++++++++++++++------------
1 file changed, 152 insertions(+), 55 deletions(-)
diff --git a/drivers/media/i2c/ov5640.c b/drivers/media/i2c/ov5640.c
index 852026baa2e7..96f1564abdf5 100644
--- a/drivers/media/i2c/ov5640.c
+++ b/drivers/media/i2c/ov5640.c
@@ -728,67 +728,164 @@ static const struct reg_value ov5640_setting_15fps_QSXGA_2592_1944[] = {
/* power-on sensor init reg table */
static const struct ov5640_mode_info ov5640_mode_init_data = {
- 0, SUBSAMPLING, 640, 480, ov5640_init_setting_30fps_VGA,
- ARRAY_SIZE(ov5640_init_setting_30fps_VGA),
+ .id = 0,
+ .dn_mode = SUBSAMPLING,
+ .width = 640,
+ .height = 480,
+ .reg_data = ov5640_init_setting_30fps_VGA,
+ .reg_data_size = ARRAY_SIZE(ov5640_init_setting_30fps_VGA),
};
static const struct ov5640_mode_info
ov5640_mode_data[OV5640_NUM_FRAMERATES][OV5640_NUM_MODES] = {
+{
+ {
+ .id = OV5640_MODE_QCIF_176_144,
+ .dn_mode = SUBSAMPLING,
+ .width = 176,
+ .height = 144,
+ .reg_data = ov5640_setting_15fps_QCIF_176_144,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_15fps_QCIF_176_144),
+ },
+ {
+ .id = OV5640_MODE_QVGA_320_240,
+ .dn_mode = SUBSAMPLING,
+ .width = 320,
+ .height = 240,
+ .reg_data = ov5640_setting_15fps_QVGA_320_240,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_15fps_QVGA_320_240),
+ },
{
- {OV5640_MODE_QCIF_176_144, SUBSAMPLING, 176, 144,
- ov5640_setting_15fps_QCIF_176_144,
- ARRAY_SIZE(ov5640_setting_15fps_QCIF_176_144)},
- {OV5640_MODE_QVGA_320_240, SUBSAMPLING, 320, 240,
- ov5640_setting_15fps_QVGA_320_240,
- ARRAY_SIZE(ov5640_setting_15fps_QVGA_320_240)},
- {OV5640_MODE_VGA_640_480, SUBSAMPLING, 640, 480,
- ov5640_setting_15fps_VGA_640_480,
- ARRAY_SIZE(ov5640_setting_15fps_VGA_640_480)},
- {OV5640_MODE_NTSC_720_480, SUBSAMPLING, 720, 480,
- ov5640_setting_15fps_NTSC_720_480,
- ARRAY_SIZE(ov5640_setting_15fps_NTSC_720_480)},
- {OV5640_MODE_PAL_720_576, SUBSAMPLING, 720, 576,
- ov5640_setting_15fps_PAL_720_576,
- ARRAY_SIZE(ov5640_setting_15fps_PAL_720_576)},
- {OV5640_MODE_XGA_1024_768, SUBSAMPLING, 1024, 768,
- ov5640_setting_15fps_XGA_1024_768,
- ARRAY_SIZE(ov5640_setting_15fps_XGA_1024_768)},
- {OV5640_MODE_720P_1280_720, SUBSAMPLING, 1280, 720,
- ov5640_setting_15fps_720P_1280_720,
- ARRAY_SIZE(ov5640_setting_15fps_720P_1280_720)},
- {OV5640_MODE_1080P_1920_1080, SCALING, 1920, 1080,
- ov5640_setting_15fps_1080P_1920_1080,
- ARRAY_SIZE(ov5640_setting_15fps_1080P_1920_1080)},
- {OV5640_MODE_QSXGA_2592_1944, SCALING, 2592, 1944,
- ov5640_setting_15fps_QSXGA_2592_1944,
- ARRAY_SIZE(ov5640_setting_15fps_QSXGA_2592_1944)},
- }, {
- {OV5640_MODE_QCIF_176_144, SUBSAMPLING, 176, 144,
- ov5640_setting_30fps_QCIF_176_144,
- ARRAY_SIZE(ov5640_setting_30fps_QCIF_176_144)},
- {OV5640_MODE_QVGA_320_240, SUBSAMPLING, 320, 240,
- ov5640_setting_30fps_QVGA_320_240,
- ARRAY_SIZE(ov5640_setting_30fps_QVGA_320_240)},
- {OV5640_MODE_VGA_640_480, SUBSAMPLING, 640, 480,
- ov5640_setting_30fps_VGA_640_480,
- ARRAY_SIZE(ov5640_setting_30fps_VGA_640_480)},
- {OV5640_MODE_NTSC_720_480, SUBSAMPLING, 720, 480,
- ov5640_setting_30fps_NTSC_720_480,
- ARRAY_SIZE(ov5640_setting_30fps_NTSC_720_480)},
- {OV5640_MODE_PAL_720_576, SUBSAMPLING, 720, 576,
- ov5640_setting_30fps_PAL_720_576,
- ARRAY_SIZE(ov5640_setting_30fps_PAL_720_576)},
- {OV5640_MODE_XGA_1024_768, SUBSAMPLING, 1024, 768,
- ov5640_setting_30fps_XGA_1024_768,
- ARRAY_SIZE(ov5640_setting_30fps_XGA_1024_768)},
- {OV5640_MODE_720P_1280_720, SUBSAMPLING, 1280, 720,
- ov5640_setting_30fps_720P_1280_720,
- ARRAY_SIZE(ov5640_setting_30fps_720P_1280_720)},
- {OV5640_MODE_1080P_1920_1080, SCALING, 1920, 1080,
- ov5640_setting_30fps_1080P_1920_1080,
- ARRAY_SIZE(ov5640_setting_30fps_1080P_1920_1080)},
- {OV5640_MODE_QSXGA_2592_1944, -1, 0, 0, NULL, 0},
+ .id = OV5640_MODE_VGA_640_480,
+ .dn_mode = SUBSAMPLING,
+ .width = 640,
+ .height = 480,
+ .reg_data = ov5640_setting_15fps_VGA_640_480,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_15fps_VGA_640_480)
},
+ {
+ .id = OV5640_MODE_NTSC_720_480,
+ .dn_mode = SUBSAMPLING,
+ .width = 720,
+ .height = 480,
+ .reg_data = ov5640_setting_15fps_NTSC_720_480,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_15fps_NTSC_720_480),
+ },
+ {
+ .id = OV5640_MODE_PAL_720_576,
+ .dn_mode = SUBSAMPLING,
+ .width = 720,
+ .height = 576,
+ .reg_data = ov5640_setting_15fps_PAL_720_576,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_15fps_PAL_720_576),
+ },
+ {
+ .id = OV5640_MODE_XGA_1024_768,
+ .dn_mode = SUBSAMPLING,
+ .width = 1024,
+ .height = 768,
+ .reg_data = ov5640_setting_15fps_XGA_1024_768,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_15fps_XGA_1024_768),
+ },
+ {
+ .id = OV5640_MODE_720P_1280_720,
+ .dn_mode = SUBSAMPLING,
+ .width = 1280,
+ .height = 720,
+ .reg_data = ov5640_setting_15fps_720P_1280_720,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_15fps_720P_1280_720),
+ },
+ {
+ .id = OV5640_MODE_1080P_1920_1080,
+ .dn_mode = SCALING,
+ .width = 1920,
+ .height = 1080,
+ .reg_data = ov5640_setting_15fps_1080P_1920_1080,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_15fps_1080P_1920_1080),
+ },
+ {
+ .id = OV5640_MODE_QSXGA_2592_1944,
+ .dn_mode = SCALING,
+ .width = 2592,
+ .height = 1944,
+ .reg_data = ov5640_setting_15fps_QSXGA_2592_1944,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_15fps_QSXGA_2592_1944),
+ },
+},
+{
+ {
+ .id = OV5640_MODE_QCIF_176_144,
+ .dn_mode = SUBSAMPLING,
+ .width = 176,
+ .height = 144,
+ .reg_data = ov5640_setting_30fps_QCIF_176_144,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_30fps_QCIF_176_144),
+ },
+ {
+ .id = OV5640_MODE_QVGA_320_240,
+ .dn_mode = SUBSAMPLING,
+ .width = 320,
+ .height = 240,
+ .reg_data = ov5640_setting_30fps_QVGA_320_240,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_30fps_QVGA_320_240),
+ },
+ {
+ .id = OV5640_MODE_VGA_640_480,
+ .dn_mode = SUBSAMPLING,
+ .width = 640,
+ .height = 480,
+ .reg_data = ov5640_setting_30fps_VGA_640_480,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_30fps_VGA_640_480),
+ },
+ {
+ .id = OV5640_MODE_NTSC_720_480,
+ .dn_mode = SUBSAMPLING,
+ .width = 720,
+ .height = 480,
+ .reg_data = ov5640_setting_30fps_NTSC_720_480,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_30fps_NTSC_720_480),
+ },
+ {
+ .id = OV5640_MODE_PAL_720_576,
+ .dn_mode = SUBSAMPLING,
+ .width = 720,
+ .height = 576,
+ .reg_data = ov5640_setting_30fps_PAL_720_576,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_30fps_PAL_720_576),
+ },
+ {
+ .id = OV5640_MODE_XGA_1024_768,
+ .dn_mode = SUBSAMPLING,
+ .width = 1024,
+ .height = 768,
+ .reg_data = ov5640_setting_30fps_XGA_1024_768,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_30fps_XGA_1024_768),
+ },
+ {
+ .id = OV5640_MODE_720P_1280_720,
+ .dn_mode = SUBSAMPLING,
+ .width = 1280,
+ .height = 720,
+ .reg_data = ov5640_setting_30fps_720P_1280_720,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_30fps_720P_1280_720),
+ },
+ {
+ .id = OV5640_MODE_1080P_1920_1080,
+ .dn_mode = SCALING,
+ .width = 1920,
+ .height = 1080,
+ .reg_data = ov5640_setting_30fps_1080P_1920_1080,
+ .reg_data_size = ARRAY_SIZE(ov5640_setting_30fps_1080P_1920_1080),
+ },
+ {
+ .id = OV5640_MODE_QSXGA_2592_1944,
+ .dn_mode = -1,
+ .width = 0,
+ .height = 0,
+ .reg_data = NULL,
+ .reg_data_size = 0,
+ }
+}
};
static int ov5640_init_slave_id(struct ov5640_dev *sensor)
--
2.14.3
next reply other threads:[~2018-04-20 9:49 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-20 9:44 Daniel Mack [this message]
2018-04-20 9:44 ` [PATCH 2/3] media: ov5640: add PIXEL_RATE and LINK_FREQ controls Daniel Mack
2018-04-20 14:15 ` Maxime Ripard
2018-04-20 14:29 ` Daniel Mack
2018-04-20 19:00 ` Maxime Ripard
2018-04-24 10:22 ` Sakari Ailus
2018-04-24 12:03 ` Daniel Mack
2018-04-26 8:31 ` Sakari Ailus
2018-04-20 9:44 ` [PATCH 3/3] media: ov5640: add support for xclk frequency control Daniel Mack
2018-04-20 14:07 ` Maxime Ripard
2018-04-24 10:24 ` Sakari Ailus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180420094419.11267-1-daniel@zonque.org \
--to=daniel@zonque.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=slongerbeam@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox