diff options
| author | Dejin Zheng <zhengdejin5@gmail.com> | 2020-04-16 23:23:45 +0800 |
|---|---|---|
| committer | Wolfram Sang <wsa@the-dreams.de> | 2020-04-18 23:42:14 +0200 |
| commit | e42688ed5cf5936fb55c78cc365dbe0944af7c63 (patch) | |
| tree | 7494859c73c0ec0b849ed47ae615860c882b2c66 /drivers/i2c/busses/i2c-xlp9xx.c | |
| parent | e517526195de400158e05a08764d1fb61d579105 (diff) | |
i2c: busses: remove duplicate dev_err()
it will print an error message by itself when platform_get_irq()
goes wrong. so don't need dev_err() in here again.
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Suggested-by: Markus Elfring <Markus.Elfring@web.de>
Signed-off-by: Dejin Zheng <zhengdejin5@gmail.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Diffstat (limited to 'drivers/i2c/busses/i2c-xlp9xx.c')
| -rw-r--r-- | drivers/i2c/busses/i2c-xlp9xx.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/i2c/busses/i2c-xlp9xx.c b/drivers/i2c/busses/i2c-xlp9xx.c index 286faa9c855e..f2241cedf5d3 100644 --- a/drivers/i2c/busses/i2c-xlp9xx.c +++ b/drivers/i2c/busses/i2c-xlp9xx.c @@ -517,10 +517,8 @@ static int xlp9xx_i2c_probe(struct platform_device *pdev) return PTR_ERR(priv->base); priv->irq = platform_get_irq(pdev, 0); - if (priv->irq <= 0) { - dev_err(&pdev->dev, "invalid irq!\n"); + if (priv->irq <= 0) return priv->irq; - } /* SMBAlert irq */ priv->alert_data.irq = platform_get_irq(pdev, 1); if (priv->alert_data.irq <= 0) |