Commit 3900dea4 authored by Christophe JAILLET's avatar Christophe JAILLET Committed by Thomas Gleixner

irqchip/qcom: Fix error handling

'devm_ioremap()' returns NULL on error, not an error pointer.

Fixes: f20cc9b0 ("irqchip/qcom: Add IRQ combiner driver")
Signed-off-by: default avatarChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Cc: marc.zyngier@arm.com
Cc: kernel-janitors@vger.kernel.org
Cc: jason@lakedaemon.net
Link: http://lkml.kernel.org/r/20170218083434.2289-1-christophe.jaillet@wanadoo.frSigned-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent 5d4bac9a
...@@ -202,9 +202,9 @@ static acpi_status get_registers_cb(struct acpi_resource *ares, void *context) ...@@ -202,9 +202,9 @@ static acpi_status get_registers_cb(struct acpi_resource *ares, void *context)
} }
vaddr = devm_ioremap(ctx->dev, reg->address, REG_SIZE); vaddr = devm_ioremap(ctx->dev, reg->address, REG_SIZE);
if (IS_ERR(vaddr)) { if (!vaddr) {
dev_err(ctx->dev, "Can't map register @%pa\n", &paddr); dev_err(ctx->dev, "Can't map register @%pa\n", &paddr);
ctx->err = PTR_ERR(vaddr); ctx->err = -ENOMEM;
return AE_ERROR; return AE_ERROR;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment