diff options
| author | Florian Fainelli <f.fainelli@gmail.com> | 2014-02-24 16:56:11 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-02-24 20:26:36 -0500 |
| commit | 80d8e96d127a91dc3f298e9bb959473b9df1063a (patch) | |
| tree | b475b79452eb058decdf9c1f0dca7dd6222a08f7 | |
| parent | 432c5b3a108c7e5185e34c4f25dc61824740c21d (diff) | |
| download | linux-80d8e96d127a91dc3f298e9bb959473b9df1063a.tar.bz2 | |
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 192069d2745e..585645749d6d 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); |