* [PATCH 0/2] media: ov5645: Implement frame descriptor reporting and convert to CCI register access helpers @ 2026-03-26 14:21 Prabhakar 2026-03-26 14:21 ` [PATCH 1/2] media: i2c: ov5645: Report streams using frame descriptors Prabhakar 2026-03-26 14:21 ` [PATCH 2/2] media: i2c: ov5645: Convert to CCI register access helpers Prabhakar 0 siblings, 2 replies; 4+ messages in thread From: Prabhakar @ 2026-03-26 14:21 UTC (permalink / raw) To: Mauro Carvalho Chehab, Sakari Ailus, Hans Verkuil, Hans de Goede, Laurent Pinchart, Vladimir Zapolskiy, Mehdi Djait, Xiaolei Wang, Benjamin Mugnier, Sylvain Petinot, Hardevsinh Palaniya Cc: linux-media, linux-kernel, linux-renesas-soc, Prabhakar, Biju Das, Fabrizio Castro, Lad Prabhakar From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Hi all, This series implements the .get_frame_desc() subdev operation for the OV5645 camera sensor driver to report information about streams to the connected CSI-2 receiver. This is required to let the CSI-2 receiver driver know about virtual channels and data types for each stream. The series also converts the OV5645 driver to use CCI register access helpers, which simplifies the code and improves error handling. Note - patch 1/2 was origally part of series [0], patches 1-7 from series [0] have been already merged. - patches apply on top of media/next commit-id 0e2c4117c351 [0] https://lore.kernel.org/all/20241018153230.235647-1-prabhakar.mahadev-lad.rj@bp.renesas.com/ Cheers, Prabhakar Lad Prabhakar (2): media: i2c: ov5645: Report streams using frame descriptors media: i2c: ov5645: Convert to CCI register access helpers drivers/media/i2c/Kconfig | 1 + drivers/media/i2c/ov5645.c | 928 ++++++++++++++++++------------------- 2 files changed, 456 insertions(+), 473 deletions(-) -- 2.53.0 ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/2] media: i2c: ov5645: Report streams using frame descriptors 2026-03-26 14:21 [PATCH 0/2] media: ov5645: Implement frame descriptor reporting and convert to CCI register access helpers Prabhakar @ 2026-03-26 14:21 ` Prabhakar 2026-03-26 14:21 ` [PATCH 2/2] media: i2c: ov5645: Convert to CCI register access helpers Prabhakar 1 sibling, 0 replies; 4+ messages in thread From: Prabhakar @ 2026-03-26 14:21 UTC (permalink / raw) To: Mauro Carvalho Chehab, Sakari Ailus, Hans Verkuil, Hans de Goede, Laurent Pinchart, Vladimir Zapolskiy, Mehdi Djait, Xiaolei Wang, Benjamin Mugnier, Sylvain Petinot, Hardevsinh Palaniya Cc: linux-media, linux-kernel, linux-renesas-soc, Prabhakar, Biju Das, Fabrizio Castro, Lad Prabhakar, Laurent Pinchart From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Implement the .get_frame_desc() subdev operation to report information about streams to the connected CSI-2 receiver. This is required to let the CSI-2 receiver driver know about virtual channels and data types for each stream. Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> --- Changes since v3 [0], - Added a macro for the source pad index. - Updated ov5645_init_state() to use the new macro. [0] https://lore.kernel.org/all/20241018153230.235647-9-prabhakar.mahadev-lad.rj@bp.renesas.com/ Hi Laurent, Ive restored your RB tag with the above changes. Please let me know if you have any further comments. Cheers, Prabhakar --- drivers/media/i2c/ov5645.c | 29 ++++++++++++++++++++++++++++- 1 file changed, 28 insertions(+), 1 deletion(-) diff --git a/drivers/media/i2c/ov5645.c b/drivers/media/i2c/ov5645.c index b10d408034a1..df9001fce44d 100644 --- a/drivers/media/i2c/ov5645.c +++ b/drivers/media/i2c/ov5645.c @@ -28,6 +28,7 @@ #include <linux/regulator/consumer.h> #include <linux/slab.h> #include <linux/types.h> +#include <media/mipi-csi2.h> #include <media/v4l2-ctrls.h> #include <media/v4l2-fwnode.h> #include <media/v4l2-subdev.h> @@ -68,6 +69,8 @@ static const char * const ov5645_supply_name[] = { #define OV5645_NUM_SUPPLIES ARRAY_SIZE(ov5645_supply_name) +#define OV5645_PAD_SOURCE 0 + struct reg_value { u16 reg; u8 val; @@ -817,6 +820,29 @@ static const struct v4l2_ctrl_ops ov5645_ctrl_ops = { .s_ctrl = ov5645_s_ctrl, }; +static int ov5645_get_frame_desc(struct v4l2_subdev *sd, unsigned int pad, + struct v4l2_mbus_frame_desc *fd) +{ + struct v4l2_subdev_state *state; + u32 code; + + state = v4l2_subdev_lock_and_get_active_state(sd); + code = v4l2_subdev_state_get_format(state, OV5645_PAD_SOURCE, 0)->code; + v4l2_subdev_unlock_state(state); + + fd->type = V4L2_MBUS_FRAME_DESC_TYPE_CSI2; + fd->num_entries = 1; + + memset(fd->entry, 0, sizeof(fd->entry)); + + fd->entry[0].pixelcode = code; + fd->entry[0].stream = 0; + fd->entry[0].bus.csi2.vc = 0; + fd->entry[0].bus.csi2.dt = MIPI_CSI2_DT_YUV422_8B; + + return 0; +} + static int ov5645_enum_mbus_code(struct v4l2_subdev *sd, struct v4l2_subdev_state *sd_state, struct v4l2_subdev_mbus_code_enum *code) @@ -897,7 +923,7 @@ static int ov5645_init_state(struct v4l2_subdev *subdev, { struct v4l2_subdev_format fmt = { .which = V4L2_SUBDEV_FORMAT_TRY, - .pad = 0, + .pad = OV5645_PAD_SOURCE, .format = { .code = MEDIA_BUS_FMT_UYVY8_1X16, .width = ov5645_mode_info_data[1].width, @@ -988,6 +1014,7 @@ static const struct v4l2_subdev_video_ops ov5645_video_ops = { }; static const struct v4l2_subdev_pad_ops ov5645_subdev_pad_ops = { + .get_frame_desc = ov5645_get_frame_desc, .enum_mbus_code = ov5645_enum_mbus_code, .enum_frame_size = ov5645_enum_frame_size, .get_fmt = v4l2_subdev_get_fmt, -- 2.53.0 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 2/2] media: i2c: ov5645: Convert to CCI register access helpers 2026-03-26 14:21 [PATCH 0/2] media: ov5645: Implement frame descriptor reporting and convert to CCI register access helpers Prabhakar 2026-03-26 14:21 ` [PATCH 1/2] media: i2c: ov5645: Report streams using frame descriptors Prabhakar @ 2026-03-26 14:21 ` Prabhakar 2026-03-26 14:59 ` xiaolei wang 1 sibling, 1 reply; 4+ messages in thread From: Prabhakar @ 2026-03-26 14:21 UTC (permalink / raw) To: Mauro Carvalho Chehab, Sakari Ailus, Hans Verkuil, Hans de Goede, Laurent Pinchart, Vladimir Zapolskiy, Mehdi Djait, Xiaolei Wang, Benjamin Mugnier, Sylvain Petinot, Hardevsinh Palaniya Cc: linux-media, linux-kernel, linux-renesas-soc, Prabhakar, Biju Das, Fabrizio Castro, Lad Prabhakar From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Convert the ov5645 driver to use the V4L2 CCI register access helpers and regmap infrastructure instead of the custom I2C register access implementation. Keep ov5645_set_register_array() as ov5645_global_init_setting requires a delay between specific register writes, which cannot be expressed through the generic CCI multi-write helper. Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> --- drivers/media/i2c/Kconfig | 1 + drivers/media/i2c/ov5645.c | 901 ++++++++++++++++++------------------- 2 files changed, 429 insertions(+), 473 deletions(-) diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig index 20482be35f26..921186d36589 100644 --- a/drivers/media/i2c/Kconfig +++ b/drivers/media/i2c/Kconfig @@ -526,6 +526,7 @@ config VIDEO_OV5640 tristate "OmniVision OV5640 sensor support" depends on OF depends on GPIOLIB + select V4L2_CCI_I2C help This is a Video4Linux2 sensor driver for the Omnivision OV5640 camera sensor with a MIPI CSI-2 interface. diff --git a/drivers/media/i2c/ov5645.c b/drivers/media/i2c/ov5645.c index df9001fce44d..04097289e119 100644 --- a/drivers/media/i2c/ov5645.c +++ b/drivers/media/i2c/ov5645.c @@ -25,40 +25,42 @@ #include <linux/of.h> #include <linux/of_graph.h> #include <linux/pm_runtime.h> +#include <linux/regmap.h> #include <linux/regulator/consumer.h> #include <linux/slab.h> #include <linux/types.h> #include <media/mipi-csi2.h> +#include <media/v4l2-cci.h> #include <media/v4l2-ctrls.h> #include <media/v4l2-fwnode.h> #include <media/v4l2-subdev.h> -#define OV5645_SYSTEM_CTRL0 0x3008 +#define OV5645_SYSTEM_CTRL0 CCI_REG8(0x3008) #define OV5645_SYSTEM_CTRL0_START 0x02 #define OV5645_SYSTEM_CTRL0_STOP 0x42 -#define OV5645_CHIP_ID_HIGH 0x300a +#define OV5645_CHIP_ID_HIGH CCI_REG8(0x300a) #define OV5645_CHIP_ID_HIGH_BYTE 0x56 -#define OV5645_CHIP_ID_LOW 0x300b +#define OV5645_CHIP_ID_LOW CCI_REG8(0x300b) #define OV5645_CHIP_ID_LOW_BYTE 0x45 -#define OV5645_IO_MIPI_CTRL00 0x300e -#define OV5645_PAD_OUTPUT00 0x3019 -#define OV5645_AWB_MANUAL_CONTROL 0x3406 +#define OV5645_IO_MIPI_CTRL00 CCI_REG8(0x300e) +#define OV5645_PAD_OUTPUT00 CCI_REG8(0x3019) +#define OV5645_AWB_MANUAL_CONTROL CCI_REG8(0x3406) #define OV5645_AWB_MANUAL_ENABLE BIT(0) -#define OV5645_AEC_PK_MANUAL 0x3503 +#define OV5645_AEC_PK_MANUAL CCI_REG8(0x3503) #define OV5645_AEC_MANUAL_ENABLE BIT(0) #define OV5645_AGC_MANUAL_ENABLE BIT(1) -#define OV5645_TIMING_TC_REG20 0x3820 +#define OV5645_TIMING_TC_REG20 CCI_REG8(0x3820) #define OV5645_SENSOR_VFLIP BIT(1) #define OV5645_ISP_VFLIP BIT(2) -#define OV5645_TIMING_TC_REG21 0x3821 +#define OV5645_TIMING_TC_REG21 CCI_REG8(0x3821) #define OV5645_SENSOR_MIRROR BIT(1) -#define OV5645_MIPI_CTRL00 0x4800 -#define OV5645_PRE_ISP_TEST_SETTING_1 0x503d +#define OV5645_MIPI_CTRL00 CCI_REG8(0x4800) +#define OV5645_PRE_ISP_TEST_SETTING_1 CCI_REG8(0x503d) #define OV5645_TEST_PATTERN_MASK 0x3 #define OV5645_SET_TEST_PATTERN(x) ((x) & OV5645_TEST_PATTERN_MASK) #define OV5645_TEST_PATTERN_ENABLE BIT(7) -#define OV5645_SDE_SAT_U 0x5583 -#define OV5645_SDE_SAT_V 0x5584 +#define OV5645_SDE_SAT_U CCI_REG8(0x5583) +#define OV5645_SDE_SAT_V CCI_REG8(0x5584) /* regulator supplies */ static const char * const ov5645_supply_name[] = { @@ -71,15 +73,10 @@ static const char * const ov5645_supply_name[] = { #define OV5645_PAD_SOURCE 0 -struct reg_value { - u16 reg; - u8 val; -}; - struct ov5645_mode_info { u32 width; u32 height; - const struct reg_value *data; + const struct cci_reg_sequence *data; u32 data_size; u32 pixel_clock; u32 link_freq; @@ -88,6 +85,7 @@ struct ov5645_mode_info { struct ov5645 { struct i2c_client *i2c_client; struct device *dev; + struct regmap *regmap; struct v4l2_subdev sd; struct media_pad pad; struct v4l2_fwnode_endpoint ep; @@ -103,9 +101,9 @@ struct ov5645 { struct v4l2_ctrl *link_freq; /* Cached register values */ - u8 aec_pk_manual; - u8 timing_tc_reg20; - u8 timing_tc_reg21; + u64 aec_pk_manual; + u64 timing_tc_reg20; + u64 timing_tc_reg21; struct gpio_desc *enable_gpio; struct gpio_desc *rst_gpio; @@ -116,393 +114,393 @@ static inline struct ov5645 *to_ov5645(struct v4l2_subdev *sd) return container_of(sd, struct ov5645, sd); } -static const struct reg_value ov5645_global_init_setting[] = { - { 0x3103, 0x11 }, - { 0x3008, 0x42 }, - { 0x3103, 0x03 }, - { 0x3503, 0x07 }, - { 0x3002, 0x1c }, - { 0x3006, 0xc3 }, - { 0x3017, 0x00 }, - { 0x3018, 0x00 }, - { 0x302e, 0x0b }, - { 0x3037, 0x13 }, - { 0x3108, 0x01 }, - { 0x3611, 0x06 }, - { 0x3500, 0x00 }, - { 0x3501, 0x01 }, - { 0x3502, 0x00 }, - { 0x350a, 0x00 }, - { 0x350b, 0x3f }, - { 0x3620, 0x33 }, - { 0x3621, 0xe0 }, - { 0x3622, 0x01 }, - { 0x3630, 0x2e }, - { 0x3631, 0x00 }, - { 0x3632, 0x32 }, - { 0x3633, 0x52 }, - { 0x3634, 0x70 }, - { 0x3635, 0x13 }, - { 0x3636, 0x03 }, - { 0x3703, 0x5a }, - { 0x3704, 0xa0 }, - { 0x3705, 0x1a }, - { 0x3709, 0x12 }, - { 0x370b, 0x61 }, - { 0x370f, 0x10 }, - { 0x3715, 0x78 }, - { 0x3717, 0x01 }, - { 0x371b, 0x20 }, - { 0x3731, 0x12 }, - { 0x3901, 0x0a }, - { 0x3905, 0x02 }, - { 0x3906, 0x10 }, - { 0x3719, 0x86 }, - { 0x3810, 0x00 }, - { 0x3811, 0x10 }, - { 0x3812, 0x00 }, - { 0x3821, 0x01 }, - { 0x3824, 0x01 }, - { 0x3826, 0x03 }, - { 0x3828, 0x08 }, - { 0x3a19, 0xf8 }, - { 0x3c01, 0x34 }, - { 0x3c04, 0x28 }, - { 0x3c05, 0x98 }, - { 0x3c07, 0x07 }, - { 0x3c09, 0xc2 }, - { 0x3c0a, 0x9c }, - { 0x3c0b, 0x40 }, - { 0x3c01, 0x34 }, - { 0x4001, 0x02 }, - { 0x4514, 0x00 }, - { 0x4520, 0xb0 }, - { 0x460b, 0x37 }, - { 0x460c, 0x20 }, - { 0x4818, 0x01 }, - { 0x481d, 0xf0 }, - { 0x481f, 0x50 }, - { 0x4823, 0x70 }, - { 0x4831, 0x14 }, - { 0x5000, 0xa7 }, - { 0x5001, 0x83 }, - { 0x501d, 0x00 }, - { 0x501f, 0x00 }, - { 0x503d, 0x00 }, - { 0x505c, 0x30 }, - { 0x5181, 0x59 }, - { 0x5183, 0x00 }, - { 0x5191, 0xf0 }, - { 0x5192, 0x03 }, - { 0x5684, 0x10 }, - { 0x5685, 0xa0 }, - { 0x5686, 0x0c }, - { 0x5687, 0x78 }, - { 0x5a00, 0x08 }, - { 0x5a21, 0x00 }, - { 0x5a24, 0x00 }, - { 0x3008, 0x02 }, - { 0x3503, 0x00 }, - { 0x5180, 0xff }, - { 0x5181, 0xf2 }, - { 0x5182, 0x00 }, - { 0x5183, 0x14 }, - { 0x5184, 0x25 }, - { 0x5185, 0x24 }, - { 0x5186, 0x09 }, - { 0x5187, 0x09 }, - { 0x5188, 0x0a }, - { 0x5189, 0x75 }, - { 0x518a, 0x52 }, - { 0x518b, 0xea }, - { 0x518c, 0xa8 }, - { 0x518d, 0x42 }, - { 0x518e, 0x38 }, - { 0x518f, 0x56 }, - { 0x5190, 0x42 }, - { 0x5191, 0xf8 }, - { 0x5192, 0x04 }, - { 0x5193, 0x70 }, - { 0x5194, 0xf0 }, - { 0x5195, 0xf0 }, - { 0x5196, 0x03 }, - { 0x5197, 0x01 }, - { 0x5198, 0x04 }, - { 0x5199, 0x12 }, - { 0x519a, 0x04 }, - { 0x519b, 0x00 }, - { 0x519c, 0x06 }, - { 0x519d, 0x82 }, - { 0x519e, 0x38 }, - { 0x5381, 0x1e }, - { 0x5382, 0x5b }, - { 0x5383, 0x08 }, - { 0x5384, 0x0a }, - { 0x5385, 0x7e }, - { 0x5386, 0x88 }, - { 0x5387, 0x7c }, - { 0x5388, 0x6c }, - { 0x5389, 0x10 }, - { 0x538a, 0x01 }, - { 0x538b, 0x98 }, - { 0x5300, 0x08 }, - { 0x5301, 0x30 }, - { 0x5302, 0x10 }, - { 0x5303, 0x00 }, - { 0x5304, 0x08 }, - { 0x5305, 0x30 }, - { 0x5306, 0x08 }, - { 0x5307, 0x16 }, - { 0x5309, 0x08 }, - { 0x530a, 0x30 }, - { 0x530b, 0x04 }, - { 0x530c, 0x06 }, - { 0x5480, 0x01 }, - { 0x5481, 0x08 }, - { 0x5482, 0x14 }, - { 0x5483, 0x28 }, - { 0x5484, 0x51 }, - { 0x5485, 0x65 }, - { 0x5486, 0x71 }, - { 0x5487, 0x7d }, - { 0x5488, 0x87 }, - { 0x5489, 0x91 }, - { 0x548a, 0x9a }, - { 0x548b, 0xaa }, - { 0x548c, 0xb8 }, - { 0x548d, 0xcd }, - { 0x548e, 0xdd }, - { 0x548f, 0xea }, - { 0x5490, 0x1d }, - { 0x5580, 0x02 }, - { 0x5583, 0x40 }, - { 0x5584, 0x10 }, - { 0x5589, 0x10 }, - { 0x558a, 0x00 }, - { 0x558b, 0xf8 }, - { 0x5800, 0x3f }, - { 0x5801, 0x16 }, - { 0x5802, 0x0e }, - { 0x5803, 0x0d }, - { 0x5804, 0x17 }, - { 0x5805, 0x3f }, - { 0x5806, 0x0b }, - { 0x5807, 0x06 }, - { 0x5808, 0x04 }, - { 0x5809, 0x04 }, - { 0x580a, 0x06 }, - { 0x580b, 0x0b }, - { 0x580c, 0x09 }, - { 0x580d, 0x03 }, - { 0x580e, 0x00 }, - { 0x580f, 0x00 }, - { 0x5810, 0x03 }, - { 0x5811, 0x08 }, - { 0x5812, 0x0a }, - { 0x5813, 0x03 }, - { 0x5814, 0x00 }, - { 0x5815, 0x00 }, - { 0x5816, 0x04 }, - { 0x5817, 0x09 }, - { 0x5818, 0x0f }, - { 0x5819, 0x08 }, - { 0x581a, 0x06 }, - { 0x581b, 0x06 }, - { 0x581c, 0x08 }, - { 0x581d, 0x0c }, - { 0x581e, 0x3f }, - { 0x581f, 0x1e }, - { 0x5820, 0x12 }, - { 0x5821, 0x13 }, - { 0x5822, 0x21 }, - { 0x5823, 0x3f }, - { 0x5824, 0x68 }, - { 0x5825, 0x28 }, - { 0x5826, 0x2c }, - { 0x5827, 0x28 }, - { 0x5828, 0x08 }, - { 0x5829, 0x48 }, - { 0x582a, 0x64 }, - { 0x582b, 0x62 }, - { 0x582c, 0x64 }, - { 0x582d, 0x28 }, - { 0x582e, 0x46 }, - { 0x582f, 0x62 }, - { 0x5830, 0x60 }, - { 0x5831, 0x62 }, - { 0x5832, 0x26 }, - { 0x5833, 0x48 }, - { 0x5834, 0x66 }, - { 0x5835, 0x44 }, - { 0x5836, 0x64 }, - { 0x5837, 0x28 }, - { 0x5838, 0x66 }, - { 0x5839, 0x48 }, - { 0x583a, 0x2c }, - { 0x583b, 0x28 }, - { 0x583c, 0x26 }, - { 0x583d, 0xae }, - { 0x5025, 0x00 }, - { 0x3a0f, 0x30 }, - { 0x3a10, 0x28 }, - { 0x3a1b, 0x30 }, - { 0x3a1e, 0x26 }, - { 0x3a11, 0x60 }, - { 0x3a1f, 0x14 }, - { 0x0601, 0x02 }, - { 0x3008, 0x42 }, - { 0x3008, 0x02 }, +static const struct cci_reg_sequence ov5645_global_init_setting[] = { + { CCI_REG8(0x3103), 0x11 }, + { CCI_REG8(0x3008), 0x42 }, + { CCI_REG8(0x3103), 0x03 }, + { CCI_REG8(0x3503), 0x07 }, + { CCI_REG8(0x3002), 0x1c }, + { CCI_REG8(0x3006), 0xc3 }, + { CCI_REG8(0x3017), 0x00 }, + { CCI_REG8(0x3018), 0x00 }, + { CCI_REG8(0x302e), 0x0b }, + { CCI_REG8(0x3037), 0x13 }, + { CCI_REG8(0x3108), 0x01 }, + { CCI_REG8(0x3611), 0x06 }, + { CCI_REG8(0x3500), 0x00 }, + { CCI_REG8(0x3501), 0x01 }, + { CCI_REG8(0x3502), 0x00 }, + { CCI_REG8(0x350a), 0x00 }, + { CCI_REG8(0x350b), 0x3f }, + { CCI_REG8(0x3620), 0x33 }, + { CCI_REG8(0x3621), 0xe0 }, + { CCI_REG8(0x3622), 0x01 }, + { CCI_REG8(0x3630), 0x2e }, + { CCI_REG8(0x3631), 0x00 }, + { CCI_REG8(0x3632), 0x32 }, + { CCI_REG8(0x3633), 0x52 }, + { CCI_REG8(0x3634), 0x70 }, + { CCI_REG8(0x3635), 0x13 }, + { CCI_REG8(0x3636), 0x03 }, + { CCI_REG8(0x3703), 0x5a }, + { CCI_REG8(0x3704), 0xa0 }, + { CCI_REG8(0x3705), 0x1a }, + { CCI_REG8(0x3709), 0x12 }, + { CCI_REG8(0x370b), 0x61 }, + { CCI_REG8(0x370f), 0x10 }, + { CCI_REG8(0x3715), 0x78 }, + { CCI_REG8(0x3717), 0x01 }, + { CCI_REG8(0x371b), 0x20 }, + { CCI_REG8(0x3731), 0x12 }, + { CCI_REG8(0x3901), 0x0a }, + { CCI_REG8(0x3905), 0x02 }, + { CCI_REG8(0x3906), 0x10 }, + { CCI_REG8(0x3719), 0x86 }, + { CCI_REG8(0x3810), 0x00 }, + { CCI_REG8(0x3811), 0x10 }, + { CCI_REG8(0x3812), 0x00 }, + { CCI_REG8(0x3821), 0x01 }, + { CCI_REG8(0x3824), 0x01 }, + { CCI_REG8(0x3826), 0x03 }, + { CCI_REG8(0x3828), 0x08 }, + { CCI_REG8(0x3a19), 0xf8 }, + { CCI_REG8(0x3c01), 0x34 }, + { CCI_REG8(0x3c04), 0x28 }, + { CCI_REG8(0x3c05), 0x98 }, + { CCI_REG8(0x3c07), 0x07 }, + { CCI_REG8(0x3c09), 0xc2 }, + { CCI_REG8(0x3c0a), 0x9c }, + { CCI_REG8(0x3c0b), 0x40 }, + { CCI_REG8(0x3c01), 0x34 }, + { CCI_REG8(0x4001), 0x02 }, + { CCI_REG8(0x4514), 0x00 }, + { CCI_REG8(0x4520), 0xb0 }, + { CCI_REG8(0x460b), 0x37 }, + { CCI_REG8(0x460c), 0x20 }, + { CCI_REG8(0x4818), 0x01 }, + { CCI_REG8(0x481d), 0xf0 }, + { CCI_REG8(0x481f), 0x50 }, + { CCI_REG8(0x4823), 0x70 }, + { CCI_REG8(0x4831), 0x14 }, + { CCI_REG8(0x5000), 0xa7 }, + { CCI_REG8(0x5001), 0x83 }, + { CCI_REG8(0x501d), 0x00 }, + { CCI_REG8(0x501f), 0x00 }, + { CCI_REG8(0x503d), 0x00 }, + { CCI_REG8(0x505c), 0x30 }, + { CCI_REG8(0x5181), 0x59 }, + { CCI_REG8(0x5183), 0x00 }, + { CCI_REG8(0x5191), 0xf0 }, + { CCI_REG8(0x5192), 0x03 }, + { CCI_REG8(0x5684), 0x10 }, + { CCI_REG8(0x5685), 0xa0 }, + { CCI_REG8(0x5686), 0x0c }, + { CCI_REG8(0x5687), 0x78 }, + { CCI_REG8(0x5a00), 0x08 }, + { CCI_REG8(0x5a21), 0x00 }, + { CCI_REG8(0x5a24), 0x00 }, + { CCI_REG8(0x3008), 0x02 }, + { CCI_REG8(0x3503), 0x00 }, + { CCI_REG8(0x5180), 0xff }, + { CCI_REG8(0x5181), 0xf2 }, + { CCI_REG8(0x5182), 0x00 }, + { CCI_REG8(0x5183), 0x14 }, + { CCI_REG8(0x5184), 0x25 }, + { CCI_REG8(0x5185), 0x24 }, + { CCI_REG8(0x5186), 0x09 }, + { CCI_REG8(0x5187), 0x09 }, + { CCI_REG8(0x5188), 0x0a }, + { CCI_REG8(0x5189), 0x75 }, + { CCI_REG8(0x518a), 0x52 }, + { CCI_REG8(0x518b), 0xea }, + { CCI_REG8(0x518c), 0xa8 }, + { CCI_REG8(0x518d), 0x42 }, + { CCI_REG8(0x518e), 0x38 }, + { CCI_REG8(0x518f), 0x56 }, + { CCI_REG8(0x5190), 0x42 }, + { CCI_REG8(0x5191), 0xf8 }, + { CCI_REG8(0x5192), 0x04 }, + { CCI_REG8(0x5193), 0x70 }, + { CCI_REG8(0x5194), 0xf0 }, + { CCI_REG8(0x5195), 0xf0 }, + { CCI_REG8(0x5196), 0x03 }, + { CCI_REG8(0x5197), 0x01 }, + { CCI_REG8(0x5198), 0x04 }, + { CCI_REG8(0x5199), 0x12 }, + { CCI_REG8(0x519a), 0x04 }, + { CCI_REG8(0x519b), 0x00 }, + { CCI_REG8(0x519c), 0x06 }, + { CCI_REG8(0x519d), 0x82 }, + { CCI_REG8(0x519e), 0x38 }, + { CCI_REG8(0x5381), 0x1e }, + { CCI_REG8(0x5382), 0x5b }, + { CCI_REG8(0x5383), 0x08 }, + { CCI_REG8(0x5384), 0x0a }, + { CCI_REG8(0x5385), 0x7e }, + { CCI_REG8(0x5386), 0x88 }, + { CCI_REG8(0x5387), 0x7c }, + { CCI_REG8(0x5388), 0x6c }, + { CCI_REG8(0x5389), 0x10 }, + { CCI_REG8(0x538a), 0x01 }, + { CCI_REG8(0x538b), 0x98 }, + { CCI_REG8(0x5300), 0x08 }, + { CCI_REG8(0x5301), 0x30 }, + { CCI_REG8(0x5302), 0x10 }, + { CCI_REG8(0x5303), 0x00 }, + { CCI_REG8(0x5304), 0x08 }, + { CCI_REG8(0x5305), 0x30 }, + { CCI_REG8(0x5306), 0x08 }, + { CCI_REG8(0x5307), 0x16 }, + { CCI_REG8(0x5309), 0x08 }, + { CCI_REG8(0x530a), 0x30 }, + { CCI_REG8(0x530b), 0x04 }, + { CCI_REG8(0x530c), 0x06 }, + { CCI_REG8(0x5480), 0x01 }, + { CCI_REG8(0x5481), 0x08 }, + { CCI_REG8(0x5482), 0x14 }, + { CCI_REG8(0x5483), 0x28 }, + { CCI_REG8(0x5484), 0x51 }, + { CCI_REG8(0x5485), 0x65 }, + { CCI_REG8(0x5486), 0x71 }, + { CCI_REG8(0x5487), 0x7d }, + { CCI_REG8(0x5488), 0x87 }, + { CCI_REG8(0x5489), 0x91 }, + { CCI_REG8(0x548a), 0x9a }, + { CCI_REG8(0x548b), 0xaa }, + { CCI_REG8(0x548c), 0xb8 }, + { CCI_REG8(0x548d), 0xcd }, + { CCI_REG8(0x548e), 0xdd }, + { CCI_REG8(0x548f), 0xea }, + { CCI_REG8(0x5490), 0x1d }, + { CCI_REG8(0x5580), 0x02 }, + { CCI_REG8(0x5583), 0x40 }, + { CCI_REG8(0x5584), 0x10 }, + { CCI_REG8(0x5589), 0x10 }, + { CCI_REG8(0x558a), 0x00 }, + { CCI_REG8(0x558b), 0xf8 }, + { CCI_REG8(0x5800), 0x3f }, + { CCI_REG8(0x5801), 0x16 }, + { CCI_REG8(0x5802), 0x0e }, + { CCI_REG8(0x5803), 0x0d }, + { CCI_REG8(0x5804), 0x17 }, + { CCI_REG8(0x5805), 0x3f }, + { CCI_REG8(0x5806), 0x0b }, + { CCI_REG8(0x5807), 0x06 }, + { CCI_REG8(0x5808), 0x04 }, + { CCI_REG8(0x5809), 0x04 }, + { CCI_REG8(0x580a), 0x06 }, + { CCI_REG8(0x580b), 0x0b }, + { CCI_REG8(0x580c), 0x09 }, + { CCI_REG8(0x580d), 0x03 }, + { CCI_REG8(0x580e), 0x00 }, + { CCI_REG8(0x580f), 0x00 }, + { CCI_REG8(0x5810), 0x03 }, + { CCI_REG8(0x5811), 0x08 }, + { CCI_REG8(0x5812), 0x0a }, + { CCI_REG8(0x5813), 0x03 }, + { CCI_REG8(0x5814), 0x00 }, + { CCI_REG8(0x5815), 0x00 }, + { CCI_REG8(0x5816), 0x04 }, + { CCI_REG8(0x5817), 0x09 }, + { CCI_REG8(0x5818), 0x0f }, + { CCI_REG8(0x5819), 0x08 }, + { CCI_REG8(0x581a), 0x06 }, + { CCI_REG8(0x581b), 0x06 }, + { CCI_REG8(0x581c), 0x08 }, + { CCI_REG8(0x581d), 0x0c }, + { CCI_REG8(0x581e), 0x3f }, + { CCI_REG8(0x581f), 0x1e }, + { CCI_REG8(0x5820), 0x12 }, + { CCI_REG8(0x5821), 0x13 }, + { CCI_REG8(0x5822), 0x21 }, + { CCI_REG8(0x5823), 0x3f }, + { CCI_REG8(0x5824), 0x68 }, + { CCI_REG8(0x5825), 0x28 }, + { CCI_REG8(0x5826), 0x2c }, + { CCI_REG8(0x5827), 0x28 }, + { CCI_REG8(0x5828), 0x08 }, + { CCI_REG8(0x5829), 0x48 }, + { CCI_REG8(0x582a), 0x64 }, + { CCI_REG8(0x582b), 0x62 }, + { CCI_REG8(0x582c), 0x64 }, + { CCI_REG8(0x582d), 0x28 }, + { CCI_REG8(0x582e), 0x46 }, + { CCI_REG8(0x582f), 0x62 }, + { CCI_REG8(0x5830), 0x60 }, + { CCI_REG8(0x5831), 0x62 }, + { CCI_REG8(0x5832), 0x26 }, + { CCI_REG8(0x5833), 0x48 }, + { CCI_REG8(0x5834), 0x66 }, + { CCI_REG8(0x5835), 0x44 }, + { CCI_REG8(0x5836), 0x64 }, + { CCI_REG8(0x5837), 0x28 }, + { CCI_REG8(0x5838), 0x66 }, + { CCI_REG8(0x5839), 0x48 }, + { CCI_REG8(0x583a), 0x2c }, + { CCI_REG8(0x583b), 0x28 }, + { CCI_REG8(0x583c), 0x26 }, + { CCI_REG8(0x583d), 0xae }, + { CCI_REG8(0x5025), 0x00 }, + { CCI_REG8(0x3a0f), 0x30 }, + { CCI_REG8(0x3a10), 0x28 }, + { CCI_REG8(0x3a1b), 0x30 }, + { CCI_REG8(0x3a1e), 0x26 }, + { CCI_REG8(0x3a11), 0x60 }, + { CCI_REG8(0x3a1f), 0x14 }, + { CCI_REG8(0x0601), 0x02 }, + { CCI_REG8(0x3008), 0x42 }, + { CCI_REG8(0x3008), 0x02 }, { OV5645_IO_MIPI_CTRL00, 0x40 }, { OV5645_MIPI_CTRL00, 0x24 }, { OV5645_PAD_OUTPUT00, 0x70 } }; -static const struct reg_value ov5645_setting_sxga[] = { - { 0x3612, 0xa9 }, - { 0x3614, 0x50 }, - { 0x3618, 0x00 }, - { 0x3034, 0x18 }, - { 0x3035, 0x21 }, - { 0x3036, 0x70 }, - { 0x3600, 0x09 }, - { 0x3601, 0x43 }, - { 0x3708, 0x66 }, - { 0x370c, 0xc3 }, - { 0x3800, 0x00 }, - { 0x3801, 0x00 }, - { 0x3802, 0x00 }, - { 0x3803, 0x06 }, - { 0x3804, 0x0a }, - { 0x3805, 0x3f }, - { 0x3806, 0x07 }, - { 0x3807, 0x9d }, - { 0x3808, 0x05 }, - { 0x3809, 0x00 }, - { 0x380a, 0x03 }, - { 0x380b, 0xc0 }, - { 0x380c, 0x07 }, - { 0x380d, 0x68 }, - { 0x380e, 0x03 }, - { 0x380f, 0xd8 }, - { 0x3813, 0x06 }, - { 0x3814, 0x31 }, - { 0x3815, 0x31 }, - { 0x3820, 0x47 }, - { 0x3a02, 0x03 }, - { 0x3a03, 0xd8 }, - { 0x3a08, 0x01 }, - { 0x3a09, 0xf8 }, - { 0x3a0a, 0x01 }, - { 0x3a0b, 0xa4 }, - { 0x3a0e, 0x02 }, - { 0x3a0d, 0x02 }, - { 0x3a14, 0x03 }, - { 0x3a15, 0xd8 }, - { 0x3a18, 0x00 }, - { 0x4004, 0x02 }, - { 0x4005, 0x18 }, - { 0x4300, 0x32 }, - { 0x4202, 0x00 } +static const struct cci_reg_sequence ov5645_setting_sxga[] = { + { CCI_REG8(0x3612), 0xa9 }, + { CCI_REG8(0x3614), 0x50 }, + { CCI_REG8(0x3618), 0x00 }, + { CCI_REG8(0x3034), 0x18 }, + { CCI_REG8(0x3035), 0x21 }, + { CCI_REG8(0x3036), 0x70 }, + { CCI_REG8(0x3600), 0x09 }, + { CCI_REG8(0x3601), 0x43 }, + { CCI_REG8(0x3708), 0x66 }, + { CCI_REG8(0x370c), 0xc3 }, + { CCI_REG8(0x3800), 0x00 }, + { CCI_REG8(0x3801), 0x00 }, + { CCI_REG8(0x3802), 0x00 }, + { CCI_REG8(0x3803), 0x06 }, + { CCI_REG8(0x3804), 0x0a }, + { CCI_REG8(0x3805), 0x3f }, + { CCI_REG8(0x3806), 0x07 }, + { CCI_REG8(0x3807), 0x9d }, + { CCI_REG8(0x3808), 0x05 }, + { CCI_REG8(0x3809), 0x00 }, + { CCI_REG8(0x380a), 0x03 }, + { CCI_REG8(0x380b), 0xc0 }, + { CCI_REG8(0x380c), 0x07 }, + { CCI_REG8(0x380d), 0x68 }, + { CCI_REG8(0x380e), 0x03 }, + { CCI_REG8(0x380f), 0xd8 }, + { CCI_REG8(0x3813), 0x06 }, + { CCI_REG8(0x3814), 0x31 }, + { CCI_REG8(0x3815), 0x31 }, + { CCI_REG8(0x3820), 0x47 }, + { CCI_REG8(0x3a02), 0x03 }, + { CCI_REG8(0x3a03), 0xd8 }, + { CCI_REG8(0x3a08), 0x01 }, + { CCI_REG8(0x3a09), 0xf8 }, + { CCI_REG8(0x3a0a), 0x01 }, + { CCI_REG8(0x3a0b), 0xa4 }, + { CCI_REG8(0x3a0e), 0x02 }, + { CCI_REG8(0x3a0d), 0x02 }, + { CCI_REG8(0x3a14), 0x03 }, + { CCI_REG8(0x3a15), 0xd8 }, + { CCI_REG8(0x3a18), 0x00 }, + { CCI_REG8(0x4004), 0x02 }, + { CCI_REG8(0x4005), 0x18 }, + { CCI_REG8(0x4300), 0x32 }, + { CCI_REG8(0x4202), 0x00 } }; -static const struct reg_value ov5645_setting_1080p[] = { - { 0x3612, 0xab }, - { 0x3614, 0x50 }, - { 0x3618, 0x04 }, - { 0x3034, 0x18 }, - { 0x3035, 0x11 }, - { 0x3036, 0x54 }, - { 0x3600, 0x08 }, - { 0x3601, 0x33 }, - { 0x3708, 0x63 }, - { 0x370c, 0xc0 }, - { 0x3800, 0x01 }, - { 0x3801, 0x50 }, - { 0x3802, 0x01 }, - { 0x3803, 0xb2 }, - { 0x3804, 0x08 }, - { 0x3805, 0xef }, - { 0x3806, 0x05 }, - { 0x3807, 0xf1 }, - { 0x3808, 0x07 }, - { 0x3809, 0x80 }, - { 0x380a, 0x04 }, - { 0x380b, 0x38 }, - { 0x380c, 0x09 }, - { 0x380d, 0xc4 }, - { 0x380e, 0x04 }, - { 0x380f, 0x60 }, - { 0x3813, 0x04 }, - { 0x3814, 0x11 }, - { 0x3815, 0x11 }, - { 0x3820, 0x47 }, - { 0x4514, 0x88 }, - { 0x3a02, 0x04 }, - { 0x3a03, 0x60 }, - { 0x3a08, 0x01 }, - { 0x3a09, 0x50 }, - { 0x3a0a, 0x01 }, - { 0x3a0b, 0x18 }, - { 0x3a0e, 0x03 }, - { 0x3a0d, 0x04 }, - { 0x3a14, 0x04 }, - { 0x3a15, 0x60 }, - { 0x3a18, 0x00 }, - { 0x4004, 0x06 }, - { 0x4005, 0x18 }, - { 0x4300, 0x32 }, - { 0x4202, 0x00 }, - { 0x4837, 0x0b } +static const struct cci_reg_sequence ov5645_setting_1080p[] = { + { CCI_REG8(0x3612), 0xab }, + { CCI_REG8(0x3614), 0x50 }, + { CCI_REG8(0x3618), 0x04 }, + { CCI_REG8(0x3034), 0x18 }, + { CCI_REG8(0x3035), 0x11 }, + { CCI_REG8(0x3036), 0x54 }, + { CCI_REG8(0x3600), 0x08 }, + { CCI_REG8(0x3601), 0x33 }, + { CCI_REG8(0x3708), 0x63 }, + { CCI_REG8(0x370c), 0xc0 }, + { CCI_REG8(0x3800), 0x01 }, + { CCI_REG8(0x3801), 0x50 }, + { CCI_REG8(0x3802), 0x01 }, + { CCI_REG8(0x3803), 0xb2 }, + { CCI_REG8(0x3804), 0x08 }, + { CCI_REG8(0x3805), 0xef }, + { CCI_REG8(0x3806), 0x05 }, + { CCI_REG8(0x3807), 0xf1 }, + { CCI_REG8(0x3808), 0x07 }, + { CCI_REG8(0x3809), 0x80 }, + { CCI_REG8(0x380a), 0x04 }, + { CCI_REG8(0x380b), 0x38 }, + { CCI_REG8(0x380c), 0x09 }, + { CCI_REG8(0x380d), 0xc4 }, + { CCI_REG8(0x380e), 0x04 }, + { CCI_REG8(0x380f), 0x60 }, + { CCI_REG8(0x3813), 0x04 }, + { CCI_REG8(0x3814), 0x11 }, + { CCI_REG8(0x3815), 0x11 }, + { CCI_REG8(0x3820), 0x47 }, + { CCI_REG8(0x4514), 0x88 }, + { CCI_REG8(0x3a02), 0x04 }, + { CCI_REG8(0x3a03), 0x60 }, + { CCI_REG8(0x3a08), 0x01 }, + { CCI_REG8(0x3a09), 0x50 }, + { CCI_REG8(0x3a0a), 0x01 }, + { CCI_REG8(0x3a0b), 0x18 }, + { CCI_REG8(0x3a0e), 0x03 }, + { CCI_REG8(0x3a0d), 0x04 }, + { CCI_REG8(0x3a14), 0x04 }, + { CCI_REG8(0x3a15), 0x60 }, + { CCI_REG8(0x3a18), 0x00 }, + { CCI_REG8(0x4004), 0x06 }, + { CCI_REG8(0x4005), 0x18 }, + { CCI_REG8(0x4300), 0x32 }, + { CCI_REG8(0x4202), 0x00 }, + { CCI_REG8(0x4837), 0x0b } }; -static const struct reg_value ov5645_setting_full[] = { - { 0x3612, 0xab }, - { 0x3614, 0x50 }, - { 0x3618, 0x04 }, - { 0x3034, 0x18 }, - { 0x3035, 0x11 }, - { 0x3036, 0x54 }, - { 0x3600, 0x08 }, - { 0x3601, 0x33 }, - { 0x3708, 0x63 }, - { 0x370c, 0xc0 }, - { 0x3800, 0x00 }, - { 0x3801, 0x00 }, - { 0x3802, 0x00 }, - { 0x3803, 0x00 }, - { 0x3804, 0x0a }, - { 0x3805, 0x3f }, - { 0x3806, 0x07 }, - { 0x3807, 0x9f }, - { 0x3808, 0x0a }, - { 0x3809, 0x20 }, - { 0x380a, 0x07 }, - { 0x380b, 0x98 }, - { 0x380c, 0x0b }, - { 0x380d, 0x1c }, - { 0x380e, 0x07 }, - { 0x380f, 0xb0 }, - { 0x3813, 0x06 }, - { 0x3814, 0x11 }, - { 0x3815, 0x11 }, - { 0x3820, 0x47 }, - { 0x4514, 0x88 }, - { 0x3a02, 0x07 }, - { 0x3a03, 0xb0 }, - { 0x3a08, 0x01 }, - { 0x3a09, 0x27 }, - { 0x3a0a, 0x00 }, - { 0x3a0b, 0xf6 }, - { 0x3a0e, 0x06 }, - { 0x3a0d, 0x08 }, - { 0x3a14, 0x07 }, - { 0x3a15, 0xb0 }, - { 0x3a18, 0x01 }, - { 0x4004, 0x06 }, - { 0x4005, 0x18 }, - { 0x4300, 0x32 }, - { 0x4837, 0x0b }, - { 0x4202, 0x00 } +static const struct cci_reg_sequence ov5645_setting_full[] = { + { CCI_REG8(0x3612), 0xab }, + { CCI_REG8(0x3614), 0x50 }, + { CCI_REG8(0x3618), 0x04 }, + { CCI_REG8(0x3034), 0x18 }, + { CCI_REG8(0x3035), 0x11 }, + { CCI_REG8(0x3036), 0x54 }, + { CCI_REG8(0x3600), 0x08 }, + { CCI_REG8(0x3601), 0x33 }, + { CCI_REG8(0x3708), 0x63 }, + { CCI_REG8(0x370c), 0xc0 }, + { CCI_REG8(0x3800), 0x00 }, + { CCI_REG8(0x3801), 0x00 }, + { CCI_REG8(0x3802), 0x00 }, + { CCI_REG8(0x3803), 0x00 }, + { CCI_REG8(0x3804), 0x0a }, + { CCI_REG8(0x3805), 0x3f }, + { CCI_REG8(0x3806), 0x07 }, + { CCI_REG8(0x3807), 0x9f }, + { CCI_REG8(0x3808), 0x0a }, + { CCI_REG8(0x3809), 0x20 }, + { CCI_REG8(0x380a), 0x07 }, + { CCI_REG8(0x380b), 0x98 }, + { CCI_REG8(0x380c), 0x0b }, + { CCI_REG8(0x380d), 0x1c }, + { CCI_REG8(0x380e), 0x07 }, + { CCI_REG8(0x380f), 0xb0 }, + { CCI_REG8(0x3813), 0x06 }, + { CCI_REG8(0x3814), 0x11 }, + { CCI_REG8(0x3815), 0x11 }, + { CCI_REG8(0x3820), 0x47 }, + { CCI_REG8(0x4514), 0x88 }, + { CCI_REG8(0x3a02), 0x07 }, + { CCI_REG8(0x3a03), 0xb0 }, + { CCI_REG8(0x3a08), 0x01 }, + { CCI_REG8(0x3a09), 0x27 }, + { CCI_REG8(0x3a0a), 0x00 }, + { CCI_REG8(0x3a0b), 0xf6 }, + { CCI_REG8(0x3a0e), 0x06 }, + { CCI_REG8(0x3a0d), 0x08 }, + { CCI_REG8(0x3a14), 0x07 }, + { CCI_REG8(0x3a15), 0xb0 }, + { CCI_REG8(0x3a18), 0x01 }, + { CCI_REG8(0x4004), 0x06 }, + { CCI_REG8(0x4005), 0x18 }, + { CCI_REG8(0x4300), 0x32 }, + { CCI_REG8(0x4837), 0x0b }, + { CCI_REG8(0x4202), 0x00 } }; static const s64 link_freq[] = { @@ -537,50 +535,6 @@ static const struct ov5645_mode_info ov5645_mode_info_data[] = { }, }; -static int ov5645_write_reg(struct ov5645 *ov5645, u16 reg, u8 val) -{ - u8 regbuf[3]; - int ret; - - regbuf[0] = reg >> 8; - regbuf[1] = reg & 0xff; - regbuf[2] = val; - - ret = i2c_master_send(ov5645->i2c_client, regbuf, 3); - if (ret < 0) { - dev_err(ov5645->dev, "%s: write reg error %d: reg=%x, val=%x\n", - __func__, ret, reg, val); - return ret; - } - - return 0; -} - -static int ov5645_read_reg(struct ov5645 *ov5645, u16 reg, u8 *val) -{ - u8 regbuf[2]; - int ret; - - regbuf[0] = reg >> 8; - regbuf[1] = reg & 0xff; - - ret = i2c_master_send(ov5645->i2c_client, regbuf, 2); - if (ret < 0) { - dev_err(ov5645->dev, "%s: write reg error %d: reg=%x\n", - __func__, ret, reg); - return ret; - } - - ret = i2c_master_recv(ov5645->i2c_client, val, 1); - if (ret < 0) { - dev_err(ov5645->dev, "%s: read reg error %d: reg=%x\n", - __func__, ret, reg); - return ret; - } - - return 0; -} - static int ov5645_set_aec_mode(struct ov5645 *ov5645, u32 mode) { u8 val = ov5645->aec_pk_manual; @@ -591,7 +545,7 @@ static int ov5645_set_aec_mode(struct ov5645 *ov5645, u32 mode) else /* V4L2_EXPOSURE_MANUAL */ val |= OV5645_AEC_MANUAL_ENABLE; - ret = ov5645_write_reg(ov5645, OV5645_AEC_PK_MANUAL, val); + ret = cci_write(ov5645->regmap, OV5645_AEC_PK_MANUAL, val, NULL); if (!ret) ov5645->aec_pk_manual = val; @@ -608,7 +562,7 @@ static int ov5645_set_agc_mode(struct ov5645 *ov5645, u32 enable) else val |= OV5645_AGC_MANUAL_ENABLE; - ret = ov5645_write_reg(ov5645, OV5645_AEC_PK_MANUAL, val); + ret = cci_write(ov5645->regmap, OV5645_AEC_PK_MANUAL, val, NULL); if (!ret) ov5645->aec_pk_manual = val; @@ -616,14 +570,14 @@ static int ov5645_set_agc_mode(struct ov5645 *ov5645, u32 enable) } static int ov5645_set_register_array(struct ov5645 *ov5645, - const struct reg_value *settings, + const struct cci_reg_sequence *settings, unsigned int num_settings) { unsigned int i; int ret; for (i = 0; i < num_settings; ++i, ++settings) { - ret = ov5645_write_reg(ov5645, settings->reg, settings->val); + ret = cci_write(ov5645->regmap, settings->reg, settings->val, NULL); if (ret < 0) return ret; @@ -640,7 +594,7 @@ static void __ov5645_set_power_off(struct device *dev) struct v4l2_subdev *sd = dev_get_drvdata(dev); struct ov5645 *ov5645 = to_ov5645(sd); - ov5645_write_reg(ov5645, OV5645_IO_MIPI_CTRL00, 0x58); + cci_write(ov5645->regmap, OV5645_IO_MIPI_CTRL00, 0x58, NULL); gpiod_set_value_cansleep(ov5645->rst_gpio, 1); gpiod_set_value_cansleep(ov5645->enable_gpio, 0); regulator_bulk_disable(OV5645_NUM_SUPPLIES, ov5645->supplies); @@ -704,11 +658,11 @@ static int ov5645_set_saturation(struct ov5645 *ov5645, s32 value) u32 reg_value = (value * 0x10) + 0x40; int ret; - ret = ov5645_write_reg(ov5645, OV5645_SDE_SAT_U, reg_value); + ret = cci_write(ov5645->regmap, OV5645_SDE_SAT_U, reg_value, NULL); if (ret < 0) return ret; - return ov5645_write_reg(ov5645, OV5645_SDE_SAT_V, reg_value); + return cci_write(ov5645->regmap, OV5645_SDE_SAT_V, reg_value, NULL); } static int ov5645_set_hflip(struct ov5645 *ov5645, s32 value) @@ -721,7 +675,7 @@ static int ov5645_set_hflip(struct ov5645 *ov5645, s32 value) else val |= (OV5645_SENSOR_MIRROR); - ret = ov5645_write_reg(ov5645, OV5645_TIMING_TC_REG21, val); + ret = cci_write(ov5645->regmap, OV5645_TIMING_TC_REG21, val, NULL); if (!ret) ov5645->timing_tc_reg21 = val; @@ -738,7 +692,7 @@ static int ov5645_set_vflip(struct ov5645 *ov5645, s32 value) else val &= ~(OV5645_SENSOR_VFLIP | OV5645_ISP_VFLIP); - ret = ov5645_write_reg(ov5645, OV5645_TIMING_TC_REG20, val); + ret = cci_write(ov5645->regmap, OV5645_TIMING_TC_REG20, val, NULL); if (!ret) ov5645->timing_tc_reg20 = val; @@ -754,7 +708,7 @@ static int ov5645_set_test_pattern(struct ov5645 *ov5645, s32 value) val |= OV5645_TEST_PATTERN_ENABLE; } - return ov5645_write_reg(ov5645, OV5645_PRE_ISP_TEST_SETTING_1, val); + return cci_write(ov5645->regmap, OV5645_PRE_ISP_TEST_SETTING_1, val, NULL); } static const char * const ov5645_test_pattern_menu[] = { @@ -772,7 +726,7 @@ static int ov5645_set_awb(struct ov5645 *ov5645, s32 enable_auto) if (!enable_auto) val = OV5645_AWB_MANUAL_ENABLE; - return ov5645_write_reg(ov5645, OV5645_AWB_MANUAL_CONTROL, val); + return cci_write(ov5645->regmap, OV5645_AWB_MANUAL_CONTROL, val, NULL); } static int ov5645_s_ctrl(struct v4l2_ctrl *ctrl) @@ -958,9 +912,8 @@ static int ov5645_enable_streams(struct v4l2_subdev *sd, if (ret < 0) return ret; - ret = ov5645_set_register_array(ov5645, - ov5645->current_mode->data, - ov5645->current_mode->data_size); + ret = cci_multi_reg_write(ov5645->regmap, ov5645->current_mode->data, + ov5645->current_mode->data_size, NULL); if (ret < 0) { dev_err(ov5645->dev, "could not set mode %dx%d\n", ov5645->current_mode->width, @@ -973,12 +926,12 @@ static int ov5645_enable_streams(struct v4l2_subdev *sd, goto err_rpm_put; } - ret = ov5645_write_reg(ov5645, OV5645_IO_MIPI_CTRL00, 0x45); + ret = cci_write(ov5645->regmap, OV5645_IO_MIPI_CTRL00, 0x45, NULL); if (ret < 0) goto err_rpm_put; - ret = ov5645_write_reg(ov5645, OV5645_SYSTEM_CTRL0, - OV5645_SYSTEM_CTRL0_START); + ret = cci_write(ov5645->regmap, OV5645_SYSTEM_CTRL0, + OV5645_SYSTEM_CTRL0_START, NULL); if (ret < 0) goto err_rpm_put; @@ -996,12 +949,12 @@ static int ov5645_disable_streams(struct v4l2_subdev *sd, struct ov5645 *ov5645 = to_ov5645(sd); int ret; - ret = ov5645_write_reg(ov5645, OV5645_IO_MIPI_CTRL00, 0x40); + ret = cci_write(ov5645->regmap, OV5645_IO_MIPI_CTRL00, 0x40, NULL); if (ret < 0) goto rpm_put; - ret = ov5645_write_reg(ov5645, OV5645_SYSTEM_CTRL0, - OV5645_SYSTEM_CTRL0_STOP); + ret = cci_write(ov5645->regmap, OV5645_SYSTEM_CTRL0, + OV5645_SYSTEM_CTRL0_STOP, NULL); rpm_put: pm_runtime_put_autosuspend(ov5645->dev); @@ -1036,9 +989,9 @@ static const struct v4l2_subdev_internal_ops ov5645_internal_ops = { static int ov5645_probe(struct i2c_client *client) { struct device *dev = &client->dev; + u64 chip_id_high, chip_id_low; struct device_node *endpoint; struct ov5645 *ov5645; - u8 chip_id_high, chip_id_low; unsigned int i; u32 xclk_freq; int ret; @@ -1068,6 +1021,11 @@ static int ov5645_probe(struct i2c_client *client) return dev_err_probe(dev, -EINVAL, "invalid bus type, must be CSI2\n"); + ov5645->regmap = devm_cci_regmap_init_i2c(client, 16); + if (IS_ERR(ov5645->regmap)) + return dev_err_probe(ov5645->dev, PTR_ERR(ov5645->regmap), + "Failed to init CCI\n"); + /* get system clock (xclk) */ ov5645->xclk = devm_v4l2_sensor_clk_get_legacy(dev, NULL, false, 0); if (IS_ERR(ov5645->xclk)) @@ -1154,13 +1112,13 @@ static int ov5645_probe(struct i2c_client *client) if (ret) goto free_entity; - ret = ov5645_read_reg(ov5645, OV5645_CHIP_ID_HIGH, &chip_id_high); + ret = cci_read(ov5645->regmap, OV5645_CHIP_ID_HIGH, &chip_id_high, NULL); if (ret < 0 || chip_id_high != OV5645_CHIP_ID_HIGH_BYTE) { ret = -ENODEV; dev_err_probe(dev, ret, "could not read ID high\n"); goto power_down; } - ret = ov5645_read_reg(ov5645, OV5645_CHIP_ID_LOW, &chip_id_low); + ret = cci_read(ov5645->regmap, OV5645_CHIP_ID_LOW, &chip_id_low, NULL); if (ret < 0 || chip_id_low != OV5645_CHIP_ID_LOW_BYTE) { ret = -ENODEV; dev_err_probe(dev, ret, "could not read ID low\n"); @@ -1169,24 +1127,21 @@ static int ov5645_probe(struct i2c_client *client) dev_info(dev, "OV5645 detected at address 0x%02x\n", client->addr); - ret = ov5645_read_reg(ov5645, OV5645_AEC_PK_MANUAL, - &ov5645->aec_pk_manual); + ret = cci_read(ov5645->regmap, OV5645_AEC_PK_MANUAL, &ov5645->aec_pk_manual, NULL); if (ret < 0) { ret = -ENODEV; dev_err_probe(dev, ret, "could not read AEC/AGC mode\n"); goto power_down; } - ret = ov5645_read_reg(ov5645, OV5645_TIMING_TC_REG20, - &ov5645->timing_tc_reg20); + ret = cci_read(ov5645->regmap, OV5645_TIMING_TC_REG20, &ov5645->timing_tc_reg20, NULL); if (ret < 0) { ret = -ENODEV; dev_err_probe(dev, ret, "could not read vflip value\n"); goto power_down; } - ret = ov5645_read_reg(ov5645, OV5645_TIMING_TC_REG21, - &ov5645->timing_tc_reg21); + ret = cci_read(ov5645->regmap, OV5645_TIMING_TC_REG21, &ov5645->timing_tc_reg21, NULL); if (ret < 0) { ret = -ENODEV; dev_err_probe(dev, ret, "could not read hflip value\n"); -- 2.53.0 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 2/2] media: i2c: ov5645: Convert to CCI register access helpers 2026-03-26 14:21 ` [PATCH 2/2] media: i2c: ov5645: Convert to CCI register access helpers Prabhakar @ 2026-03-26 14:59 ` xiaolei wang 0 siblings, 0 replies; 4+ messages in thread From: xiaolei wang @ 2026-03-26 14:59 UTC (permalink / raw) To: Prabhakar, Mauro Carvalho Chehab, Sakari Ailus, Hans Verkuil, Hans de Goede, Laurent Pinchart, Vladimir Zapolskiy, Mehdi Djait, Benjamin Mugnier, Sylvain Petinot, Hardevsinh Palaniya Cc: linux-media, linux-kernel, linux-renesas-soc, Biju Das, Fabrizio Castro, Lad Prabhakar Hi Prabhakar, On 3/26/26 22:21, Prabhakar wrote: > CAUTION: This email comes from a non Wind River email account! > Do not click links or open attachments unless you recognize the sender and know the content is safe. > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > Convert the ov5645 driver to use the V4L2 CCI register access helpers > and regmap infrastructure instead of the custom I2C register access > implementation. > > Keep ov5645_set_register_array() as ov5645_global_init_setting requires > a delay between specific register writes, which cannot be expressed > through the generic CCI multi-write helper. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > --- > drivers/media/i2c/Kconfig | 1 + > drivers/media/i2c/ov5645.c | 901 ++++++++++++++++++------------------- > 2 files changed, 429 insertions(+), 473 deletions(-) > > diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig > index 20482be35f26..921186d36589 100644 > --- a/drivers/media/i2c/Kconfig > +++ b/drivers/media/i2c/Kconfig > @@ -526,6 +526,7 @@ config VIDEO_OV5640 > tristate "OmniVision OV5640 sensor support" > depends on OF > depends on GPIOLIB > + select V4L2_CCI_I2C The V4L2_CCI_I2C select option seems to have been placed here on the OV5640. thanks xiaolei > help > This is a Video4Linux2 sensor driver for the Omnivision > OV5640 camera sensor with a MIPI CSI-2 interface. > diff --git a/drivers/media/i2c/ov5645.c b/drivers/media/i2c/ov5645.c > index df9001fce44d..04097289e119 100644 > --- a/drivers/media/i2c/ov5645.c > +++ b/drivers/media/i2c/ov5645.c > @@ -25,40 +25,42 @@ > #include <linux/of.h> > #include <linux/of_graph.h> > #include <linux/pm_runtime.h> > +#include <linux/regmap.h> > #include <linux/regulator/consumer.h> > #include <linux/slab.h> > #include <linux/types.h> > #include <media/mipi-csi2.h> > +#include <media/v4l2-cci.h> > #include <media/v4l2-ctrls.h> > #include <media/v4l2-fwnode.h> > #include <media/v4l2-subdev.h> > > -#define OV5645_SYSTEM_CTRL0 0x3008 > +#define OV5645_SYSTEM_CTRL0 CCI_REG8(0x3008) > #define OV5645_SYSTEM_CTRL0_START 0x02 > #define OV5645_SYSTEM_CTRL0_STOP 0x42 > -#define OV5645_CHIP_ID_HIGH 0x300a > +#define OV5645_CHIP_ID_HIGH CCI_REG8(0x300a) > #define OV5645_CHIP_ID_HIGH_BYTE 0x56 > -#define OV5645_CHIP_ID_LOW 0x300b > +#define OV5645_CHIP_ID_LOW CCI_REG8(0x300b) > #define OV5645_CHIP_ID_LOW_BYTE 0x45 > -#define OV5645_IO_MIPI_CTRL00 0x300e > -#define OV5645_PAD_OUTPUT00 0x3019 > -#define OV5645_AWB_MANUAL_CONTROL 0x3406 > +#define OV5645_IO_MIPI_CTRL00 CCI_REG8(0x300e) > +#define OV5645_PAD_OUTPUT00 CCI_REG8(0x3019) > +#define OV5645_AWB_MANUAL_CONTROL CCI_REG8(0x3406) > #define OV5645_AWB_MANUAL_ENABLE BIT(0) > -#define OV5645_AEC_PK_MANUAL 0x3503 > +#define OV5645_AEC_PK_MANUAL CCI_REG8(0x3503) > #define OV5645_AEC_MANUAL_ENABLE BIT(0) > #define OV5645_AGC_MANUAL_ENABLE BIT(1) > -#define OV5645_TIMING_TC_REG20 0x3820 > +#define OV5645_TIMING_TC_REG20 CCI_REG8(0x3820) > #define OV5645_SENSOR_VFLIP BIT(1) > #define OV5645_ISP_VFLIP BIT(2) > -#define OV5645_TIMING_TC_REG21 0x3821 > +#define OV5645_TIMING_TC_REG21 CCI_REG8(0x3821) > #define OV5645_SENSOR_MIRROR BIT(1) > -#define OV5645_MIPI_CTRL00 0x4800 > -#define OV5645_PRE_ISP_TEST_SETTING_1 0x503d > +#define OV5645_MIPI_CTRL00 CCI_REG8(0x4800) > +#define OV5645_PRE_ISP_TEST_SETTING_1 CCI_REG8(0x503d) > #define OV5645_TEST_PATTERN_MASK 0x3 > #define OV5645_SET_TEST_PATTERN(x) ((x) & OV5645_TEST_PATTERN_MASK) > #define OV5645_TEST_PATTERN_ENABLE BIT(7) > -#define OV5645_SDE_SAT_U 0x5583 > -#define OV5645_SDE_SAT_V 0x5584 > +#define OV5645_SDE_SAT_U CCI_REG8(0x5583) > +#define OV5645_SDE_SAT_V CCI_REG8(0x5584) > > /* regulator supplies */ > static const char * const ov5645_supply_name[] = { > @@ -71,15 +73,10 @@ static const char * const ov5645_supply_name[] = { > > #define OV5645_PAD_SOURCE 0 > > -struct reg_value { > - u16 reg; > - u8 val; > -}; > - > struct ov5645_mode_info { > u32 width; > u32 height; > - const struct reg_value *data; > + const struct cci_reg_sequence *data; > u32 data_size; > u32 pixel_clock; > u32 link_freq; > @@ -88,6 +85,7 @@ struct ov5645_mode_info { > struct ov5645 { > struct i2c_client *i2c_client; > struct device *dev; > + struct regmap *regmap; > struct v4l2_subdev sd; > struct media_pad pad; > struct v4l2_fwnode_endpoint ep; > @@ -103,9 +101,9 @@ struct ov5645 { > struct v4l2_ctrl *link_freq; > > /* Cached register values */ > - u8 aec_pk_manual; > - u8 timing_tc_reg20; > - u8 timing_tc_reg21; > + u64 aec_pk_manual; > + u64 timing_tc_reg20; > + u64 timing_tc_reg21; > > struct gpio_desc *enable_gpio; > struct gpio_desc *rst_gpio; > @@ -116,393 +114,393 @@ static inline struct ov5645 *to_ov5645(struct v4l2_subdev *sd) > return container_of(sd, struct ov5645, sd); > } > > -static const struct reg_value ov5645_global_init_setting[] = { > - { 0x3103, 0x11 }, > - { 0x3008, 0x42 }, > - { 0x3103, 0x03 }, > - { 0x3503, 0x07 }, > - { 0x3002, 0x1c }, > - { 0x3006, 0xc3 }, > - { 0x3017, 0x00 }, > - { 0x3018, 0x00 }, > - { 0x302e, 0x0b }, > - { 0x3037, 0x13 }, > - { 0x3108, 0x01 }, > - { 0x3611, 0x06 }, > - { 0x3500, 0x00 }, > - { 0x3501, 0x01 }, > - { 0x3502, 0x00 }, > - { 0x350a, 0x00 }, > - { 0x350b, 0x3f }, > - { 0x3620, 0x33 }, > - { 0x3621, 0xe0 }, > - { 0x3622, 0x01 }, > - { 0x3630, 0x2e }, > - { 0x3631, 0x00 }, > - { 0x3632, 0x32 }, > - { 0x3633, 0x52 }, > - { 0x3634, 0x70 }, > - { 0x3635, 0x13 }, > - { 0x3636, 0x03 }, > - { 0x3703, 0x5a }, > - { 0x3704, 0xa0 }, > - { 0x3705, 0x1a }, > - { 0x3709, 0x12 }, > - { 0x370b, 0x61 }, > - { 0x370f, 0x10 }, > - { 0x3715, 0x78 }, > - { 0x3717, 0x01 }, > - { 0x371b, 0x20 }, > - { 0x3731, 0x12 }, > - { 0x3901, 0x0a }, > - { 0x3905, 0x02 }, > - { 0x3906, 0x10 }, > - { 0x3719, 0x86 }, > - { 0x3810, 0x00 }, > - { 0x3811, 0x10 }, > - { 0x3812, 0x00 }, > - { 0x3821, 0x01 }, > - { 0x3824, 0x01 }, > - { 0x3826, 0x03 }, > - { 0x3828, 0x08 }, > - { 0x3a19, 0xf8 }, > - { 0x3c01, 0x34 }, > - { 0x3c04, 0x28 }, > - { 0x3c05, 0x98 }, > - { 0x3c07, 0x07 }, > - { 0x3c09, 0xc2 }, > - { 0x3c0a, 0x9c }, > - { 0x3c0b, 0x40 }, > - { 0x3c01, 0x34 }, > - { 0x4001, 0x02 }, > - { 0x4514, 0x00 }, > - { 0x4520, 0xb0 }, > - { 0x460b, 0x37 }, > - { 0x460c, 0x20 }, > - { 0x4818, 0x01 }, > - { 0x481d, 0xf0 }, > - { 0x481f, 0x50 }, > - { 0x4823, 0x70 }, > - { 0x4831, 0x14 }, > - { 0x5000, 0xa7 }, > - { 0x5001, 0x83 }, > - { 0x501d, 0x00 }, > - { 0x501f, 0x00 }, > - { 0x503d, 0x00 }, > - { 0x505c, 0x30 }, > - { 0x5181, 0x59 }, > - { 0x5183, 0x00 }, > - { 0x5191, 0xf0 }, > - { 0x5192, 0x03 }, > - { 0x5684, 0x10 }, > - { 0x5685, 0xa0 }, > - { 0x5686, 0x0c }, > - { 0x5687, 0x78 }, > - { 0x5a00, 0x08 }, > - { 0x5a21, 0x00 }, > - { 0x5a24, 0x00 }, > - { 0x3008, 0x02 }, > - { 0x3503, 0x00 }, > - { 0x5180, 0xff }, > - { 0x5181, 0xf2 }, > - { 0x5182, 0x00 }, > - { 0x5183, 0x14 }, > - { 0x5184, 0x25 }, > - { 0x5185, 0x24 }, > - { 0x5186, 0x09 }, > - { 0x5187, 0x09 }, > - { 0x5188, 0x0a }, > - { 0x5189, 0x75 }, > - { 0x518a, 0x52 }, > - { 0x518b, 0xea }, > - { 0x518c, 0xa8 }, > - { 0x518d, 0x42 }, > - { 0x518e, 0x38 }, > - { 0x518f, 0x56 }, > - { 0x5190, 0x42 }, > - { 0x5191, 0xf8 }, > - { 0x5192, 0x04 }, > - { 0x5193, 0x70 }, > - { 0x5194, 0xf0 }, > - { 0x5195, 0xf0 }, > - { 0x5196, 0x03 }, > - { 0x5197, 0x01 }, > - { 0x5198, 0x04 }, > - { 0x5199, 0x12 }, > - { 0x519a, 0x04 }, > - { 0x519b, 0x00 }, > - { 0x519c, 0x06 }, > - { 0x519d, 0x82 }, > - { 0x519e, 0x38 }, > - { 0x5381, 0x1e }, > - { 0x5382, 0x5b }, > - { 0x5383, 0x08 }, > - { 0x5384, 0x0a }, > - { 0x5385, 0x7e }, > - { 0x5386, 0x88 }, > - { 0x5387, 0x7c }, > - { 0x5388, 0x6c }, > - { 0x5389, 0x10 }, > - { 0x538a, 0x01 }, > - { 0x538b, 0x98 }, > - { 0x5300, 0x08 }, > - { 0x5301, 0x30 }, > - { 0x5302, 0x10 }, > - { 0x5303, 0x00 }, > - { 0x5304, 0x08 }, > - { 0x5305, 0x30 }, > - { 0x5306, 0x08 }, > - { 0x5307, 0x16 }, > - { 0x5309, 0x08 }, > - { 0x530a, 0x30 }, > - { 0x530b, 0x04 }, > - { 0x530c, 0x06 }, > - { 0x5480, 0x01 }, > - { 0x5481, 0x08 }, > - { 0x5482, 0x14 }, > - { 0x5483, 0x28 }, > - { 0x5484, 0x51 }, > - { 0x5485, 0x65 }, > - { 0x5486, 0x71 }, > - { 0x5487, 0x7d }, > - { 0x5488, 0x87 }, > - { 0x5489, 0x91 }, > - { 0x548a, 0x9a }, > - { 0x548b, 0xaa }, > - { 0x548c, 0xb8 }, > - { 0x548d, 0xcd }, > - { 0x548e, 0xdd }, > - { 0x548f, 0xea }, > - { 0x5490, 0x1d }, > - { 0x5580, 0x02 }, > - { 0x5583, 0x40 }, > - { 0x5584, 0x10 }, > - { 0x5589, 0x10 }, > - { 0x558a, 0x00 }, > - { 0x558b, 0xf8 }, > - { 0x5800, 0x3f }, > - { 0x5801, 0x16 }, > - { 0x5802, 0x0e }, > - { 0x5803, 0x0d }, > - { 0x5804, 0x17 }, > - { 0x5805, 0x3f }, > - { 0x5806, 0x0b }, > - { 0x5807, 0x06 }, > - { 0x5808, 0x04 }, > - { 0x5809, 0x04 }, > - { 0x580a, 0x06 }, > - { 0x580b, 0x0b }, > - { 0x580c, 0x09 }, > - { 0x580d, 0x03 }, > - { 0x580e, 0x00 }, > - { 0x580f, 0x00 }, > - { 0x5810, 0x03 }, > - { 0x5811, 0x08 }, > - { 0x5812, 0x0a }, > - { 0x5813, 0x03 }, > - { 0x5814, 0x00 }, > - { 0x5815, 0x00 }, > - { 0x5816, 0x04 }, > - { 0x5817, 0x09 }, > - { 0x5818, 0x0f }, > - { 0x5819, 0x08 }, > - { 0x581a, 0x06 }, > - { 0x581b, 0x06 }, > - { 0x581c, 0x08 }, > - { 0x581d, 0x0c }, > - { 0x581e, 0x3f }, > - { 0x581f, 0x1e }, > - { 0x5820, 0x12 }, > - { 0x5821, 0x13 }, > - { 0x5822, 0x21 }, > - { 0x5823, 0x3f }, > - { 0x5824, 0x68 }, > - { 0x5825, 0x28 }, > - { 0x5826, 0x2c }, > - { 0x5827, 0x28 }, > - { 0x5828, 0x08 }, > - { 0x5829, 0x48 }, > - { 0x582a, 0x64 }, > - { 0x582b, 0x62 }, > - { 0x582c, 0x64 }, > - { 0x582d, 0x28 }, > - { 0x582e, 0x46 }, > - { 0x582f, 0x62 }, > - { 0x5830, 0x60 }, > - { 0x5831, 0x62 }, > - { 0x5832, 0x26 }, > - { 0x5833, 0x48 }, > - { 0x5834, 0x66 }, > - { 0x5835, 0x44 }, > - { 0x5836, 0x64 }, > - { 0x5837, 0x28 }, > - { 0x5838, 0x66 }, > - { 0x5839, 0x48 }, > - { 0x583a, 0x2c }, > - { 0x583b, 0x28 }, > - { 0x583c, 0x26 }, > - { 0x583d, 0xae }, > - { 0x5025, 0x00 }, > - { 0x3a0f, 0x30 }, > - { 0x3a10, 0x28 }, > - { 0x3a1b, 0x30 }, > - { 0x3a1e, 0x26 }, > - { 0x3a11, 0x60 }, > - { 0x3a1f, 0x14 }, > - { 0x0601, 0x02 }, > - { 0x3008, 0x42 }, > - { 0x3008, 0x02 }, > +static const struct cci_reg_sequence ov5645_global_init_setting[] = { > + { CCI_REG8(0x3103), 0x11 }, > + { CCI_REG8(0x3008), 0x42 }, > + { CCI_REG8(0x3103), 0x03 }, > + { CCI_REG8(0x3503), 0x07 }, > + { CCI_REG8(0x3002), 0x1c }, > + { CCI_REG8(0x3006), 0xc3 }, > + { CCI_REG8(0x3017), 0x00 }, > + { CCI_REG8(0x3018), 0x00 }, > + { CCI_REG8(0x302e), 0x0b }, > + { CCI_REG8(0x3037), 0x13 }, > + { CCI_REG8(0x3108), 0x01 }, > + { CCI_REG8(0x3611), 0x06 }, > + { CCI_REG8(0x3500), 0x00 }, > + { CCI_REG8(0x3501), 0x01 }, > + { CCI_REG8(0x3502), 0x00 }, > + { CCI_REG8(0x350a), 0x00 }, > + { CCI_REG8(0x350b), 0x3f }, > + { CCI_REG8(0x3620), 0x33 }, > + { CCI_REG8(0x3621), 0xe0 }, > + { CCI_REG8(0x3622), 0x01 }, > + { CCI_REG8(0x3630), 0x2e }, > + { CCI_REG8(0x3631), 0x00 }, > + { CCI_REG8(0x3632), 0x32 }, > + { CCI_REG8(0x3633), 0x52 }, > + { CCI_REG8(0x3634), 0x70 }, > + { CCI_REG8(0x3635), 0x13 }, > + { CCI_REG8(0x3636), 0x03 }, > + { CCI_REG8(0x3703), 0x5a }, > + { CCI_REG8(0x3704), 0xa0 }, > + { CCI_REG8(0x3705), 0x1a }, > + { CCI_REG8(0x3709), 0x12 }, > + { CCI_REG8(0x370b), 0x61 }, > + { CCI_REG8(0x370f), 0x10 }, > + { CCI_REG8(0x3715), 0x78 }, > + { CCI_REG8(0x3717), 0x01 }, > + { CCI_REG8(0x371b), 0x20 }, > + { CCI_REG8(0x3731), 0x12 }, > + { CCI_REG8(0x3901), 0x0a }, > + { CCI_REG8(0x3905), 0x02 }, > + { CCI_REG8(0x3906), 0x10 }, > + { CCI_REG8(0x3719), 0x86 }, > + { CCI_REG8(0x3810), 0x00 }, > + { CCI_REG8(0x3811), 0x10 }, > + { CCI_REG8(0x3812), 0x00 }, > + { CCI_REG8(0x3821), 0x01 }, > + { CCI_REG8(0x3824), 0x01 }, > + { CCI_REG8(0x3826), 0x03 }, > + { CCI_REG8(0x3828), 0x08 }, > + { CCI_REG8(0x3a19), 0xf8 }, > + { CCI_REG8(0x3c01), 0x34 }, > + { CCI_REG8(0x3c04), 0x28 }, > + { CCI_REG8(0x3c05), 0x98 }, > + { CCI_REG8(0x3c07), 0x07 }, > + { CCI_REG8(0x3c09), 0xc2 }, > + { CCI_REG8(0x3c0a), 0x9c }, > + { CCI_REG8(0x3c0b), 0x40 }, > + { CCI_REG8(0x3c01), 0x34 }, > + { CCI_REG8(0x4001), 0x02 }, > + { CCI_REG8(0x4514), 0x00 }, > + { CCI_REG8(0x4520), 0xb0 }, > + { CCI_REG8(0x460b), 0x37 }, > + { CCI_REG8(0x460c), 0x20 }, > + { CCI_REG8(0x4818), 0x01 }, > + { CCI_REG8(0x481d), 0xf0 }, > + { CCI_REG8(0x481f), 0x50 }, > + { CCI_REG8(0x4823), 0x70 }, > + { CCI_REG8(0x4831), 0x14 }, > + { CCI_REG8(0x5000), 0xa7 }, > + { CCI_REG8(0x5001), 0x83 }, > + { CCI_REG8(0x501d), 0x00 }, > + { CCI_REG8(0x501f), 0x00 }, > + { CCI_REG8(0x503d), 0x00 }, > + { CCI_REG8(0x505c), 0x30 }, > + { CCI_REG8(0x5181), 0x59 }, > + { CCI_REG8(0x5183), 0x00 }, > + { CCI_REG8(0x5191), 0xf0 }, > + { CCI_REG8(0x5192), 0x03 }, > + { CCI_REG8(0x5684), 0x10 }, > + { CCI_REG8(0x5685), 0xa0 }, > + { CCI_REG8(0x5686), 0x0c }, > + { CCI_REG8(0x5687), 0x78 }, > + { CCI_REG8(0x5a00), 0x08 }, > + { CCI_REG8(0x5a21), 0x00 }, > + { CCI_REG8(0x5a24), 0x00 }, > + { CCI_REG8(0x3008), 0x02 }, > + { CCI_REG8(0x3503), 0x00 }, > + { CCI_REG8(0x5180), 0xff }, > + { CCI_REG8(0x5181), 0xf2 }, > + { CCI_REG8(0x5182), 0x00 }, > + { CCI_REG8(0x5183), 0x14 }, > + { CCI_REG8(0x5184), 0x25 }, > + { CCI_REG8(0x5185), 0x24 }, > + { CCI_REG8(0x5186), 0x09 }, > + { CCI_REG8(0x5187), 0x09 }, > + { CCI_REG8(0x5188), 0x0a }, > + { CCI_REG8(0x5189), 0x75 }, > + { CCI_REG8(0x518a), 0x52 }, > + { CCI_REG8(0x518b), 0xea }, > + { CCI_REG8(0x518c), 0xa8 }, > + { CCI_REG8(0x518d), 0x42 }, > + { CCI_REG8(0x518e), 0x38 }, > + { CCI_REG8(0x518f), 0x56 }, > + { CCI_REG8(0x5190), 0x42 }, > + { CCI_REG8(0x5191), 0xf8 }, > + { CCI_REG8(0x5192), 0x04 }, > + { CCI_REG8(0x5193), 0x70 }, > + { CCI_REG8(0x5194), 0xf0 }, > + { CCI_REG8(0x5195), 0xf0 }, > + { CCI_REG8(0x5196), 0x03 }, > + { CCI_REG8(0x5197), 0x01 }, > + { CCI_REG8(0x5198), 0x04 }, > + { CCI_REG8(0x5199), 0x12 }, > + { CCI_REG8(0x519a), 0x04 }, > + { CCI_REG8(0x519b), 0x00 }, > + { CCI_REG8(0x519c), 0x06 }, > + { CCI_REG8(0x519d), 0x82 }, > + { CCI_REG8(0x519e), 0x38 }, > + { CCI_REG8(0x5381), 0x1e }, > + { CCI_REG8(0x5382), 0x5b }, > + { CCI_REG8(0x5383), 0x08 }, > + { CCI_REG8(0x5384), 0x0a }, > + { CCI_REG8(0x5385), 0x7e }, > + { CCI_REG8(0x5386), 0x88 }, > + { CCI_REG8(0x5387), 0x7c }, > + { CCI_REG8(0x5388), 0x6c }, > + { CCI_REG8(0x5389), 0x10 }, > + { CCI_REG8(0x538a), 0x01 }, > + { CCI_REG8(0x538b), 0x98 }, > + { CCI_REG8(0x5300), 0x08 }, > + { CCI_REG8(0x5301), 0x30 }, > + { CCI_REG8(0x5302), 0x10 }, > + { CCI_REG8(0x5303), 0x00 }, > + { CCI_REG8(0x5304), 0x08 }, > + { CCI_REG8(0x5305), 0x30 }, > + { CCI_REG8(0x5306), 0x08 }, > + { CCI_REG8(0x5307), 0x16 }, > + { CCI_REG8(0x5309), 0x08 }, > + { CCI_REG8(0x530a), 0x30 }, > + { CCI_REG8(0x530b), 0x04 }, > + { CCI_REG8(0x530c), 0x06 }, > + { CCI_REG8(0x5480), 0x01 }, > + { CCI_REG8(0x5481), 0x08 }, > + { CCI_REG8(0x5482), 0x14 }, > + { CCI_REG8(0x5483), 0x28 }, > + { CCI_REG8(0x5484), 0x51 }, > + { CCI_REG8(0x5485), 0x65 }, > + { CCI_REG8(0x5486), 0x71 }, > + { CCI_REG8(0x5487), 0x7d }, > + { CCI_REG8(0x5488), 0x87 }, > + { CCI_REG8(0x5489), 0x91 }, > + { CCI_REG8(0x548a), 0x9a }, > + { CCI_REG8(0x548b), 0xaa }, > + { CCI_REG8(0x548c), 0xb8 }, > + { CCI_REG8(0x548d), 0xcd }, > + { CCI_REG8(0x548e), 0xdd }, > + { CCI_REG8(0x548f), 0xea }, > + { CCI_REG8(0x5490), 0x1d }, > + { CCI_REG8(0x5580), 0x02 }, > + { CCI_REG8(0x5583), 0x40 }, > + { CCI_REG8(0x5584), 0x10 }, > + { CCI_REG8(0x5589), 0x10 }, > + { CCI_REG8(0x558a), 0x00 }, > + { CCI_REG8(0x558b), 0xf8 }, > + { CCI_REG8(0x5800), 0x3f }, > + { CCI_REG8(0x5801), 0x16 }, > + { CCI_REG8(0x5802), 0x0e }, > + { CCI_REG8(0x5803), 0x0d }, > + { CCI_REG8(0x5804), 0x17 }, > + { CCI_REG8(0x5805), 0x3f }, > + { CCI_REG8(0x5806), 0x0b }, > + { CCI_REG8(0x5807), 0x06 }, > + { CCI_REG8(0x5808), 0x04 }, > + { CCI_REG8(0x5809), 0x04 }, > + { CCI_REG8(0x580a), 0x06 }, > + { CCI_REG8(0x580b), 0x0b }, > + { CCI_REG8(0x580c), 0x09 }, > + { CCI_REG8(0x580d), 0x03 }, > + { CCI_REG8(0x580e), 0x00 }, > + { CCI_REG8(0x580f), 0x00 }, > + { CCI_REG8(0x5810), 0x03 }, > + { CCI_REG8(0x5811), 0x08 }, > + { CCI_REG8(0x5812), 0x0a }, > + { CCI_REG8(0x5813), 0x03 }, > + { CCI_REG8(0x5814), 0x00 }, > + { CCI_REG8(0x5815), 0x00 }, > + { CCI_REG8(0x5816), 0x04 }, > + { CCI_REG8(0x5817), 0x09 }, > + { CCI_REG8(0x5818), 0x0f }, > + { CCI_REG8(0x5819), 0x08 }, > + { CCI_REG8(0x581a), 0x06 }, > + { CCI_REG8(0x581b), 0x06 }, > + { CCI_REG8(0x581c), 0x08 }, > + { CCI_REG8(0x581d), 0x0c }, > + { CCI_REG8(0x581e), 0x3f }, > + { CCI_REG8(0x581f), 0x1e }, > + { CCI_REG8(0x5820), 0x12 }, > + { CCI_REG8(0x5821), 0x13 }, > + { CCI_REG8(0x5822), 0x21 }, > + { CCI_REG8(0x5823), 0x3f }, > + { CCI_REG8(0x5824), 0x68 }, > + { CCI_REG8(0x5825), 0x28 }, > + { CCI_REG8(0x5826), 0x2c }, > + { CCI_REG8(0x5827), 0x28 }, > + { CCI_REG8(0x5828), 0x08 }, > + { CCI_REG8(0x5829), 0x48 }, > + { CCI_REG8(0x582a), 0x64 }, > + { CCI_REG8(0x582b), 0x62 }, > + { CCI_REG8(0x582c), 0x64 }, > + { CCI_REG8(0x582d), 0x28 }, > + { CCI_REG8(0x582e), 0x46 }, > + { CCI_REG8(0x582f), 0x62 }, > + { CCI_REG8(0x5830), 0x60 }, > + { CCI_REG8(0x5831), 0x62 }, > + { CCI_REG8(0x5832), 0x26 }, > + { CCI_REG8(0x5833), 0x48 }, > + { CCI_REG8(0x5834), 0x66 }, > + { CCI_REG8(0x5835), 0x44 }, > + { CCI_REG8(0x5836), 0x64 }, > + { CCI_REG8(0x5837), 0x28 }, > + { CCI_REG8(0x5838), 0x66 }, > + { CCI_REG8(0x5839), 0x48 }, > + { CCI_REG8(0x583a), 0x2c }, > + { CCI_REG8(0x583b), 0x28 }, > + { CCI_REG8(0x583c), 0x26 }, > + { CCI_REG8(0x583d), 0xae }, > + { CCI_REG8(0x5025), 0x00 }, > + { CCI_REG8(0x3a0f), 0x30 }, > + { CCI_REG8(0x3a10), 0x28 }, > + { CCI_REG8(0x3a1b), 0x30 }, > + { CCI_REG8(0x3a1e), 0x26 }, > + { CCI_REG8(0x3a11), 0x60 }, > + { CCI_REG8(0x3a1f), 0x14 }, > + { CCI_REG8(0x0601), 0x02 }, > + { CCI_REG8(0x3008), 0x42 }, > + { CCI_REG8(0x3008), 0x02 }, > { OV5645_IO_MIPI_CTRL00, 0x40 }, > { OV5645_MIPI_CTRL00, 0x24 }, > { OV5645_PAD_OUTPUT00, 0x70 } > }; > > -static const struct reg_value ov5645_setting_sxga[] = { > - { 0x3612, 0xa9 }, > - { 0x3614, 0x50 }, > - { 0x3618, 0x00 }, > - { 0x3034, 0x18 }, > - { 0x3035, 0x21 }, > - { 0x3036, 0x70 }, > - { 0x3600, 0x09 }, > - { 0x3601, 0x43 }, > - { 0x3708, 0x66 }, > - { 0x370c, 0xc3 }, > - { 0x3800, 0x00 }, > - { 0x3801, 0x00 }, > - { 0x3802, 0x00 }, > - { 0x3803, 0x06 }, > - { 0x3804, 0x0a }, > - { 0x3805, 0x3f }, > - { 0x3806, 0x07 }, > - { 0x3807, 0x9d }, > - { 0x3808, 0x05 }, > - { 0x3809, 0x00 }, > - { 0x380a, 0x03 }, > - { 0x380b, 0xc0 }, > - { 0x380c, 0x07 }, > - { 0x380d, 0x68 }, > - { 0x380e, 0x03 }, > - { 0x380f, 0xd8 }, > - { 0x3813, 0x06 }, > - { 0x3814, 0x31 }, > - { 0x3815, 0x31 }, > - { 0x3820, 0x47 }, > - { 0x3a02, 0x03 }, > - { 0x3a03, 0xd8 }, > - { 0x3a08, 0x01 }, > - { 0x3a09, 0xf8 }, > - { 0x3a0a, 0x01 }, > - { 0x3a0b, 0xa4 }, > - { 0x3a0e, 0x02 }, > - { 0x3a0d, 0x02 }, > - { 0x3a14, 0x03 }, > - { 0x3a15, 0xd8 }, > - { 0x3a18, 0x00 }, > - { 0x4004, 0x02 }, > - { 0x4005, 0x18 }, > - { 0x4300, 0x32 }, > - { 0x4202, 0x00 } > +static const struct cci_reg_sequence ov5645_setting_sxga[] = { > + { CCI_REG8(0x3612), 0xa9 }, > + { CCI_REG8(0x3614), 0x50 }, > + { CCI_REG8(0x3618), 0x00 }, > + { CCI_REG8(0x3034), 0x18 }, > + { CCI_REG8(0x3035), 0x21 }, > + { CCI_REG8(0x3036), 0x70 }, > + { CCI_REG8(0x3600), 0x09 }, > + { CCI_REG8(0x3601), 0x43 }, > + { CCI_REG8(0x3708), 0x66 }, > + { CCI_REG8(0x370c), 0xc3 }, > + { CCI_REG8(0x3800), 0x00 }, > + { CCI_REG8(0x3801), 0x00 }, > + { CCI_REG8(0x3802), 0x00 }, > + { CCI_REG8(0x3803), 0x06 }, > + { CCI_REG8(0x3804), 0x0a }, > + { CCI_REG8(0x3805), 0x3f }, > + { CCI_REG8(0x3806), 0x07 }, > + { CCI_REG8(0x3807), 0x9d }, > + { CCI_REG8(0x3808), 0x05 }, > + { CCI_REG8(0x3809), 0x00 }, > + { CCI_REG8(0x380a), 0x03 }, > + { CCI_REG8(0x380b), 0xc0 }, > + { CCI_REG8(0x380c), 0x07 }, > + { CCI_REG8(0x380d), 0x68 }, > + { CCI_REG8(0x380e), 0x03 }, > + { CCI_REG8(0x380f), 0xd8 }, > + { CCI_REG8(0x3813), 0x06 }, > + { CCI_REG8(0x3814), 0x31 }, > + { CCI_REG8(0x3815), 0x31 }, > + { CCI_REG8(0x3820), 0x47 }, > + { CCI_REG8(0x3a02), 0x03 }, > + { CCI_REG8(0x3a03), 0xd8 }, > + { CCI_REG8(0x3a08), 0x01 }, > + { CCI_REG8(0x3a09), 0xf8 }, > + { CCI_REG8(0x3a0a), 0x01 }, > + { CCI_REG8(0x3a0b), 0xa4 }, > + { CCI_REG8(0x3a0e), 0x02 }, > + { CCI_REG8(0x3a0d), 0x02 }, > + { CCI_REG8(0x3a14), 0x03 }, > + { CCI_REG8(0x3a15), 0xd8 }, > + { CCI_REG8(0x3a18), 0x00 }, > + { CCI_REG8(0x4004), 0x02 }, > + { CCI_REG8(0x4005), 0x18 }, > + { CCI_REG8(0x4300), 0x32 }, > + { CCI_REG8(0x4202), 0x00 } > }; > > -static const struct reg_value ov5645_setting_1080p[] = { > - { 0x3612, 0xab }, > - { 0x3614, 0x50 }, > - { 0x3618, 0x04 }, > - { 0x3034, 0x18 }, > - { 0x3035, 0x11 }, > - { 0x3036, 0x54 }, > - { 0x3600, 0x08 }, > - { 0x3601, 0x33 }, > - { 0x3708, 0x63 }, > - { 0x370c, 0xc0 }, > - { 0x3800, 0x01 }, > - { 0x3801, 0x50 }, > - { 0x3802, 0x01 }, > - { 0x3803, 0xb2 }, > - { 0x3804, 0x08 }, > - { 0x3805, 0xef }, > - { 0x3806, 0x05 }, > - { 0x3807, 0xf1 }, > - { 0x3808, 0x07 }, > - { 0x3809, 0x80 }, > - { 0x380a, 0x04 }, > - { 0x380b, 0x38 }, > - { 0x380c, 0x09 }, > - { 0x380d, 0xc4 }, > - { 0x380e, 0x04 }, > - { 0x380f, 0x60 }, > - { 0x3813, 0x04 }, > - { 0x3814, 0x11 }, > - { 0x3815, 0x11 }, > - { 0x3820, 0x47 }, > - { 0x4514, 0x88 }, > - { 0x3a02, 0x04 }, > - { 0x3a03, 0x60 }, > - { 0x3a08, 0x01 }, > - { 0x3a09, 0x50 }, > - { 0x3a0a, 0x01 }, > - { 0x3a0b, 0x18 }, > - { 0x3a0e, 0x03 }, > - { 0x3a0d, 0x04 }, > - { 0x3a14, 0x04 }, > - { 0x3a15, 0x60 }, > - { 0x3a18, 0x00 }, > - { 0x4004, 0x06 }, > - { 0x4005, 0x18 }, > - { 0x4300, 0x32 }, > - { 0x4202, 0x00 }, > - { 0x4837, 0x0b } > +static const struct cci_reg_sequence ov5645_setting_1080p[] = { > + { CCI_REG8(0x3612), 0xab }, > + { CCI_REG8(0x3614), 0x50 }, > + { CCI_REG8(0x3618), 0x04 }, > + { CCI_REG8(0x3034), 0x18 }, > + { CCI_REG8(0x3035), 0x11 }, > + { CCI_REG8(0x3036), 0x54 }, > + { CCI_REG8(0x3600), 0x08 }, > + { CCI_REG8(0x3601), 0x33 }, > + { CCI_REG8(0x3708), 0x63 }, > + { CCI_REG8(0x370c), 0xc0 }, > + { CCI_REG8(0x3800), 0x01 }, > + { CCI_REG8(0x3801), 0x50 }, > + { CCI_REG8(0x3802), 0x01 }, > + { CCI_REG8(0x3803), 0xb2 }, > + { CCI_REG8(0x3804), 0x08 }, > + { CCI_REG8(0x3805), 0xef }, > + { CCI_REG8(0x3806), 0x05 }, > + { CCI_REG8(0x3807), 0xf1 }, > + { CCI_REG8(0x3808), 0x07 }, > + { CCI_REG8(0x3809), 0x80 }, > + { CCI_REG8(0x380a), 0x04 }, > + { CCI_REG8(0x380b), 0x38 }, > + { CCI_REG8(0x380c), 0x09 }, > + { CCI_REG8(0x380d), 0xc4 }, > + { CCI_REG8(0x380e), 0x04 }, > + { CCI_REG8(0x380f), 0x60 }, > + { CCI_REG8(0x3813), 0x04 }, > + { CCI_REG8(0x3814), 0x11 }, > + { CCI_REG8(0x3815), 0x11 }, > + { CCI_REG8(0x3820), 0x47 }, > + { CCI_REG8(0x4514), 0x88 }, > + { CCI_REG8(0x3a02), 0x04 }, > + { CCI_REG8(0x3a03), 0x60 }, > + { CCI_REG8(0x3a08), 0x01 }, > + { CCI_REG8(0x3a09), 0x50 }, > + { CCI_REG8(0x3a0a), 0x01 }, > + { CCI_REG8(0x3a0b), 0x18 }, > + { CCI_REG8(0x3a0e), 0x03 }, > + { CCI_REG8(0x3a0d), 0x04 }, > + { CCI_REG8(0x3a14), 0x04 }, > + { CCI_REG8(0x3a15), 0x60 }, > + { CCI_REG8(0x3a18), 0x00 }, > + { CCI_REG8(0x4004), 0x06 }, > + { CCI_REG8(0x4005), 0x18 }, > + { CCI_REG8(0x4300), 0x32 }, > + { CCI_REG8(0x4202), 0x00 }, > + { CCI_REG8(0x4837), 0x0b } > }; > > -static const struct reg_value ov5645_setting_full[] = { > - { 0x3612, 0xab }, > - { 0x3614, 0x50 }, > - { 0x3618, 0x04 }, > - { 0x3034, 0x18 }, > - { 0x3035, 0x11 }, > - { 0x3036, 0x54 }, > - { 0x3600, 0x08 }, > - { 0x3601, 0x33 }, > - { 0x3708, 0x63 }, > - { 0x370c, 0xc0 }, > - { 0x3800, 0x00 }, > - { 0x3801, 0x00 }, > - { 0x3802, 0x00 }, > - { 0x3803, 0x00 }, > - { 0x3804, 0x0a }, > - { 0x3805, 0x3f }, > - { 0x3806, 0x07 }, > - { 0x3807, 0x9f }, > - { 0x3808, 0x0a }, > - { 0x3809, 0x20 }, > - { 0x380a, 0x07 }, > - { 0x380b, 0x98 }, > - { 0x380c, 0x0b }, > - { 0x380d, 0x1c }, > - { 0x380e, 0x07 }, > - { 0x380f, 0xb0 }, > - { 0x3813, 0x06 }, > - { 0x3814, 0x11 }, > - { 0x3815, 0x11 }, > - { 0x3820, 0x47 }, > - { 0x4514, 0x88 }, > - { 0x3a02, 0x07 }, > - { 0x3a03, 0xb0 }, > - { 0x3a08, 0x01 }, > - { 0x3a09, 0x27 }, > - { 0x3a0a, 0x00 }, > - { 0x3a0b, 0xf6 }, > - { 0x3a0e, 0x06 }, > - { 0x3a0d, 0x08 }, > - { 0x3a14, 0x07 }, > - { 0x3a15, 0xb0 }, > - { 0x3a18, 0x01 }, > - { 0x4004, 0x06 }, > - { 0x4005, 0x18 }, > - { 0x4300, 0x32 }, > - { 0x4837, 0x0b }, > - { 0x4202, 0x00 } > +static const struct cci_reg_sequence ov5645_setting_full[] = { > + { CCI_REG8(0x3612), 0xab }, > + { CCI_REG8(0x3614), 0x50 }, > + { CCI_REG8(0x3618), 0x04 }, > + { CCI_REG8(0x3034), 0x18 }, > + { CCI_REG8(0x3035), 0x11 }, > + { CCI_REG8(0x3036), 0x54 }, > + { CCI_REG8(0x3600), 0x08 }, > + { CCI_REG8(0x3601), 0x33 }, > + { CCI_REG8(0x3708), 0x63 }, > + { CCI_REG8(0x370c), 0xc0 }, > + { CCI_REG8(0x3800), 0x00 }, > + { CCI_REG8(0x3801), 0x00 }, > + { CCI_REG8(0x3802), 0x00 }, > + { CCI_REG8(0x3803), 0x00 }, > + { CCI_REG8(0x3804), 0x0a }, > + { CCI_REG8(0x3805), 0x3f }, > + { CCI_REG8(0x3806), 0x07 }, > + { CCI_REG8(0x3807), 0x9f }, > + { CCI_REG8(0x3808), 0x0a }, > + { CCI_REG8(0x3809), 0x20 }, > + { CCI_REG8(0x380a), 0x07 }, > + { CCI_REG8(0x380b), 0x98 }, > + { CCI_REG8(0x380c), 0x0b }, > + { CCI_REG8(0x380d), 0x1c }, > + { CCI_REG8(0x380e), 0x07 }, > + { CCI_REG8(0x380f), 0xb0 }, > + { CCI_REG8(0x3813), 0x06 }, > + { CCI_REG8(0x3814), 0x11 }, > + { CCI_REG8(0x3815), 0x11 }, > + { CCI_REG8(0x3820), 0x47 }, > + { CCI_REG8(0x4514), 0x88 }, > + { CCI_REG8(0x3a02), 0x07 }, > + { CCI_REG8(0x3a03), 0xb0 }, > + { CCI_REG8(0x3a08), 0x01 }, > + { CCI_REG8(0x3a09), 0x27 }, > + { CCI_REG8(0x3a0a), 0x00 }, > + { CCI_REG8(0x3a0b), 0xf6 }, > + { CCI_REG8(0x3a0e), 0x06 }, > + { CCI_REG8(0x3a0d), 0x08 }, > + { CCI_REG8(0x3a14), 0x07 }, > + { CCI_REG8(0x3a15), 0xb0 }, > + { CCI_REG8(0x3a18), 0x01 }, > + { CCI_REG8(0x4004), 0x06 }, > + { CCI_REG8(0x4005), 0x18 }, > + { CCI_REG8(0x4300), 0x32 }, > + { CCI_REG8(0x4837), 0x0b }, > + { CCI_REG8(0x4202), 0x00 } > }; > > static const s64 link_freq[] = { > @@ -537,50 +535,6 @@ static const struct ov5645_mode_info ov5645_mode_info_data[] = { > }, > }; > > -static int ov5645_write_reg(struct ov5645 *ov5645, u16 reg, u8 val) > -{ > - u8 regbuf[3]; > - int ret; > - > - regbuf[0] = reg >> 8; > - regbuf[1] = reg & 0xff; > - regbuf[2] = val; > - > - ret = i2c_master_send(ov5645->i2c_client, regbuf, 3); > - if (ret < 0) { > - dev_err(ov5645->dev, "%s: write reg error %d: reg=%x, val=%x\n", > - __func__, ret, reg, val); > - return ret; > - } > - > - return 0; > -} > - > -static int ov5645_read_reg(struct ov5645 *ov5645, u16 reg, u8 *val) > -{ > - u8 regbuf[2]; > - int ret; > - > - regbuf[0] = reg >> 8; > - regbuf[1] = reg & 0xff; > - > - ret = i2c_master_send(ov5645->i2c_client, regbuf, 2); > - if (ret < 0) { > - dev_err(ov5645->dev, "%s: write reg error %d: reg=%x\n", > - __func__, ret, reg); > - return ret; > - } > - > - ret = i2c_master_recv(ov5645->i2c_client, val, 1); > - if (ret < 0) { > - dev_err(ov5645->dev, "%s: read reg error %d: reg=%x\n", > - __func__, ret, reg); > - return ret; > - } > - > - return 0; > -} > - > static int ov5645_set_aec_mode(struct ov5645 *ov5645, u32 mode) > { > u8 val = ov5645->aec_pk_manual; > @@ -591,7 +545,7 @@ static int ov5645_set_aec_mode(struct ov5645 *ov5645, u32 mode) > else /* V4L2_EXPOSURE_MANUAL */ > val |= OV5645_AEC_MANUAL_ENABLE; > > - ret = ov5645_write_reg(ov5645, OV5645_AEC_PK_MANUAL, val); > + ret = cci_write(ov5645->regmap, OV5645_AEC_PK_MANUAL, val, NULL); > if (!ret) > ov5645->aec_pk_manual = val; > > @@ -608,7 +562,7 @@ static int ov5645_set_agc_mode(struct ov5645 *ov5645, u32 enable) > else > val |= OV5645_AGC_MANUAL_ENABLE; > > - ret = ov5645_write_reg(ov5645, OV5645_AEC_PK_MANUAL, val); > + ret = cci_write(ov5645->regmap, OV5645_AEC_PK_MANUAL, val, NULL); > if (!ret) > ov5645->aec_pk_manual = val; > > @@ -616,14 +570,14 @@ static int ov5645_set_agc_mode(struct ov5645 *ov5645, u32 enable) > } > > static int ov5645_set_register_array(struct ov5645 *ov5645, > - const struct reg_value *settings, > + const struct cci_reg_sequence *settings, > unsigned int num_settings) > { > unsigned int i; > int ret; > > for (i = 0; i < num_settings; ++i, ++settings) { > - ret = ov5645_write_reg(ov5645, settings->reg, settings->val); > + ret = cci_write(ov5645->regmap, settings->reg, settings->val, NULL); > if (ret < 0) > return ret; > > @@ -640,7 +594,7 @@ static void __ov5645_set_power_off(struct device *dev) > struct v4l2_subdev *sd = dev_get_drvdata(dev); > struct ov5645 *ov5645 = to_ov5645(sd); > > - ov5645_write_reg(ov5645, OV5645_IO_MIPI_CTRL00, 0x58); > + cci_write(ov5645->regmap, OV5645_IO_MIPI_CTRL00, 0x58, NULL); > gpiod_set_value_cansleep(ov5645->rst_gpio, 1); > gpiod_set_value_cansleep(ov5645->enable_gpio, 0); > regulator_bulk_disable(OV5645_NUM_SUPPLIES, ov5645->supplies); > @@ -704,11 +658,11 @@ static int ov5645_set_saturation(struct ov5645 *ov5645, s32 value) > u32 reg_value = (value * 0x10) + 0x40; > int ret; > > - ret = ov5645_write_reg(ov5645, OV5645_SDE_SAT_U, reg_value); > + ret = cci_write(ov5645->regmap, OV5645_SDE_SAT_U, reg_value, NULL); > if (ret < 0) > return ret; > > - return ov5645_write_reg(ov5645, OV5645_SDE_SAT_V, reg_value); > + return cci_write(ov5645->regmap, OV5645_SDE_SAT_V, reg_value, NULL); > } > > static int ov5645_set_hflip(struct ov5645 *ov5645, s32 value) > @@ -721,7 +675,7 @@ static int ov5645_set_hflip(struct ov5645 *ov5645, s32 value) > else > val |= (OV5645_SENSOR_MIRROR); > > - ret = ov5645_write_reg(ov5645, OV5645_TIMING_TC_REG21, val); > + ret = cci_write(ov5645->regmap, OV5645_TIMING_TC_REG21, val, NULL); > if (!ret) > ov5645->timing_tc_reg21 = val; > > @@ -738,7 +692,7 @@ static int ov5645_set_vflip(struct ov5645 *ov5645, s32 value) > else > val &= ~(OV5645_SENSOR_VFLIP | OV5645_ISP_VFLIP); > > - ret = ov5645_write_reg(ov5645, OV5645_TIMING_TC_REG20, val); > + ret = cci_write(ov5645->regmap, OV5645_TIMING_TC_REG20, val, NULL); > if (!ret) > ov5645->timing_tc_reg20 = val; > > @@ -754,7 +708,7 @@ static int ov5645_set_test_pattern(struct ov5645 *ov5645, s32 value) > val |= OV5645_TEST_PATTERN_ENABLE; > } > > - return ov5645_write_reg(ov5645, OV5645_PRE_ISP_TEST_SETTING_1, val); > + return cci_write(ov5645->regmap, OV5645_PRE_ISP_TEST_SETTING_1, val, NULL); > } > > static const char * const ov5645_test_pattern_menu[] = { > @@ -772,7 +726,7 @@ static int ov5645_set_awb(struct ov5645 *ov5645, s32 enable_auto) > if (!enable_auto) > val = OV5645_AWB_MANUAL_ENABLE; > > - return ov5645_write_reg(ov5645, OV5645_AWB_MANUAL_CONTROL, val); > + return cci_write(ov5645->regmap, OV5645_AWB_MANUAL_CONTROL, val, NULL); > } > > static int ov5645_s_ctrl(struct v4l2_ctrl *ctrl) > @@ -958,9 +912,8 @@ static int ov5645_enable_streams(struct v4l2_subdev *sd, > if (ret < 0) > return ret; > > - ret = ov5645_set_register_array(ov5645, > - ov5645->current_mode->data, > - ov5645->current_mode->data_size); > + ret = cci_multi_reg_write(ov5645->regmap, ov5645->current_mode->data, > + ov5645->current_mode->data_size, NULL); > if (ret < 0) { > dev_err(ov5645->dev, "could not set mode %dx%d\n", > ov5645->current_mode->width, > @@ -973,12 +926,12 @@ static int ov5645_enable_streams(struct v4l2_subdev *sd, > goto err_rpm_put; > } > > - ret = ov5645_write_reg(ov5645, OV5645_IO_MIPI_CTRL00, 0x45); > + ret = cci_write(ov5645->regmap, OV5645_IO_MIPI_CTRL00, 0x45, NULL); > if (ret < 0) > goto err_rpm_put; > > - ret = ov5645_write_reg(ov5645, OV5645_SYSTEM_CTRL0, > - OV5645_SYSTEM_CTRL0_START); > + ret = cci_write(ov5645->regmap, OV5645_SYSTEM_CTRL0, > + OV5645_SYSTEM_CTRL0_START, NULL); > if (ret < 0) > goto err_rpm_put; > > @@ -996,12 +949,12 @@ static int ov5645_disable_streams(struct v4l2_subdev *sd, > struct ov5645 *ov5645 = to_ov5645(sd); > int ret; > > - ret = ov5645_write_reg(ov5645, OV5645_IO_MIPI_CTRL00, 0x40); > + ret = cci_write(ov5645->regmap, OV5645_IO_MIPI_CTRL00, 0x40, NULL); > if (ret < 0) > goto rpm_put; > > - ret = ov5645_write_reg(ov5645, OV5645_SYSTEM_CTRL0, > - OV5645_SYSTEM_CTRL0_STOP); > + ret = cci_write(ov5645->regmap, OV5645_SYSTEM_CTRL0, > + OV5645_SYSTEM_CTRL0_STOP, NULL); > > rpm_put: > pm_runtime_put_autosuspend(ov5645->dev); > @@ -1036,9 +989,9 @@ static const struct v4l2_subdev_internal_ops ov5645_internal_ops = { > static int ov5645_probe(struct i2c_client *client) > { > struct device *dev = &client->dev; > + u64 chip_id_high, chip_id_low; > struct device_node *endpoint; > struct ov5645 *ov5645; > - u8 chip_id_high, chip_id_low; > unsigned int i; > u32 xclk_freq; > int ret; > @@ -1068,6 +1021,11 @@ static int ov5645_probe(struct i2c_client *client) > return dev_err_probe(dev, -EINVAL, > "invalid bus type, must be CSI2\n"); > > + ov5645->regmap = devm_cci_regmap_init_i2c(client, 16); > + if (IS_ERR(ov5645->regmap)) > + return dev_err_probe(ov5645->dev, PTR_ERR(ov5645->regmap), > + "Failed to init CCI\n"); > + > /* get system clock (xclk) */ > ov5645->xclk = devm_v4l2_sensor_clk_get_legacy(dev, NULL, false, 0); > if (IS_ERR(ov5645->xclk)) > @@ -1154,13 +1112,13 @@ static int ov5645_probe(struct i2c_client *client) > if (ret) > goto free_entity; > > - ret = ov5645_read_reg(ov5645, OV5645_CHIP_ID_HIGH, &chip_id_high); > + ret = cci_read(ov5645->regmap, OV5645_CHIP_ID_HIGH, &chip_id_high, NULL); > if (ret < 0 || chip_id_high != OV5645_CHIP_ID_HIGH_BYTE) { > ret = -ENODEV; > dev_err_probe(dev, ret, "could not read ID high\n"); > goto power_down; > } > - ret = ov5645_read_reg(ov5645, OV5645_CHIP_ID_LOW, &chip_id_low); > + ret = cci_read(ov5645->regmap, OV5645_CHIP_ID_LOW, &chip_id_low, NULL); > if (ret < 0 || chip_id_low != OV5645_CHIP_ID_LOW_BYTE) { > ret = -ENODEV; > dev_err_probe(dev, ret, "could not read ID low\n"); > @@ -1169,24 +1127,21 @@ static int ov5645_probe(struct i2c_client *client) > > dev_info(dev, "OV5645 detected at address 0x%02x\n", client->addr); > > - ret = ov5645_read_reg(ov5645, OV5645_AEC_PK_MANUAL, > - &ov5645->aec_pk_manual); > + ret = cci_read(ov5645->regmap, OV5645_AEC_PK_MANUAL, &ov5645->aec_pk_manual, NULL); > if (ret < 0) { > ret = -ENODEV; > dev_err_probe(dev, ret, "could not read AEC/AGC mode\n"); > goto power_down; > } > > - ret = ov5645_read_reg(ov5645, OV5645_TIMING_TC_REG20, > - &ov5645->timing_tc_reg20); > + ret = cci_read(ov5645->regmap, OV5645_TIMING_TC_REG20, &ov5645->timing_tc_reg20, NULL); > if (ret < 0) { > ret = -ENODEV; > dev_err_probe(dev, ret, "could not read vflip value\n"); > goto power_down; > } > > - ret = ov5645_read_reg(ov5645, OV5645_TIMING_TC_REG21, > - &ov5645->timing_tc_reg21); > + ret = cci_read(ov5645->regmap, OV5645_TIMING_TC_REG21, &ov5645->timing_tc_reg21, NULL); > if (ret < 0) { > ret = -ENODEV; > dev_err_probe(dev, ret, "could not read hflip value\n"); > -- > 2.53.0 > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-03-26 15:00 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-03-26 14:21 [PATCH 0/2] media: ov5645: Implement frame descriptor reporting and convert to CCI register access helpers Prabhakar 2026-03-26 14:21 ` [PATCH 1/2] media: i2c: ov5645: Report streams using frame descriptors Prabhakar 2026-03-26 14:21 ` [PATCH 2/2] media: i2c: ov5645: Convert to CCI register access helpers Prabhakar 2026-03-26 14:59 ` xiaolei wang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox