This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: ccs: Better separate CCS static data access
Author:  Sakari Ailus <sakari.ai...@linux.intel.com>
Date:    Tue Nov 7 11:37:21 2023 +0200

Separate CCS static data read-only register access in ccs-reg-access.c by
naming them differently.

The code in this file generally deals with reading and writing registers
where as static data (when it comes to ccs_static_data_read_ro_reg())
contains the read-only register values but no hardware registers are
accessed in that case.

Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/i2c/ccs/ccs-reg-access.c | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)

---

diff --git a/drivers/media/i2c/ccs/ccs-reg-access.c 
b/drivers/media/i2c/ccs/ccs-reg-access.c
index 25993445f4fe..03daaa8310a6 100644
--- a/drivers/media/i2c/ccs/ccs-reg-access.c
+++ b/drivers/media/i2c/ccs/ccs-reg-access.c
@@ -197,8 +197,8 @@ static int __ccs_read_addr(struct ccs_sensor *sensor, u32 
reg, u32 *val,
        return 0;
 }
 
-static int __ccs_read_data(struct ccs_reg *regs, size_t num_regs,
-                          u32 reg, u32 *val)
+static int __ccs_static_data_read_ro_reg(struct ccs_reg *regs, size_t num_regs,
+                                        u32 reg, u32 *val)
 {
        unsigned int width = ccs_reg_width(reg);
        size_t i;
@@ -235,16 +235,17 @@ static int __ccs_read_data(struct ccs_reg *regs, size_t 
num_regs,
        return -ENOENT;
 }
 
-static int ccs_read_data(struct ccs_sensor *sensor, u32 reg, u32 *val)
+static int
+ccs_static_data_read_ro_reg(struct ccs_sensor *sensor, u32 reg, u32 *val)
 {
-       if (!__ccs_read_data(sensor->sdata.sensor_read_only_regs,
-                            sensor->sdata.num_sensor_read_only_regs,
-                            reg, val))
+       if (!__ccs_static_data_read_ro_reg(sensor->sdata.sensor_read_only_regs,
+                                          
sensor->sdata.num_sensor_read_only_regs,
+                                          reg, val))
                return 0;
 
-       return __ccs_read_data(sensor->mdata.module_read_only_regs,
-                              sensor->mdata.num_module_read_only_regs,
-                              reg, val);
+       return 
__ccs_static_data_read_ro_reg(sensor->mdata.module_read_only_regs,
+                                            
sensor->mdata.num_module_read_only_regs,
+                                            reg, val);
 }
 
 static int ccs_read_addr_raw(struct ccs_sensor *sensor, u32 reg, u32 *val,
@@ -253,7 +254,7 @@ static int ccs_read_addr_raw(struct ccs_sensor *sensor, u32 
reg, u32 *val,
        int rval;
 
        if (data) {
-               rval = ccs_read_data(sensor, reg, val);
+               rval = ccs_static_data_read_ro_reg(sensor, reg, val);
                if (!rval)
                        return 0;
        }

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to