diff options
author | Markus Pargmann <mpa@pengutronix.de> | 2014-09-29 02:53:14 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-09-30 01:30:42 -0400 |
commit | 4d507dffe766b8e29f29d17ae8e0611b5638286e (patch) | |
tree | ca7f336afa883fb5fa1b719b7f0ada859490101a /drivers/net/ethernet/ti/cpsw.c | |
parent | e4a9839b8548cf2668c6ccf961f30bd565e8333f (diff) |
net: cpsw: Add missing return value
ret is set 0 at this point, so jumping to that error label would result
in a return value of 0. Set ret to -ENOMEM to return a proper error
value.
Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
Reviewed-by: Wolfram Sang <wsa@the-dreams.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/ti/cpsw.c')
-rw-r--r-- | drivers/net/ethernet/ti/cpsw.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c index 45ba50e4eaec..187c6ef54885 100644 --- a/drivers/net/ethernet/ti/cpsw.c +++ b/drivers/net/ethernet/ti/cpsw.c | |||
@@ -2123,6 +2123,7 @@ static int cpsw_probe(struct platform_device *pdev) | |||
2123 | priv->irq_enabled = true; | 2123 | priv->irq_enabled = true; |
2124 | if (!priv->cpts) { | 2124 | if (!priv->cpts) { |
2125 | dev_err(&pdev->dev, "error allocating cpts\n"); | 2125 | dev_err(&pdev->dev, "error allocating cpts\n"); |
2126 | ret = -ENOMEM; | ||
2126 | goto clean_ndev_ret; | 2127 | goto clean_ndev_ret; |
2127 | } | 2128 | } |
2128 | 2129 | ||