Hi Laurent,
thanks for the update.
Am Samstag, 14. Januar 2023, 18:18:02 CET schrieb Laurent Pinchart:
> Check the error status returned by imx290_set_data_lanes() in its
> caller and propagate it.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
Reviewed-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx>
> ---
> Changes since v1:
>
> - New patch
> ---
> drivers/media/i2c/imx290.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/media/i2c/imx290.c b/drivers/media/i2c/imx290.c
> index a8167119534b..62d8d75aec3e 100644
> --- a/drivers/media/i2c/imx290.c
> +++ b/drivers/media/i2c/imx290.c
> @@ -761,7 +761,11 @@ static int imx290_start_streaming(struct imx290
> *imx290, }
>
> /* Set data lane count */
> - imx290_set_data_lanes(imx290);
> + ret = imx290_set_data_lanes(imx290);
> + if (ret < 0) {
> + dev_err(imx290->dev, "Could not set data lanes\n");
> + return ret;
> + }
>
> /* Apply the register values related to current frame format */
> format = v4l2_subdev_get_pad_format(&imx290->sd, state, 0);
