]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
media: i2c: ov01a10: Remove struct ov01a10_reg_list
authorHans de Goede <hansg@kernel.org>
Tue, 14 Oct 2025 17:40:25 +0000 (19:40 +0200)
committerHans Verkuil <hverkuil+cisco@kernel.org>
Tue, 13 Jan 2026 11:28:49 +0000 (12:28 +0100)
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>
drivers/media/i2c/ov01a10.c

index 349fd3d06df5cfd1eff230be7037ddd52b6d80b7..1af0af00df24c30c7d68e2c588c22493c37c5ab7 100644 (file)
 #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;