diff options
author | Matthias Brugger <mbrugger@suse.com> | 2016-05-03 10:05:07 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-05-04 14:00:44 -0400 |
commit | 20decb7e486d7eefff3931f58d092d2d7c024a1c (patch) | |
tree | 0e9a8948d66215575953b65b0c934ddeb5159054 | |
parent | 7391daf2ffc780679d6ab3fad1db2619e5dd2c2a (diff) |
drivers: net: xgene: Fix error handling
When probe bails out with an error, we try to unregister the
netdev before we have even registered it. Fix the goto statements
for that.
Signed-off-by: Matthias Brugger <mbrugger@suse.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/apm/xgene/xgene_enet_main.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/net/ethernet/apm/xgene/xgene_enet_main.c b/drivers/net/ethernet/apm/xgene/xgene_enet_main.c index 8d4c1ad2fc60..99d7e580e166 100644 --- a/drivers/net/ethernet/apm/xgene/xgene_enet_main.c +++ b/drivers/net/ethernet/apm/xgene/xgene_enet_main.c | |||
@@ -1595,21 +1595,22 @@ static int xgene_enet_probe(struct platform_device *pdev) | |||
1595 | 1595 | ||
1596 | ret = xgene_enet_init_hw(pdata); | 1596 | ret = xgene_enet_init_hw(pdata); |
1597 | if (ret) | 1597 | if (ret) |
1598 | goto err; | 1598 | goto err_netdev; |
1599 | 1599 | ||
1600 | mac_ops = pdata->mac_ops; | 1600 | mac_ops = pdata->mac_ops; |
1601 | if (pdata->phy_mode == PHY_INTERFACE_MODE_RGMII) { | 1601 | if (pdata->phy_mode == PHY_INTERFACE_MODE_RGMII) { |
1602 | ret = xgene_enet_mdio_config(pdata); | 1602 | ret = xgene_enet_mdio_config(pdata); |
1603 | if (ret) | 1603 | if (ret) |
1604 | goto err; | 1604 | goto err_netdev; |
1605 | } else { | 1605 | } else { |
1606 | INIT_DELAYED_WORK(&pdata->link_work, mac_ops->link_state); | 1606 | INIT_DELAYED_WORK(&pdata->link_work, mac_ops->link_state); |
1607 | } | 1607 | } |
1608 | 1608 | ||
1609 | xgene_enet_napi_add(pdata); | 1609 | xgene_enet_napi_add(pdata); |
1610 | return 0; | 1610 | return 0; |
1611 | err: | 1611 | err_netdev: |
1612 | unregister_netdev(ndev); | 1612 | unregister_netdev(ndev); |
1613 | err: | ||
1613 | free_netdev(ndev); | 1614 | free_netdev(ndev); |
1614 | return ret; | 1615 | return ret; |
1615 | } | 1616 | } |