Hi Alexander,
Thank you for the patch.
On Thu, Apr 06, 2023 at 11:55:51AM +0200, Alexander Stein wrote:
> Also dev_err_probe message require a trailing \n.
There should be a blank line here.
> Fixes: 63127235bebd ("media: i2c: imx290: Use dev_err_probe()")
>
And none here.
But the bug wasn't introduced in the above commit, the \n was missing
before that, right from the start when the driver was merged in commit
828dbc299278 ("media: i2c: Add IMX290 CMOS image sensor driver"). I
would use that commit in the Fixes tag, or drop the Fixes tag
altogether.
> Signed-off-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx>
Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> ---
> drivers/media/i2c/imx290.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/i2c/imx290.c b/drivers/media/i2c/imx290.c
> index 94bba482af38..5ea25b7acc55 100644
> --- a/drivers/media/i2c/imx290.c
> +++ b/drivers/media/i2c/imx290.c
> @@ -1602,7 +1602,7 @@ static int imx290_probe(struct i2c_client *client)
> imx290->xclk = devm_clk_get(dev, "xclk");
> if (IS_ERR(imx290->xclk))
> return dev_err_probe(dev, PTR_ERR(imx290->xclk),
> - "Could not get xclk");
> + "Could not get xclk\n");
>
> ret = imx290_get_regulators(dev, imx290);
> if (ret < 0)
--
Regards,
Laurent Pinchart
