media: ov9734: use group write for digital gain

As the RGB digital gains of ov9734 were not applied as group, some
artifacts were observed in low light environment, use group write for
digital gain can make the RGB digital can be guaranteed to applied
together at frame boundary.

Signed-off-by: Bingbu Cao <bingbu.cao@intel.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>

diff --git a/drivers/media/i2c/ov9734.c b/drivers/media/i2c/ov9734.c
index af50c66cf5ce2082c318aae1b6e7ea8b6cb01818..df538ceb71c36c089c521a007283bec388131a2e 100644 (file)
--- a/drivers/media/i2c/ov9734.c
+++ b/drivers/media/i2c/ov9734.c
@@ -60,6 +60,12 @@
 #define OV9734_TEST_PATTERN_ENABLE     BIT(7)
 #define OV9734_TEST_PATTERN_BAR_SHIFT  2
 
+/* Group Access */
+#define OV9734_REG_GROUP_ACCESS                0x3208
+#define OV9734_GROUP_HOLD_START                0x0
+#define OV9734_GROUP_HOLD_END          0x10
+#define OV9734_GROUP_HOLD_LAUNCH       0xa0
+
 enum {
        OV9734_LINK_FREQ_180MHZ_INDEX,
 };
@@ -433,6 +439,11 @@ static int ov9734_update_digital_gain(struct ov9734 *ov9734, u32 d_gain)
 {
        int ret;
 
+       ret = ov9734_write_reg(ov9734, OV9734_REG_GROUP_ACCESS, 1,
+                              OV9734_GROUP_HOLD_START);
+       if (ret)
+               return ret;
+
        ret = ov9734_write_reg(ov9734, OV9734_REG_MWB_R_GAIN, 2, d_gain);
        if (ret)
                return ret;
@@ -441,7 +452,18 @@ static int ov9734_update_digital_gain(struct ov9734 *ov9734, u32 d_gain)
        if (ret)
                return ret;
 
-       return ov9734_write_reg(ov9734, OV9734_REG_MWB_B_GAIN, 2, d_gain);
+       ret = ov9734_write_reg(ov9734, OV9734_REG_MWB_B_GAIN, 2, d_gain);
+       if (ret)
+               return ret;
+
+       ret = ov9734_write_reg(ov9734, OV9734_REG_GROUP_ACCESS, 1,
+                              OV9734_GROUP_HOLD_END);
+       if (ret)
+               return ret;
+
+       ret = ov9734_write_reg(ov9734, OV9734_REG_GROUP_ACCESS, 1,
+                              OV9734_GROUP_HOLD_LAUNCH);
+       return ret;
 }
 
 static int ov9734_test_pattern(struct ov9734 *ov9734, u32 pattern)