github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-06-03 12:03:54 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

media: i2c: ov01a10: Remove struct ov01a10_reg_list

Browse Source 
After the conversion to the CCI register access helpers, struct
ov01a10_reg_list is only used inside struct ov01a10_link_freq_config.

Simplify things by embedding the ov01a10_reg_list members directly into
struct ov01a10_link_freq_config.

Signed-off-by: Hans de Goede <hansg@kernel.org>
Tested-by: Mehdi Djait <mehdi.djait@linux.intel.com> # Dell XPS 9315
Reviewed-by: Mehdi Djait <mehdi.djait@linux.intel.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>
This commit is contained in:
Hans de Goede 2025-10-14 19:40:25 +02:00 committed by Hans Verkuil
parent 896f4e21af
commit d4fdb4e6e0
1 changed files with 8 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
drivers/media/i2c/ov01a10.c
View File

@ -98,13 +98,9 @@
#define OV01A10_MEDIA_BUS_FMT MEDIA_BUS_FMT_SBGGR10_1X10
#define OV01A10_BAYER_PATTERN_SIZE 2 /* 2x2 */
struct ov01a10_reg_list {
u32 num_of_regs;
const struct reg_sequence *regs;
};
struct ov01a10_link_freq_config {
const struct ov01a10_reg_list reg_list;
const struct reg_sequence *regs;
int regs_len;
};
static const struct reg_sequence mipi_data_rate_720mbps[] = {
@ -237,10 +233,8 @@ static const s64 link_freq_menu_items[] = {
static const struct ov01a10_link_freq_config link_freq_configs[] = {
{
.reg_list = {
.num_of_regs = ARRAY_SIZE(mipi_data_rate_720mbps),
.regs = mipi_data_rate_720mbps,
}
.regs = mipi_data_rate_720mbps,
.regs_len = ARRAY_SIZE(mipi_data_rate_720mbps),
},
};
@ -550,12 +544,12 @@ static int ov01a10_set_mode(struct ov01a10 *ov01a10)
static int ov01a10_start_streaming(struct ov01a10 *ov01a10)
{
const struct ov01a10_reg_list *reg_list;
const struct ov01a10_link_freq_config *freq_cfg;
int ret;
reg_list = &link_freq_configs[ov01a10->link_freq_index].reg_list;
ret = regmap_multi_reg_write(ov01a10->regmap, reg_list->regs,
reg_list->num_of_regs);
freq_cfg = &link_freq_configs[ov01a10->link_freq_index];
ret = regmap_multi_reg_write(ov01a10->regmap, freq_cfg->regs,
freq_cfg->regs_len);
if (ret) {
dev_err(ov01a10->dev, "failed to set plls\n");
return ret;