This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: bcm2835-unicam: Do not print error when irq not found
Author:  Ricardo Ribalda <[email protected]>
Date:    Mon May 6 19:24:47 2024 +0000

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 <[email protected]>
Signed-off-by: Ricardo Ribalda <[email protected]>
Link: 
https://lore.kernel.org/r/[email protected]
Signed-off-by: Laurent Pinchart <[email protected]>

 drivers/media/platform/broadcom/bcm2835-unicam.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

---

diff --git a/drivers/media/platform/broadcom/bcm2835-unicam.c 
b/drivers/media/platform/broadcom/bcm2835-unicam.c
index 94cd66255b7e..bdcb04d63338 100644
--- a/drivers/media/platform/broadcom/bcm2835-unicam.c
+++ b/drivers/media/platform/broadcom/bcm2835-unicam.c
@@ -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);

Reply via email to