Commit-ID:  3900dea4cda7c28d7921370bc4d22b08463ed94c
Gitweb:     http://git.kernel.org/tip/3900dea4cda7c28d7921370bc4d22b08463ed94c
Author:     Christophe JAILLET <christophe.jail...@wanadoo.fr>
AuthorDate: Sat, 18 Feb 2017 09:34:34 +0100
Committer:  Thomas Gleixner <t...@linutronix.de>
CommitDate: Sun, 19 Feb 2017 08:17:25 +0100

irqchip/qcom: Fix error handling

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

Fixes: f20cc9b00c7b ("irqchip/qcom: Add IRQ combiner driver")
Signed-off-by: Christophe JAILLET <christophe.jail...@wanadoo.fr>
Cc: marc.zyng...@arm.com
Cc: kernel-janit...@vger.kernel.org
Cc: ja...@lakedaemon.net
Link: 
http://lkml.kernel.org/r/20170218083434.2289-1-christophe.jail...@wanadoo.fr
Signed-off-by: Thomas Gleixner <t...@linutronix.de>

---
 drivers/irqchip/qcom-irq-combiner.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/irqchip/qcom-irq-combiner.c 
b/drivers/irqchip/qcom-irq-combiner.c
index 03251da..2265586 100644
--- a/drivers/irqchip/qcom-irq-combiner.c
+++ b/drivers/irqchip/qcom-irq-combiner.c
@@ -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);
-       if (IS_ERR(vaddr)) {
+       if (!vaddr) {
                dev_err(ctx->dev, "Can't map register @%pa\n", &paddr);
-               ctx->err = PTR_ERR(vaddr);
+               ctx->err = -ENOMEM;
                return AE_ERROR;
        }
 

Reply via email to