diff options
| author | Dan Carpenter <[email protected]> | 2024-08-20 11:42:40 +0300 |
|---|---|---|
| committer | Thomas Gleixner <[email protected]> | 2024-08-20 17:05:32 +0200 |
| commit | efe81b7bdf7d882d0ce3d183f1571321046da8f1 (patch) | |
| tree | c33257f107940f70f6570395cd8d5a35bf4442a3 | |
| parent | f97fd458763a4801d04dbb4a79d9ca6282d293ec (diff) | |
irqchip/riscv-aplic: Fix an IS_ERR() vs NULL bug in probe()
The devm_platform_ioremap_resource() function doesn't return NULL, it
returns error pointers. Fix the error handling to match.
Fixes: 2333df5ae51e ("irqchip: Add RISC-V advanced PLIC driver for direct-mode")
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Thomas Gleixner <[email protected]>
Reviewed-by: Jinjie Ruan <[email protected]>
Reviewed-by: Anup Patel <[email protected]>
Cc: [email protected]
Link: https://lore.kernel.org/all/[email protected]
| -rw-r--r-- | drivers/irqchip/irq-riscv-aplic-main.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/irqchip/irq-riscv-aplic-main.c b/drivers/irqchip/irq-riscv-aplic-main.c index 28dd175b5764..981fad6fb8f7 100644 --- a/drivers/irqchip/irq-riscv-aplic-main.c +++ b/drivers/irqchip/irq-riscv-aplic-main.c @@ -175,9 +175,9 @@ static int aplic_probe(struct platform_device *pdev) /* Map the MMIO registers */ regs = devm_platform_ioremap_resource(pdev, 0); - if (!regs) { + if (IS_ERR(regs)) { dev_err(dev, "failed map MMIO registers\n"); - return -ENOMEM; + return PTR_ERR(regs); } /* |