diff options
author | Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> | 2017-07-29 22:18:41 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-07-31 17:56:47 -0700 |
commit | cfbcb61f6c2ac9aeb7731d2ec59fe3bf53786c49 (patch) | |
tree | 1a914df8766d523f2480a3edb2348a450f276d84 /drivers/net/ethernet | |
parent | 13332db54fb5a821c86432238d33be1ff0429394 (diff) | |
download | linux-cfbcb61f6c2ac9aeb7731d2ec59fe3bf53786c49.tar.bz2 |
mv643xx_eth: fix of_irq_to_resource() error check
of_irq_to_resource() has recently been fixed to return negative error #'s
along with 0 in case of failure, however the Marvell MV643xx Ethernet
driver still only regards 0 as invalid IRQ -- fix it up.
Fixes: 7a4228bbff76 ("of: irq: use of_irq_get() in of_irq_to_resource()")
Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet')
-rw-r--r-- | drivers/net/ethernet/marvell/mv643xx_eth.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/marvell/mv643xx_eth.c b/drivers/net/ethernet/marvell/mv643xx_eth.c index 5794d98d946f..9c94ea9b2b80 100644 --- a/drivers/net/ethernet/marvell/mv643xx_eth.c +++ b/drivers/net/ethernet/marvell/mv643xx_eth.c @@ -2734,7 +2734,7 @@ static int mv643xx_eth_shared_of_add_port(struct platform_device *pdev, ppd.shared = pdev; memset(&res, 0, sizeof(res)); - if (!of_irq_to_resource(pnp, 0, &res)) { + if (of_irq_to_resource(pnp, 0, &res) <= 0) { dev_err(&pdev->dev, "missing interrupt on %s\n", pnp->name); return -EINVAL; } |