media: bcm2835-unicam: Do not print error when irq not found
authorRicardo Ribalda <ribalda@chromium.org>
Mon, 6 May 2024 19:24:47 +0000 (19:24 +0000)
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Thu, 9 May 2024 22:08:04 +0000 (01:08 +0300)
platform_get_irq() already prints an error for us.

Fix this cocci warning:
drivers/media/platform/broadcom/bcm2835-unicam.c:2664:2-9: line 2664 is redundant because platform_get_irq() already prints an error

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
Link: https://lore.kernel.org/r/20240506-fix-broad-v2-2-e6a2a5c0d609@chromium.org
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
drivers/media/platform/broadcom/bcm2835-unicam.c

index 94cd66255b7e5a17aec2d406dd8a7b383d29cfc1..bdcb04d633380089732ec7a75a7c484a9162464f 100644 (file)
@@ -2661,10 +2661,8 @@ static int unicam_probe(struct platform_device *pdev)
        }
 
        ret = platform_get_irq(pdev, 0);
-       if (ret < 0) {
-               dev_err(&pdev->dev, "No IRQ resource\n");
+       if (ret < 0)
                goto err_unicam_put;
-       }
 
        ret = devm_request_irq(&pdev->dev, ret, unicam_isr, 0,
                               "unicam_capture0", unicam);