Commit 196754c2 authored by Dan Carpenter's avatar Dan Carpenter Committed by Thomas Gleixner
Browse files

irqchip/aspeed-scu-ic: Fix an IS_ERR() vs NULL check 



of_iomap() doesn't return error pointers, it returns NULL.  Fix the error
checking to check for NULL pointers.

Fixes: 86cd4301 ("irqchip/aspeed-scu-ic: Refactor driver to support variant-based initialization")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent c746c3b5

drivers/irqchip/irq-aspeed-scu-ic.c

+2 −2
Original line number Diff line number Diff line
@@ -215,8 +215,8 @@ static int aspeed_scu_ic_of_init_common(struct aspeed_scu_ic *scu_ic, 
	int irq, rc = 0;



	scu_ic->base = of_iomap(node, 0);

	if (IS_ERR(scu_ic->base)) {

		rc = PTR_ERR(scu_ic->base);

	if (!scu_ic->base) {

		rc = -ENOMEM;

		goto err;

	}