diff options
author | Florian Fainelli <f.fainelli@gmail.com> | 2014-02-25 00:56:11 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-02-25 01:26:36 (GMT) |
commit | 80d8e96d127a91dc3f298e9bb959473b9df1063a (patch) | |
tree | b475b79452eb058decdf9c1f0dca7dd6222a08f7 | |
parent | 432c5b3a108c7e5185e34c4f25dc61824740c21d (diff) | |
download | linux-80d8e96d127a91dc3f298e9bb959473b9df1063a.tar.xz |
net: bcmgenet: drop checks on priv->phydev
Drop all the checks on priv->phydev since we will refuse probing the
driver if we cannot attach to a PHY device. Drop all checks on
priv->phydev. This also fixes some smatch issues reported by Dan
Carpenter.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/broadcom/genet/bcmgenet.c | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/drivers/net/ethernet/broadcom/genet/bcmgenet.c b/drivers/net/ethernet/broadcom/genet/bcmgenet.c index 192069d..5856457 100644 --- a/drivers/net/ethernet/broadcom/genet/bcmgenet.c +++ b/drivers/net/ethernet/broadcom/genet/bcmgenet.c @@ -732,8 +732,7 @@ static void bcmgenet_power_down(struct bcmgenet_priv *priv, switch (mode) { case GENET_POWER_CABLE_SENSE: - if (priv->phydev) - phy_detach(priv->phydev); + phy_detach(priv->phydev); break; case GENET_POWER_PASSIVE: @@ -1811,9 +1810,8 @@ static void bcmgenet_irq_task(struct work_struct *work) /* Link UP/DOWN event */ if ((priv->hw_params->flags & GENET_HAS_MDIO_INTR) && (priv->irq0_stat & (UMAC_IRQ_LINK_UP|UMAC_IRQ_LINK_DOWN))) { - if (priv->phydev) - phy_mac_interrupt(priv->phydev, - (priv->irq0_stat & UMAC_IRQ_LINK_UP)); + phy_mac_interrupt(priv->phydev, + priv->irq0_stat & UMAC_IRQ_LINK_UP); priv->irq0_stat &= ~(UMAC_IRQ_LINK_UP|UMAC_IRQ_LINK_DOWN); } } @@ -1931,8 +1929,7 @@ static int bcmgenet_wol_resume(struct bcmgenet_priv *priv) if (ret) return ret; - if (priv->phydev) - phy_init_hw(priv->phydev); + phy_init_hw(priv->phydev); /* Speed settings must be restored */ bcmgenet_mii_config(priv->dev); @@ -2058,8 +2055,7 @@ static int bcmgenet_open(struct net_device *dev) netif_tx_start_all_queues(dev); - if (priv->phydev) - phy_start(priv->phydev); + phy_start(priv->phydev); return 0; @@ -2134,8 +2130,7 @@ static int bcmgenet_close(struct net_device *dev) netif_dbg(priv, ifdown, dev, "bcmgenet_close\n"); - if (priv->phydev) - phy_stop(priv->phydev); + phy_stop(priv->phydev); /* Disable MAC receive */ reg = bcmgenet_umac_readl(priv, UMAC_CMD); |