diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2015-11-07 10:30:34 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-11-07 13:17:32 -0500 |
commit | 3694bfbdb3bd5cf6632140c7f7fdf6f31d43ee66 (patch) | |
tree | e36c618911cd85187001e3812654646feb964fc5 | |
parent | 40baec225765c54eefa870530dd613bad9829bb7 (diff) |
dwc_eth_qos: Delete an unnecessary check before the function call "of_node_put"
The of_node_put() function tests whether its argument is NULL
and then returns immediately.
Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/synopsys/dwc_eth_qos.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ethernet/synopsys/dwc_eth_qos.c b/drivers/net/ethernet/synopsys/dwc_eth_qos.c index 85b3326775b8..9066d7a8483c 100644 --- a/drivers/net/ethernet/synopsys/dwc_eth_qos.c +++ b/drivers/net/ethernet/synopsys/dwc_eth_qos.c | |||
@@ -2970,8 +2970,7 @@ err_out_unregister_netdev: | |||
2970 | err_out_clk_dis_aper: | 2970 | err_out_clk_dis_aper: |
2971 | clk_disable_unprepare(lp->apb_pclk); | 2971 | clk_disable_unprepare(lp->apb_pclk); |
2972 | err_out_free_netdev: | 2972 | err_out_free_netdev: |
2973 | if (lp->phy_node) | 2973 | of_node_put(lp->phy_node); |
2974 | of_node_put(lp->phy_node); | ||
2975 | free_netdev(ndev); | 2974 | free_netdev(ndev); |
2976 | platform_set_drvdata(pdev, NULL); | 2975 | platform_set_drvdata(pdev, NULL); |
2977 | return ret; | 2976 | return ret; |