media: i2c: imx290: Configure data lanes at start time

There's no need to configure the data lanes in the runtime PM resume
handler. Do so in imx290_start_streaming() instead.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>

diff --git a/drivers/media/i2c/imx290.c b/drivers/media/i2c/imx290.c
index 4185835f065d9beaf3bcea0f8858ca19b06f3b09..34278d098218e17c967640f68a029b8f8cad35a3 100644 (file)
--- a/drivers/media/i2c/imx290.c
+++ b/drivers/media/i2c/imx290.c
@@ -762,6 +762,9 @@ static int imx290_start_streaming(struct imx290 *imx290,
                return ret;
        }
 
+       /* Set data lane count */
+       imx290_set_data_lanes(imx290);
+
        /* Apply the register values related to current frame format */
        format = v4l2_subdev_get_pad_format(&imx290->sd, state, 0);
        ret = imx290_setup_format(imx290, format);
@@ -1069,9 +1072,6 @@ static int imx290_power_on(struct imx290 *imx290)
        gpiod_set_value_cansleep(imx290->rst_gpio, 0);
        usleep_range(30000, 31000);
 
-       /* Set data lane count */
-       imx290_set_data_lanes(imx290);
-
        return 0;
 }