diff options
| author |   Zhang Changzhong <zhangchangzhong@huawei.com> | 2020-09-09 22:06:37 +0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2020-09-09 14:16:33 -0700 |
| commit | a0d48518cdc855ed188be385abb4df1752a4003c (patch) | |
| tree | ecbfd342658c021c6c003318b63c0bb91f74b154 /drivers/net/ethernet/marvell/pxa168_eth.c | |
| parent | Merge branch 'SMSC-Cleanups-and-clock-setup' (diff) | |
| download | linux-dev-a0d48518cdc855ed188be385abb4df1752a4003c.tar.xz linux-dev-a0d48518cdc855ed188be385abb4df1752a4003c.zip | |
net: pxa168_eth: remove redundant null check before clk_disable_unprepare()
Because clk_prepare_enable() and clk_disable_unprepare() already checked
NULL clock parameter, so the additional checks are unnecessary, just
remove them.
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Zhang Changzhong <zhangchangzhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/ethernet/marvell/pxa168_eth.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/net/ethernet/marvell/pxa168_eth.c b/drivers/net/ethernet/marvell/pxa168_eth.c index eb8cf60ecf12..faac94b2d5ff 100644 --- a/drivers/net/ethernet/marvell/pxa168_eth.c +++ b/drivers/net/ethernet/marvell/pxa168_eth.c @@ -1541,10 +1541,8 @@ static int pxa168_eth_remove(struct platform_device *pdev) } if (dev->phydev) phy_disconnect(dev->phydev); - if (pep->clk) { - clk_disable_unprepare(pep->clk); - } + clk_disable_unprepare(pep->clk); mdiobus_unregister(pep->smi_bus); mdiobus_free(pep->smi_bus); unregister_netdev(dev); |