diff mbox series

[1/1] media: i2c: imx290: Add missing \n on dev_err_probe() message

Message ID 20230406095551.952091-1-alexander.stein@ew.tq-group.com
State Accepted
Commit d56f39f705080f4ff96b92f054b90c6f0826fd02
Headers show
Series [1/1] media: i2c: imx290: Add missing \n on dev_err_probe() message | expand

Commit Message

Alexander Stein April 6, 2023, 9:55 a.m. UTC
Also dev_err_probe message require a trailing \n.
Fixes: 63127235bebd ("media: i2c: imx290: Use dev_err_probe()")

Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
 drivers/media/i2c/imx290.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Laurent Pinchart April 7, 2023, 1:44 a.m. UTC | #1
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@ew.tq-group.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  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)
Sakari Ailus April 7, 2023, 6:35 p.m. UTC | #2
On Fri, Apr 07, 2023 at 04:44:07AM +0300, Laurent Pinchart wrote:
> 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.

I dropped it. This is not a grave problem.

> 
> > Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> 
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Thanks!
diff mbox series

Patch

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)