regulator: mt6360: remove redundant error print
authorJian Dong <dongjian@yulong.com>
Wed, 24 Mar 2021 03:11:14 +0000 (11:11 +0800)
committerMark Brown <broonie@kernel.org>
Wed, 24 Mar 2021 19:50:21 +0000 (19:50 +0000)
fixes coccicheck warning:

drivers/regulator/mt6360-regulator.c:384:3-10: line 384 is
redundant because platform_get_irq() already prints an error

in fact it is not platform_get_irq but platform_get_irq_byname print error

Signed-off-by: Jian Dong <dongjian@yulong.com>
Link: https://lore.kernel.org/r/1616555474-158789-1-git-send-email-dj0227@163.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/mt6360-regulator.c

index 15308ee..4d34be9 100644 (file)
@@ -380,10 +380,8 @@ static int mt6360_regulator_irq_register(struct platform_device *pdev,
                const struct mt6360_irq_mapping *irq_desc = tbls + i;
 
                irq = platform_get_irq_byname(pdev, irq_desc->name);
-               if (irq < 0) {
-                       dev_err(&pdev->dev, "Fail to get %s irq\n", irq_desc->name);
+               if (irq < 0)
                        return irq;
-               }
 
                ret = devm_request_threaded_irq(&pdev->dev, irq, NULL, irq_desc->handler, 0,
                                                irq_desc->name, rdev);