Message ID | 20230728083137.3528303-1-chenjiahao16@huawei.com |
---|---|
State | Superseded |
Delegated to: | Andi Shyti |
Headers | show |
Series | [-next] i2c: microchip-corei2c: clean up redundant dev_err_probe() | expand |
On Fri, Jul 28, 2023 at 04:31:37PM +0800, Chen Jiahao wrote: > Referring to platform_get_irq()'s definition, the return value has > already been checked, error message also been printed via > dev_err_probe() if ret < 0. Calling dev_err_probe() one more time > outside platform_get_irq() is obviously redundant. > > Furthermore, platform_get_irq() will never return irq equals 0, > removing spi->irq == 0 checking to clean it up. > > Signed-off-by: Chen Jiahao <chenjiahao16@huawei.com> Reviewed-by: Conor Dooley <conor.dooley@microchip.com> Thanks, Conor.
diff --git a/drivers/i2c/busses/i2c-microchip-corei2c.c b/drivers/i2c/busses/i2c-microchip-corei2c.c index 7f58f7eaabb6..0b0a1c4d17ca 100644 --- a/drivers/i2c/busses/i2c-microchip-corei2c.c +++ b/drivers/i2c/busses/i2c-microchip-corei2c.c @@ -378,9 +378,8 @@ static int mchp_corei2c_probe(struct platform_device *pdev) return PTR_ERR(idev->base); irq = platform_get_irq(pdev, 0); - if (irq <= 0) - return dev_err_probe(&pdev->dev, -ENXIO, - "invalid IRQ %d for I2C controller\n", irq); + if (irq < 0) + return irq; idev->i2c_clk = devm_clk_get(&pdev->dev, NULL); if (IS_ERR(idev->i2c_clk))
Referring to platform_get_irq()'s definition, the return value has already been checked, error message also been printed via dev_err_probe() if ret < 0. Calling dev_err_probe() one more time outside platform_get_irq() is obviously redundant. Furthermore, platform_get_irq() will never return irq equals 0, removing spi->irq == 0 checking to clean it up. Signed-off-by: Chen Jiahao <chenjiahao16@huawei.com> --- drivers/i2c/busses/i2c-microchip-corei2c.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)