regulator: drivers: Remove unnecessary print function dev_err()
authorYang Li <yang.lee@linux.alibaba.com>
Thu, 25 Aug 2022 07:04:38 +0000 (15:04 +0800)
committerMark Brown <broonie@kernel.org>
Thu, 25 Aug 2022 11:29:35 +0000 (12:29 +0100)
The print function dev_err() is redundant because
platform_get_irq_byname() already prints an error.

Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=1986
Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
Link: https://lore.kernel.org/r/20220825070438.128093-1-yang.lee@linux.alibaba.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/tps65219-regulator.c

index 7ba2594..ab16e66 100644 (file)
@@ -363,11 +363,9 @@ static int tps65219_regulator_probe(struct platform_device *pdev)
                irq_type = &tps65219_regulator_irq_types[i];
 
                irq = platform_get_irq_byname(pdev, irq_type->irq_name);
-               if (irq < 0) {
-                       dev_err(tps->dev, "Failed to get IRQ %s: %d\n",
-                               irq_type->irq_name, irq);
+               if (irq < 0)
                        return -EINVAL;
-               }
+
                irq_data[i].dev = tps->dev;
                irq_data[i].type = irq_type;