media: staging: rkisp1: replace 3 fields 'ct_offset_*' with one array

The struct rkisp1_cif_isp_ctk_config contains 3 fields
ct_offset_{rgb}. Replace them with one array field 'ct_offset[3].

Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com>
Acked-by: Helen Koike <helen.koike@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>

diff --git a/drivers/staging/media/rkisp1/rkisp1-params.c b/drivers/staging/media/rkisp1/rkisp1-params.c
index c41109fc70700232e15fdf449f791d9da6457751..6140e6c41fc6d629b0054c2fad6ab94246170662 100644 (file)
--- a/drivers/staging/media/rkisp1/rkisp1-params.c
+++ b/drivers/staging/media/rkisp1/rkisp1-params.c
@@ -408,12 +408,9 @@ static void rkisp1_ctk_config(struct rkisp1_params *params,
                for (j = 0; j < 3; j++)
                        rkisp1_write(params->rkisp1, arg->coeff[i][j],
                                     RKISP1_CIF_ISP_CT_COEFF_0 + 4 * k++);
-       rkisp1_write(params->rkisp1, arg->ct_offset_r,
-                    RKISP1_CIF_ISP_CT_OFFSET_R);
-       rkisp1_write(params->rkisp1, arg->ct_offset_g,
-                    RKISP1_CIF_ISP_CT_OFFSET_G);
-       rkisp1_write(params->rkisp1, arg->ct_offset_b,
-                    RKISP1_CIF_ISP_CT_OFFSET_B);
+       for (i = 0; i < 3; i++)
+               rkisp1_write(params->rkisp1, arg->ct_offset[i],
+                            RKISP1_CIF_ISP_CT_OFFSET_R + i * 4);
 }
 
 static void rkisp1_ctk_enable(struct rkisp1_params *params, bool en)

diff --git a/drivers/staging/media/rkisp1/uapi/rkisp1-config.h b/drivers/staging/media/rkisp1/uapi/rkisp1-config.h
index 790d56ac931009e999f21e31ac97ce953e5bd137..e89ec359b537bc1deaf0b3c9714ed1d2187d112f 100644 (file)
--- a/drivers/staging/media/rkisp1/uapi/rkisp1-config.h
+++ b/drivers/staging/media/rkisp1/uapi/rkisp1-config.h
@@ -420,13 +420,11 @@ struct rkisp1_cif_isp_bdm_config {
  * @coeff: color correction matrix. Values are 11-bit signed fixed-point numbers with 4 bit integer
  *             and 7 bit fractional part, ranging from -8 (0x400) to +7.992 (0x3FF). 0 is
  *             represented by 0x000 and a coefficient value of 1 as 0x080.
- * @ct_offset_b: offset for the crosstalk correction matrix
+ * @ct_offset: Red, Green, Blue offsets for the crosstalk correction matrix
  */
 struct rkisp1_cif_isp_ctk_config {
        __u16 coeff[3][3];
-       __u16 ct_offset_r;
-       __u16 ct_offset_g;
-       __u16 ct_offset_b;
+       __u16 ct_offset[3];
 } __packed;
 
 enum rkisp1_cif_isp_goc_mode {