diff options
author | YueHaibing <yuehaibing@huawei.com> | 2018-05-16 19:18:22 +0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-05-16 14:44:29 -0400 |
commit | 76e597eb15297d402685ee1442405d3eea70136b (patch) | |
tree | 0212cb0d31c9e488fcdcbdd5384562848c43a426 /drivers/net/ethernet/ethoc.c | |
parent | 93120eba4890e0fcf1e42059d495a7d852db31ed (diff) | |
download | lwn-76e597eb15297d402685ee1442405d3eea70136b.tar.gz lwn-76e597eb15297d402685ee1442405d3eea70136b.zip |
net: ethoc: Remove useless test before clk_disable_unprepare
clk_disable_unprepare() already checks that the clock pointer is valid.
No need to test it before calling it.
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Reviewed-by: Tobias Klauser <tklauser@distanz.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/ethoc.c')
-rw-r--r-- | drivers/net/ethernet/ethoc.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/ethernet/ethoc.c b/drivers/net/ethernet/ethoc.c index 8bb0db990c8f..00a57273b753 100644 --- a/drivers/net/ethernet/ethoc.c +++ b/drivers/net/ethernet/ethoc.c @@ -1246,8 +1246,7 @@ error: mdiobus_unregister(priv->mdio); mdiobus_free(priv->mdio); free2: - if (priv->clk) - clk_disable_unprepare(priv->clk); + clk_disable_unprepare(priv->clk); free: free_netdev(netdev); out: @@ -1271,8 +1270,7 @@ static int ethoc_remove(struct platform_device *pdev) mdiobus_unregister(priv->mdio); mdiobus_free(priv->mdio); } - if (priv->clk) - clk_disable_unprepare(priv->clk); + clk_disable_unprepare(priv->clk); unregister_netdev(netdev); free_netdev(netdev); } |