[PATCH v2 3/9] media: i2c: imx334: Remove redundant register entries

From: Tarang Raval
Date: Sat Mar 29 2025 - 01:46:08 EST


IMX334_REG_{ADBIT, MDBIT, ADBIT1}: Already written in
imx334_set_framefmt function.

IMX334_REG_THSPREPARE: Unnecessary repeated writes removed.

CCI_REG8(0x3e04): Unnecessary repeated writes removed.

Signed-off-by: Tarang Raval <tarang.raval@xxxxxxxxxxxxxxxxx>
---
drivers/media/i2c/imx334.c | 8 --------
1 file changed, 8 deletions(-)

diff --git a/drivers/media/i2c/imx334.c b/drivers/media/i2c/imx334.c
index 0785bf213d91..9d4d15df8dcf 100644
--- a/drivers/media/i2c/imx334.c
+++ b/drivers/media/i2c/imx334.c
@@ -226,7 +226,6 @@ static const struct cci_reg_sequence common_mode_regs[] = {
{ IMX334_REG_OPB_SIZE_V, 0x00 },
{ IMX334_REG_HREVERSE, 0x00 },
{ IMX334_REG_VREVERSE, 0x00 },
- { IMX334_REG_ADBIT, 0x00 },
{ IMX334_REG_UNREAD_PARAM5, 0x0000 },
{ IMX334_REG_UNREAD_PARAM6, 0x0008 },
{ IMX334_REG_XVS_XHS_OUTSEL, 0x20 },
@@ -239,19 +238,13 @@ static const struct cci_reg_sequence common_mode_regs[] = {
{ IMX334_REG_INCKSEL4, 0x7e },
{ IMX334_REG_SYS_MODE, 0x02 },
{ IMX334_REG_HADD_VADD, 0x00 },
- { IMX334_REG_MDBIT, 0x00 },
{ IMX334_REG_VALID_EXPAND, 0x03 },
{ IMX334_REG_TCYCLE, 0x00 },
- { IMX334_REG_ADBIT1, 0x01ff },
{ IMX334_REG_LANEMODE, 0x03 },
{ IMX334_REG_TCLKPOST, 0x007f },
{ IMX334_REG_TCLKPREPARE, 0x0037 },
{ IMX334_REG_TCLKTRAIL, 0x0037 },
{ IMX334_REG_TCLKZERO, 0xf7 },
- { IMX334_REG_THSPREPARE, 0x003f },
- { IMX334_REG_THSPREPARE, 0x006f },
- { IMX334_REG_THSPREPARE, 0x003f },
- { IMX334_REG_THSPREPARE, 0x005f },
{ IMX334_REG_THSPREPARE, 0x002f },
{ CCI_REG8(0x3078), 0x02 },
{ CCI_REG8(0x3079), 0x00 },
@@ -339,7 +332,6 @@ static const struct cci_reg_sequence common_mode_regs[] = {
{ CCI_REG8(0x37b0), 0x37 },
{ CCI_REG8(0x3e04), 0x0e },
{ IMX334_REG_AGAIN, 0x0050 },
- { CCI_REG8(0x3e04), 0x0e },
{ IMX334_REG_MASTER_MODE, 0x00 },
};

--
2.34.1