soc/tegra: cbb: Remove unnecessary print function dev_err()
authorJiapeng Chong <jiapeng.chong@linux.alibaba.com>
Sat, 6 May 2023 09:07:35 +0000 (17:07 +0800)
committerThierry Reding <treding@nvidia.com>
Thu, 27 Jul 2023 16:46:37 +0000 (18:46 +0200)
The print function dev_err() is redundant because platform_get_irq()
already prints an error.

./drivers/soc/tegra/cbb/tegra-cbb.c:130:3-10: line 130 is redundant because platform_get_irq() already prints an error.
./drivers/soc/tegra/cbb/tegra-cbb.c:140:2-9: line 140 is redundant because platform_get_irq() already prints an error.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=4879
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
drivers/soc/tegra/cbb/tegra-cbb.c

index be76c6aa131c5c34dda739cadebaf77e396fe7e2..84ab46c9d9f533f0ff9961ee42dde0868f062fe3 100644 (file)
@@ -122,20 +122,16 @@ int tegra_cbb_get_irq(struct platform_device *pdev, unsigned int *nonsec_irq,
 
        if (num_intr == 2) {
                irq = platform_get_irq(pdev, index);
-               if (irq <= 0) {
-                       dev_err(&pdev->dev, "failed to get non-secure IRQ: %d\n", irq);
+               if (irq <= 0)
                        return -ENOENT;
-               }
 
                *nonsec_irq = irq;
                index++;
        }
 
        irq = platform_get_irq(pdev, index);
-       if (irq <= 0) {
-               dev_err(&pdev->dev, "failed to get secure IRQ: %d\n", irq);
+       if (irq <= 0)
                return -ENOENT;
-       }
 
        *sec_irq = irq;