gpio: thunderx: fix error return code in thunderx_gpio_probe()
authorWei Yongjun <weiyongjun1@huawei.com>
Wed, 10 Jan 2018 14:37:13 +0000 (14:37 +0000)
committerLinus Walleij <linus.walleij@linaro.org>
Thu, 11 Jan 2018 09:49:32 +0000 (10:49 +0100)
Fix to return error code -ENOMEM from the error handling
case instead of 0, as done elsewhere in this function.

Fixes: 5a2a30024d8c ("gpio: Add gpio driver support for ThunderX and OCTEON-TX")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Acked-by: David Daney <david.daney@cavium.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-thunderx.c

index b5adb79a631a6d88b3921d5210ddf72bc26a4c59..d16e9d4a129bdd86874305d3d443b9eed9f9df7f 100644 (file)
@@ -553,8 +553,10 @@ static int thunderx_gpio_probe(struct pci_dev *pdev,
        txgpio->irqd = irq_domain_create_hierarchy(irq_get_irq_data(txgpio->msix_entries[0].vector)->domain,
                                                   0, 0, of_node_to_fwnode(dev->of_node),
                                                   &thunderx_gpio_irqd_ops, txgpio);
-       if (!txgpio->irqd)
+       if (!txgpio->irqd) {
+               err = -ENOMEM;
                goto out;
+       }
 
        /* Push on irq_data and the domain for each line. */
        for (i = 0; i < ngpio; i++) {