rtc: remove unnecessary error message after platform_get_irq

The function “platform_get_irq” can log an error already.
Thus omit redundant messages for the exception handling in the
calling functions.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Acked-by: Vladimir Zapolskiy <vz@mleia.com>
Link: https://lore.kernel.org/r/04116352-b464-041c-1939-96440133aa6f@web.de
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>

diff --git a/drivers/rtc/rtc-fsl-ftm-alarm.c b/drivers/rtc/rtc-fsl-ftm-alarm.c
index 756af62..23c811d 100644 (file)
--- a/drivers/rtc/rtc-fsl-ftm-alarm.c
+++ b/drivers/rtc/rtc-fsl-ftm-alarm.c
@@ -268,10 +268,8 @@ static int ftm_rtc_probe(struct platform_device *pdev)
        }
 
        irq = platform_get_irq(pdev, 0);
-       if (irq < 0) {
-               dev_err(&pdev->dev, "can't get irq number\n");
+       if (irq < 0)
                return irq;
-       }
 
        ret = devm_request_irq(&pdev->dev, irq, ftm_rtc_alarm_interrupt,
                               IRQF_NO_SUSPEND, dev_name(&pdev->dev), rtc);

diff --git a/drivers/rtc/rtc-lpc24xx.c b/drivers/rtc/rtc-lpc24xx.c
index 00ef16b..eec881a 100644 (file)
--- a/drivers/rtc/rtc-lpc24xx.c
+++ b/drivers/rtc/rtc-lpc24xx.c
@@ -205,10 +205,8 @@ static int lpc24xx_rtc_probe(struct platform_device *pdev)
                return PTR_ERR(rtc->rtc_base);
 
        irq = platform_get_irq(pdev, 0);
-       if (irq < 0) {
-               dev_warn(&pdev->dev, "can't get interrupt resource\n");
+       if (irq < 0)
                return irq;
-       }
 
        rtc->clk_rtc = devm_clk_get(&pdev->dev, "rtc");
        if (IS_ERR(rtc->clk_rtc)) {

diff --git a/drivers/rtc/rtc-mt2712.c b/drivers/rtc/rtc-mt2712.c
index 581b873..f6bdbab 100644 (file)
--- a/drivers/rtc/rtc-mt2712.c
+++ b/drivers/rtc/rtc-mt2712.c
@@ -328,10 +328,8 @@ static int mt2712_rtc_probe(struct platform_device *pdev)
        mt2712_rtc_hw_init(mt2712_rtc);
 
        mt2712_rtc->irq = platform_get_irq(pdev, 0);
-       if (mt2712_rtc->irq < 0) {
-               dev_err(&pdev->dev, "No IRQ resource\n");
+       if (mt2712_rtc->irq < 0)
                return mt2712_rtc->irq;
-       }
 
        platform_set_drvdata(pdev, mt2712_rtc);