video: fbdev: pxa168fb: Remove unnecessary print function dev_err()
authorYang Li <yang.lee@linux.alibaba.com>
Tue, 1 Feb 2022 12:32:01 +0000 (20:32 +0800)
committerHelge Deller <deller@gmx.de>
Tue, 1 Feb 2022 19:45:51 +0000 (20:45 +0100)
The print function dev_err() is redundant because platform_get_irq()
already prints an error.

Eliminate the follow coccicheck warning:
./drivers/video/fbdev/pxa168fb.c:621:2-9: line 621 is redundant because
platform_get_irq() already prints an error

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
Signed-off-by: Helge Deller <deller@gmx.de>
drivers/video/fbdev/pxa168fb.c

index c25739f..e943300 100644 (file)
@@ -617,10 +617,8 @@ static int pxa168fb_probe(struct platform_device *pdev)
        }
 
        irq = platform_get_irq(pdev, 0);
-       if (irq < 0) {
-               dev_err(&pdev->dev, "no IRQ defined\n");
+       if (irq < 0)
                return -ENOENT;
-       }
 
        info = framebuffer_alloc(sizeof(struct pxa168fb_info), &pdev->dev);
        if (info == NULL) {