Commit 10f975f8 authored May 06, 2023 by Jiapeng Chong Committed by Thierry Reding Jul 27, 2023

soc/tegra: cbb: Remove unnecessary print function dev_err()



The print function dev_err() is redundant because platform_get_irq()
already prints an error.

./drivers/soc/tegra/cbb/tegra-cbb.c:130:3-10: line 130 is redundant because platform_get_irq() already prints an error.
./drivers/soc/tegra/cbb/tegra-cbb.c:140:2-9: line 140 is redundant because platform_get_irq() already prints an error.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=4879


Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>

parent ac60f062

drivers/soc/tegra/cbb/tegra-cbb.c

+2 −6

Original line number	Diff line number	Diff line
		@@ -122,20 +122,16 @@ int tegra_cbb_get_irq(struct platform_device pdev, unsigned int nonsec_irq,

		if (num_intr == 2) {
		irq = platform_get_irq(pdev, index);
		if (irq <= 0) {
		dev_err(&pdev->dev, "failed to get non-secure IRQ: %d\n", irq);
		if (irq <= 0)
		return -ENOENT;
		}

		*nonsec_irq = irq;
		index++;
		}

		irq = platform_get_irq(pdev, index);
		if (irq <= 0) {
		dev_err(&pdev->dev, "failed to get secure IRQ: %d\n", irq);
		if (irq <= 0)
		return -ENOENT;
		}

		*sec_irq = irq;