diff options
| author | Ma Ke <make24@iscas.ac.cn> | 2024-07-24 10:40:47 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2024-07-29 02:23:23 +0200 |
| commit | 0f245463b01ea254ae90e1d0389e90b0e7d8dc75 (patch) | |
| tree | 3834bf0daf4ae14b8681297ce13380f639c941b5 /drivers/spi/spi-ppc4xx.c | |
| parent | spi: dt-bindings: mediatek,spi-mt65xx: add compatible for MT7981 (diff) | |
| download | linux-0f245463b01ea254ae90e1d0389e90b0e7d8dc75.tar.xz linux-0f245463b01ea254ae90e1d0389e90b0e7d8dc75.zip | |
spi: ppc4xx: handle irq_of_parse_and_map() errors
Zero and negative number is not a valid IRQ for in-kernel code and the
irq_of_parse_and_map() function returns zero on error. So this check for
valid IRQs should only accept values > 0.
Fixes: 44dab88e7cc9 ("spi: add spi_ppc4xx driver")
Signed-off-by: Ma Ke <make24@iscas.ac.cn>
Link: https://patch.msgid.link/20240724084047.1506084-1-make24@iscas.ac.cn
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to '')
| -rw-r--r-- | drivers/spi/spi-ppc4xx.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/spi/spi-ppc4xx.c b/drivers/spi/spi-ppc4xx.c index 942c3117ab3a..01fdecbf132d 100644 --- a/drivers/spi/spi-ppc4xx.c +++ b/drivers/spi/spi-ppc4xx.c @@ -413,6 +413,9 @@ static int spi_ppc4xx_of_probe(struct platform_device *op) /* Request IRQ */ hw->irqnum = irq_of_parse_and_map(np, 0); + if (hw->irqnum <= 0) + goto free_host; + ret = request_irq(hw->irqnum, spi_ppc4xx_int, 0, "spi_ppc4xx_of", (void *)hw); if (ret) { |