On Wed, Mar 18, 2020 at 06:07:48PM +0800, Tang Bin wrote: > We don't need dev_err() message because when something goes wrong, > platform_get_irq() has print an error message itself, so we should > remove duplicate dev_err(). > > Signed-off-by: Tang Bin <tangbin@xxxxxxxxxxxxxxxxxxxx> > --- > drivers/i2c/busses/i2c-imx.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-imx.c b/drivers/i2c/busses/i2c-imx.c > index a3b61336f..01fd46682 100644 > --- a/drivers/i2c/busses/i2c-imx.c > +++ b/drivers/i2c/busses/i2c-imx.c > @@ -1066,10 +1066,8 @@ static int i2c_imx_probe(struct platform_device *pdev) > dev_dbg(&pdev->dev, "<%s>\n", __func__); > > irq = platform_get_irq(pdev, 0); > - if (irq < 0) { > - dev_err(&pdev->dev, "can't get irq number\n"); > + if (irq < 0) > return irq; > - } Maybe add a comment for the next person who wonders about an error path without error message? Something like: irq = platform_get_irq(pdev, 0); if (irq < 0) /* * platform_get_irq() already issued an error message, so * fail silently here. */ return irq; And to get some extra kudos mention the commit that modified platform_get_irq() to emit a message in the commit log. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |