diff options
| author |   Florian Fainelli <f.fainelli@gmail.com> | 2018-05-15 16:56:19 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2018-05-16 14:20:36 -0400 |
| commit | 00e798c7d1ea5c4514401f17db8300db934291cb (patch) | |
| tree | 831558f37e80e04ed2caf7cb7fc36bd397b52cdb /drivers/net/dsa/bcm_sf2.c | |
| parent | of: mdio: Fall back to mdiobus_register() with NULL device_node (diff) | |
| download | linux-dev-00e798c7d1ea5c4514401f17db8300db934291cb.tar.xz linux-dev-00e798c7d1ea5c4514401f17db8300db934291cb.zip | |
drivers: net: Remove device_node checks with of_mdiobus_register()
A number of drivers have the following pattern:
if (np)
of_mdiobus_register()
else
mdiobus_register()
which the implementation of of_mdiobus_register() now takes care of.
Remove that pattern in drivers that strictly adhere to it.
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
Reviewed-by: Fugang Duan <fugang.duan@nxp.com>
Reviewed-by: Antoine Tenart <antoine.tenart@bootlin.com>
Reviewed-by: Jose Abreu <joabreu@synopsys.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/dsa/bcm_sf2.c')
| -rw-r--r-- | drivers/net/dsa/bcm_sf2.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/net/dsa/bcm_sf2.c b/drivers/net/dsa/bcm_sf2.c index ac621f44237a..02e8982519ce 100644 --- a/drivers/net/dsa/bcm_sf2.c +++ b/drivers/net/dsa/bcm_sf2.c @@ -450,12 +450,8 @@ static int bcm_sf2_mdio_register(struct dsa_switch *ds) priv->slave_mii_bus->parent = ds->dev->parent; priv->slave_mii_bus->phy_mask = ~priv->indir_phy_mask; - if (dn) - err = of_mdiobus_register(priv->slave_mii_bus, dn); - else - err = mdiobus_register(priv->slave_mii_bus); - - if (err) + err = of_mdiobus_register(priv->slave_mii_bus, dn); + if (err && dn) of_node_put(dn); return err; |