diff options
| author | zhong jiang <zhongjiang@huawei.com> | 2018-09-16 21:22:31 +0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2018-09-17 08:20:11 -0700 |
| commit | 1ddc5d3e5f1cf9ac8a6aad00b95a7a022e634ef4 (patch) | |
| tree | 3a7e477ff2f29217485f352a226afe96eb45d316 | |
| parent | 764ea3714a85bfc0e3e470cee2166b6ef911b9b9 (diff) | |
net: dsa: remove redundant null pointer check before of_node_put
of_node_put has taken the null pointer check into account. So it is
safe to remove the duplicated check before of_node_put.
Signed-off-by: zhong jiang <zhongjiang@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/dsa/bcm_sf2.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/dsa/bcm_sf2.c b/drivers/net/dsa/bcm_sf2.c index e0066adcd2f3..1fc27e149e7f 100644 --- a/drivers/net/dsa/bcm_sf2.c +++ b/drivers/net/dsa/bcm_sf2.c @@ -465,8 +465,7 @@ static int bcm_sf2_mdio_register(struct dsa_switch *ds) static void bcm_sf2_mdio_unregister(struct bcm_sf2_priv *priv) { mdiobus_unregister(priv->slave_mii_bus); - if (priv->master_mii_dn) - of_node_put(priv->master_mii_dn); + of_node_put(priv->master_mii_dn); } static u32 bcm_sf2_sw_get_phy_flags(struct dsa_switch *ds, int port) |