Blob Blame History Raw
From eb9913b511f10968a02cfa5329a896855dd152a3 Mon Sep 17 00:00:00 2001
From: Zhang Shurong <zhang_shurong@foxmail.com>
Date: Sat, 26 Aug 2023 18:02:54 +0800
Subject: [PATCH] spi: tegra: Fix missing IRQ check in tegra_slink_probe()
Git-commit: eb9913b511f10968a02cfa5329a896855dd152a3
Patch-mainline: v6.7-rc1
References: git-fixes

This func misses checking for platform_get_irq()'s call and may passes the
negative error codes to request_irq(), which takes unsigned IRQ #,
causing it to fail with -EINVAL, overriding an original error code.

Fix this by stop calling request_irq() with invalid IRQ #s.

Fixes: dc4dc3605639 ("spi: tegra: add spi driver for SLINK controller")
Signed-off-by: Zhang Shurong <zhang_shurong@foxmail.com>
Reviewed-by: Helen Koike <helen.koike@collabora.com>
Link: https://lore.kernel.org/r/tencent_73FCC06A3D1C14EE5175253C6FB46A07B709@qq.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Acked-by: Takashi Iwai <tiwai@suse.de>

---
 drivers/spi/spi-tegra20-slink.c |    2 ++
 1 file changed, 2 insertions(+)

--- a/drivers/spi/spi-tegra20-slink.c
+++ b/drivers/spi/spi-tegra20-slink.c
@@ -1067,6 +1067,8 @@ static int tegra_slink_probe(struct plat
 	}
 
 	spi_irq = platform_get_irq(pdev, 0);
+	if (spi_irq < 0)
+		return spi_irq;
 	tspi->irq = spi_irq;
 	ret = request_threaded_irq(tspi->irq, tegra_slink_isr,
 			tegra_slink_isr_thread, IRQF_ONESHOT,