Commit 7d399658 authored by Laurent Pinchart's avatar Laurent Pinchart Committed by Mauro Carvalho Chehab

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: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: default avatarAlexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@kernel.org>
parent 02852c01
...@@ -762,6 +762,9 @@ static int imx290_start_streaming(struct imx290 *imx290, ...@@ -762,6 +762,9 @@ static int imx290_start_streaming(struct imx290 *imx290,
return ret; return ret;
} }
/* Set data lane count */
imx290_set_data_lanes(imx290);
/* Apply the register values related to current frame format */ /* Apply the register values related to current frame format */
format = v4l2_subdev_get_pad_format(&imx290->sd, state, 0); format = v4l2_subdev_get_pad_format(&imx290->sd, state, 0);
ret = imx290_setup_format(imx290, format); ret = imx290_setup_format(imx290, format);
...@@ -1069,9 +1072,6 @@ static int imx290_power_on(struct imx290 *imx290) ...@@ -1069,9 +1072,6 @@ static int imx290_power_on(struct imx290 *imx290)
gpiod_set_value_cansleep(imx290->rst_gpio, 0); gpiod_set_value_cansleep(imx290->rst_gpio, 0);
usleep_range(30000, 31000); usleep_range(30000, 31000);
/* Set data lane count */
imx290_set_data_lanes(imx290);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment