diff options
| author | Ricardo Ribalda <ribalda@chromium.org> | 2024-05-06 19:24:47 +0000 |
|---|---|---|
| committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2024-05-10 01:08:04 +0300 |
| commit | 0cc50ced282a3236c0cac72f72c6b027beaa2660 (patch) | |
| tree | 69b03a095a66ead5b940808c1d88dce97b1a4423 /drivers/media/platform | |
| parent | 05b0b07953b7630705e364fe342689c9af340b32 (diff) | |
| download | lwn-0cc50ced282a3236c0cac72f72c6b027beaa2660.tar.gz lwn-0cc50ced282a3236c0cac72f72c6b027beaa2660.zip | |
media: bcm2835-unicam: Do not print error when irq not found
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>
Diffstat (limited to 'drivers/media/platform')
| -rw-r--r-- | drivers/media/platform/broadcom/bcm2835-unicam.c | 4 |
1 files changed, 1 insertions, 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); |