From: Wei Yongjun <[email protected]>

Fix to return a negative error code from the error handling
case instead of 0.

Signed-off-by: Wei Yongjun <[email protected]>
---
 drivers/gpio/gpio-sodaville.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/drivers/gpio/gpio-sodaville.c b/drivers/gpio/gpio-sodaville.c
index 5d171e0..9bd2c50 100644
--- a/drivers/gpio/gpio-sodaville.c
+++ b/drivers/gpio/gpio-sodaville.c
@@ -176,8 +176,10 @@ static int sdv_register_irqsupport(struct 
sdv_gpio_chip_data *sd,
 
        sd->id = irq_domain_add_legacy(pdev->dev.of_node, SDV_NUM_PUB_GPIOS,
                                sd->irq_base, 0, &irq_domain_sdv_ops, sd);
-       if (!sd->id)
+       if (!sd->id) {
+               ret = -ENODEV;
                goto out_free_irq;
+       }
        return 0;
 out_free_irq:
        free_irq(pdev->irq, sd);
@@ -212,8 +214,10 @@ static int sdv_gpio_probe(struct pci_dev *pdev,
        }
 
        addr = pci_resource_start(pdev, GPIO_BAR);
-       if (!addr)
+       if (!addr) {
+               ret = -ENODEV;
                goto release_reg;
+       }
        sd->gpio_pub_base = ioremap(addr, pci_resource_len(pdev, GPIO_BAR));
 
        prop = of_get_property(pdev->dev.of_node, "intel,muxctl", &len);

--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to