PCI: rcar: Fix error handling of irq_of_parse_and_map()
authorDmitry Torokhov <dtor@chromium.org>
Fri, 14 Nov 2014 22:21:53 +0000 (14:21 -0800)
committerBjorn Helgaas <bhelgaas@google.com>
Fri, 26 Dec 2014 23:24:27 +0000 (16:24 -0700)
Return value of irq_of_parse_and_map() is unsigned int, with 0 indicating
failure, so testing for negative result never works.

Signed-off-by: Dmitry Torokhov <dtor@chromium.org>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Phil Edworthy <phil.edworthy@renesas.com>
drivers/pci/host/pcie-rcar.c

index 748786c402fcb2b38ab0daa43498501687f2963e..b25874484833f1c75db02e56f0bc3c907f9eb076 100644 (file)
@@ -757,7 +757,7 @@ static int rcar_pcie_get_resources(struct platform_device *pdev,
                goto err_map_reg;
 
        i = irq_of_parse_and_map(pdev->dev.of_node, 0);
-       if (i < 0) {
+       if (!i) {
                dev_err(pcie->dev, "cannot get platform resources for msi interrupt\n");
                err = -ENOENT;
                goto err_map_reg;
@@ -765,7 +765,7 @@ static int rcar_pcie_get_resources(struct platform_device *pdev,
        pcie->msi.irq1 = i;
 
        i = irq_of_parse_and_map(pdev->dev.of_node, 1);
-       if (i < 0) {
+       if (!i) {
                dev_err(pcie->dev, "cannot get platform resources for msi interrupt\n");
                err = -ENOENT;
                goto err_map_reg;