]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
media: i2c: imx290: Move registers with fixed value to init array
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Sun, 16 Oct 2022 06:15:20 +0000 (09:15 +0300)
committerSakari Ailus <sakari.ailus@linux.intel.com>
Thu, 27 Oct 2022 11:38:02 +0000 (14:38 +0300)
Registers 0x3012, 0x3013 and 0x3480 are not documented and are set in
the per-mode register arrays with values indentical for all modes. Move
them to the common array.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
drivers/media/i2c/imx290.c

index d2dfda8de6a88adda16307465f830f6e8817d7fa..ba46d321a83a5314d1fb635fabb4dcdf333534f8 100644 (file)
@@ -192,6 +192,7 @@ static const struct imx290_regval imx290_global_init_settings[] = {
        { IMX290_REG_8BIT(0x300f), 0x00 },
        { IMX290_REG_8BIT(0x3010), 0x21 },
        { IMX290_REG_8BIT(0x3012), 0x64 },
+       { IMX290_REG_8BIT(0x3013), 0x00 },
        { IMX290_REG_8BIT(0x3016), 0x09 },
        { IMX290_REG_8BIT(0x3070), 0x02 },
        { IMX290_REG_8BIT(0x3071), 0x11 },
@@ -230,6 +231,7 @@ static const struct imx290_regval imx290_global_init_settings[] = {
        { IMX290_REG_8BIT(0x33b0), 0x50 },
        { IMX290_REG_8BIT(0x33b2), 0x1a },
        { IMX290_REG_8BIT(0x33b3), 0x04 },
+       { IMX290_REG_8BIT(0x3480), 0x49 },
 };
 
 static const struct imx290_regval imx290_1080p_settings[] = {
@@ -239,15 +241,12 @@ static const struct imx290_regval imx290_1080p_settings[] = {
        { IMX290_OPB_SIZE_V, 10 },
        { IMX290_X_OUT_SIZE, 1920 },
        { IMX290_Y_OUT_SIZE, 1080 },
-       { IMX290_REG_8BIT(0x3012), 0x64 },
-       { IMX290_REG_8BIT(0x3013), 0x00 },
        { IMX290_INCKSEL1, 0x18 },
        { IMX290_INCKSEL2, 0x03 },
        { IMX290_INCKSEL3, 0x20 },
        { IMX290_INCKSEL4, 0x01 },
        { IMX290_INCKSEL5, 0x1a },
        { IMX290_INCKSEL6, 0x1a },
-       { IMX290_REG_8BIT(0x3480), 0x49 },
        /* data rate settings */
        { IMX290_REPETITION, 0x10 },
        { IMX290_TCLKPOST, 87 },
@@ -267,15 +266,12 @@ static const struct imx290_regval imx290_720p_settings[] = {
        { IMX290_OPB_SIZE_V, 4 },
        { IMX290_X_OUT_SIZE, 1280 },
        { IMX290_Y_OUT_SIZE, 720 },
-       { IMX290_REG_8BIT(0x3012), 0x64 },
-       { IMX290_REG_8BIT(0x3013), 0x00 },
        { IMX290_INCKSEL1, 0x20 },
        { IMX290_INCKSEL2, 0x00 },
        { IMX290_INCKSEL3, 0x20 },
        { IMX290_INCKSEL4, 0x01 },
        { IMX290_INCKSEL5, 0x1a },
        { IMX290_INCKSEL6, 0x1a },
-       { IMX290_REG_8BIT(0x3480), 0x49 },
        /* data rate settings */
        { IMX290_REPETITION, 0x10 },
        { IMX290_TCLKPOST, 79 },