diff options
author | Fabio Estevam <fabio.estevam@nxp.com> | 2017-08-31 13:52:05 -0400 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2017-09-05 14:33:17 -0400 |
commit | 343ce0cdfae16828d18e9228e26dd19ecc3a457f (patch) | |
tree | dbb677bd7462ef3cffeaef768e6219d90061abf9 /drivers/pci/dwc | |
parent | 16f73eb02d7e1765ccab3d2018e0bd98eb93d973 (diff) |
PCI: spear13xx: Fix platform_get_irq() error handling
platform_get_irq() returns a negative number on failure, so adjust the
logic to detect such condition and propagate the real error value on
failure.
Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Cc: Pratyush Anand <pratyush.anand@gmail.com>
Diffstat (limited to 'drivers/pci/dwc')
-rw-r--r-- | drivers/pci/dwc/pcie-spear13xx.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/pci/dwc/pcie-spear13xx.c b/drivers/pci/dwc/pcie-spear13xx.c index 80897291e0fb..a1882ace3fe4 100644 --- a/drivers/pci/dwc/pcie-spear13xx.c +++ b/drivers/pci/dwc/pcie-spear13xx.c | |||
@@ -199,9 +199,9 @@ static int spear13xx_add_pcie_port(struct spear13xx_pcie *spear13xx_pcie, | |||
199 | int ret; | 199 | int ret; |
200 | 200 | ||
201 | pp->irq = platform_get_irq(pdev, 0); | 201 | pp->irq = platform_get_irq(pdev, 0); |
202 | if (!pp->irq) { | 202 | if (pp->irq < 0) { |
203 | dev_err(dev, "failed to get irq\n"); | 203 | dev_err(dev, "failed to get irq\n"); |
204 | return -ENODEV; | 204 | return pp->irq; |
205 | } | 205 | } |
206 | ret = devm_request_irq(dev, pp->irq, spear13xx_pcie_irq_handler, | 206 | ret = devm_request_irq(dev, pp->irq, spear13xx_pcie_irq_handler, |
207 | IRQF_SHARED | IRQF_NO_THREAD, | 207 | IRQF_SHARED | IRQF_NO_THREAD, |