Commit 83b1dd75 authored by Devyn Liu's avatar Devyn Liu Committed by Guo Mengqi
Browse files

spi: hisi-sfc-v3xx: Return IRQ_NONE if no interrupts were detected 

stable inclusion
from stable-v5.10.211
commit e94da8aca2e78ef9ecca02eb211869eacd5504e5
category: bugfix
bugzilla: https://gitee.com/src-openeuler/kernel/issues/I9E2NX

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=e94da8aca2e78ef9ecca02eb211869eacd5504e5



--------------------------------

[ Upstream commit de8b6e1c231a95abf95ad097b993d34b31458ec9 ]

Return IRQ_NONE from the interrupt handler when no interrupt was
detected. Because an empty interrupt will cause a null pointer error:

    Unable to handle kernel NULL pointer dereference at virtual
  address 0000000000000008
    Call trace:
        complete+0x54/0x100
        hisi_sfc_v3xx_isr+0x2c/0x40 [spi_hisi_sfc_v3xx]
        __handle_irq_event_percpu+0x64/0x1e0
        handle_irq_event+0x7c/0x1cc

Signed-off-by: default avatarDevyn Liu <liudingyuan@huawei.com>
Link: https://msgid.link/r/20240123071149.917678-1-liudingyuan@huawei.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
Signed-off-by: default avatarWang Hai <wanghai38@huawei.com>
parent 6f369928

drivers/spi/spi-hisi-sfc-v3xx.c

+5 −0
Original line number Diff line number Diff line
@@ -377,6 +377,11 @@ static const struct spi_controller_mem_ops hisi_sfc_v3xx_mem_ops = { 
static irqreturn_t hisi_sfc_v3xx_isr(int irq, void *data)

{

	struct hisi_sfc_v3xx_host *host = data;

	u32 reg;



	reg = readl(host->regbase + HISI_SFC_V3XX_INT_STAT);

	if (!reg)

		return IRQ_NONE;



	hisi_sfc_v3xx_disable_int(host);