memory: tegra: Remove superfluous error messages around platform_get_irq()
authorDmitry Osipenko <digetx@gmail.com>
Wed, 4 Nov 2020 16:49:06 +0000 (19:49 +0300)
committerKrzysztof Kozlowski <krzk@kernel.org>
Thu, 26 Nov 2020 17:50:35 +0000 (18:50 +0100)
The platform_get_irq() prints error message telling that interrupt is
missing, hence there is no need to duplicated that message in the drivers.

Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Link: https://lore.kernel.org/r/20201104164923.21238-31-digetx@gmail.com
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
drivers/memory/tegra/mc.c
drivers/memory/tegra/tegra20-emc.c
drivers/memory/tegra/tegra30-emc.c

index a72cdca..998f914 100644 (file)
@@ -707,10 +707,8 @@ static int tegra_mc_probe(struct platform_device *pdev)
        }
 
        mc->irq = platform_get_irq(pdev, 0);
-       if (mc->irq < 0) {
-               dev_err(&pdev->dev, "interrupt not specified\n");
+       if (mc->irq < 0)
                return mc->irq;
-       }
 
        WARN(!mc->soc->client_id_mask, "missing client ID mask for this SoC\n");
 
index 5ba4e49..c9fe58a 100644 (file)
@@ -665,7 +665,6 @@ static int tegra_emc_probe(struct platform_device *pdev)
 
        irq = platform_get_irq(pdev, 0);
        if (irq < 0) {
-               dev_err(&pdev->dev, "interrupt not specified\n");
                dev_err(&pdev->dev, "please update your device tree\n");
                return irq;
        }
index c58cf31..1be28e2 100644 (file)
@@ -1299,10 +1299,9 @@ static int tegra_emc_probe(struct platform_device *pdev)
                return err;
 
        err = platform_get_irq(pdev, 0);
-       if (err < 0) {
-               dev_err(&pdev->dev, "interrupt not specified: %d\n", err);
+       if (err < 0)
                return err;
-       }
+
        emc->irq = err;
 
        err = devm_request_irq(&pdev->dev, emc->irq, tegra_emc_isr, 0,