1 From 7fa63fdde703aaabaa7199ae879219737a98a3f3 Mon Sep 17 00:00:00 2001
2 From: Jonas Gorski <jonas.gorski@gmail.com>
3 Date: Fri, 6 Jan 2012 12:24:18 +0100
4 Subject: [PATCH] NET: bcm63xx_enet: move phy_(dis)connect into probe/remove
6 Only connect/disconnect the phy during probe and remove, not during any
7 open/close. The phy seldom changes during the runtime, and disconnecting
8 the phy during close will prevent it from keeping any configuration over
11 Signed-off-by: Jonas Gorski <jonas.gorski@gmail.com>
13 drivers/net/ethernet/broadcom/bcm63xx_enet.c | 158 +++++++++++++--------------
14 1 file changed, 78 insertions(+), 80 deletions(-)
16 --- a/drivers/net/ethernet/broadcom/bcm63xx_enet.c
17 +++ b/drivers/net/ethernet/broadcom/bcm63xx_enet.c
18 @@ -867,10 +867,8 @@ static int bcm_enet_open(struct net_devi
19 struct bcm_enet_priv *priv;
22 - struct phy_device *phydev;
25 - char phy_id[MII_BUS_ID_SIZE + 3];
29 @@ -878,40 +876,10 @@ static int bcm_enet_open(struct net_devi
30 kdev = &priv->pdev->dev;
33 - /* connect to PHY */
34 - snprintf(phy_id, sizeof(phy_id), PHY_ID_FMT,
35 - priv->mii_bus->id, priv->phy_id);
37 - phydev = phy_connect(dev, phy_id, bcm_enet_adjust_phy_link,
38 - PHY_INTERFACE_MODE_MII);
40 - if (IS_ERR(phydev)) {
41 - dev_err(kdev, "could not attach to PHY\n");
42 - return PTR_ERR(phydev);
45 - /* mask with MAC supported features */
46 - phydev->supported &= (SUPPORTED_10baseT_Half |
47 - SUPPORTED_10baseT_Full |
48 - SUPPORTED_100baseT_Half |
49 - SUPPORTED_100baseT_Full |
53 - phydev->advertising = phydev->supported;
55 - if (priv->pause_auto && priv->pause_rx && priv->pause_tx)
56 - phydev->advertising |= SUPPORTED_Pause;
58 - phydev->advertising &= ~SUPPORTED_Pause;
60 - phy_attached_info(phydev);
64 priv->old_duplex = -1;
70 /* mask all interrupts and request them */
71 @@ -921,7 +889,7 @@ static int bcm_enet_open(struct net_devi
73 ret = request_irq(dev->irq, bcm_enet_isr_mac, 0, dev->name, dev);
75 - goto out_phy_disconnect;
78 ret = request_irq(priv->irq_rx, bcm_enet_isr_dma, 0,
80 @@ -1083,8 +1051,8 @@ static int bcm_enet_open(struct net_devi
81 enet_dmac_writel(priv, priv->dma_chan_int_mask,
82 ENETDMAC_IRMASK, priv->tx_chan);
87 + phy_start(dev->phydev);
89 bcm_enet_adjust_link(dev);
91 @@ -1125,10 +1093,6 @@ out_freeirq_rx:
93 free_irq(dev->irq, dev);
97 - phy_disconnect(phydev);
102 @@ -1233,10 +1197,6 @@ static int bcm_enet_stop(struct net_devi
103 free_irq(priv->irq_rx, dev);
104 free_irq(dev->irq, dev);
108 - phy_disconnect(dev->phydev);
113 @@ -1802,14 +1762,47 @@ static int bcm_enet_probe(struct platfor
115 /* do minimal hardware init to be able to probe mii bus */
116 bcm_enet_hw_preinit(priv);
117 + spin_lock_init(&priv->rx_lock);
119 + /* init rx timeout (used for oom) */
120 + timer_setup(&priv->rx_timeout, bcm_enet_refill_rx_timer, 0);
122 + /* init the mib update lock&work */
123 + mutex_init(&priv->mib_update_lock);
124 + INIT_WORK(&priv->mib_update_task, bcm_enet_update_mib_counters_defer);
126 + /* zero mib counters */
127 + for (i = 0; i < ENET_MIB_REG_COUNT; i++)
128 + enet_writel(priv, 0, ENET_MIB_REG(i));
130 + /* register netdevice */
131 + dev->netdev_ops = &bcm_enet_ops;
132 + netif_napi_add(dev, &priv->napi, bcm_enet_poll, 16);
134 + dev->ethtool_ops = &bcm_enet_ethtool_ops;
135 + /* MTU range: 46 - 2028 */
136 + dev->min_mtu = ETH_ZLEN - ETH_HLEN;
137 + dev->max_mtu = BCMENET_MAX_MTU - VLAN_ETH_HLEN;
138 + SET_NETDEV_DEV(dev, &pdev->dev);
140 + ret = register_netdev(dev);
142 + goto out_uninit_hw;
144 + netif_carrier_off(dev);
145 + platform_set_drvdata(pdev, dev);
147 + priv->net_dev = dev;
149 /* MII bus registration */
151 + struct phy_device *phydev;
152 + char phy_id[MII_BUS_ID_SIZE + 3];
154 priv->mii_bus = mdiobus_alloc();
155 if (!priv->mii_bus) {
157 - goto out_uninit_hw;
158 + goto out_unregister_netdev;
162 @@ -1833,6 +1826,35 @@ static int bcm_enet_probe(struct platfor
163 dev_err(&pdev->dev, "unable to register mdio bus\n");
167 + /* connect to PHY */
168 + snprintf(phy_id, sizeof(phy_id), PHY_ID_FMT,
169 + priv->mii_bus->id, priv->phy_id);
171 + phydev = phy_connect(dev, phy_id, bcm_enet_adjust_phy_link,
172 + PHY_INTERFACE_MODE_MII);
174 + if (IS_ERR(phydev)) {
175 + dev_err(&pdev->dev, "could not attach to PHY\n");
176 + goto out_unregister_mdio;
179 + /* mask with MAC supported features */
180 + phydev->supported &= (SUPPORTED_10baseT_Half |
181 + SUPPORTED_10baseT_Full |
182 + SUPPORTED_100baseT_Half |
183 + SUPPORTED_100baseT_Full |
184 + SUPPORTED_Autoneg |
187 + phydev->advertising = phydev->supported;
189 + if (priv->pause_auto && priv->pause_rx && priv->pause_tx)
190 + phydev->advertising |= SUPPORTED_Pause;
192 + phydev->advertising &= ~SUPPORTED_Pause;
194 + phy_attached_info(phydev);
197 /* run platform code to initialize PHY device */
198 @@ -1840,45 +1862,16 @@ static int bcm_enet_probe(struct platfor
199 pd->mii_config(dev, 1, bcm_enet_mdio_read_mii,
200 bcm_enet_mdio_write_mii)) {
201 dev_err(&pdev->dev, "unable to configure mdio bus\n");
202 - goto out_uninit_hw;
203 + goto out_unregister_netdev;
207 - spin_lock_init(&priv->rx_lock);
209 - /* init rx timeout (used for oom) */
210 - timer_setup(&priv->rx_timeout, bcm_enet_refill_rx_timer, 0);
212 - /* init the mib update lock&work */
213 - mutex_init(&priv->mib_update_lock);
214 - INIT_WORK(&priv->mib_update_task, bcm_enet_update_mib_counters_defer);
216 - /* zero mib counters */
217 - for (i = 0; i < ENET_MIB_REG_COUNT; i++)
218 - enet_writel(priv, 0, ENET_MIB_REG(i));
220 - /* register netdevice */
221 - dev->netdev_ops = &bcm_enet_ops;
222 - netif_napi_add(dev, &priv->napi, bcm_enet_poll, 16);
224 - dev->ethtool_ops = &bcm_enet_ethtool_ops;
225 - /* MTU range: 46 - 2028 */
226 - dev->min_mtu = ETH_ZLEN - ETH_HLEN;
227 - dev->max_mtu = BCMENET_MAX_MTU - VLAN_ETH_HLEN;
228 - SET_NETDEV_DEV(dev, &pdev->dev);
230 - ret = register_netdev(dev);
232 - goto out_unregister_mdio;
234 - netif_carrier_off(dev);
235 - platform_set_drvdata(pdev, dev);
237 - priv->net_dev = dev;
243 + phy_disconnect(dev->phydev);
246 mdiobus_unregister(priv->mii_bus);
248 @@ -1886,6 +1879,9 @@ out_free_mdio:
250 mdiobus_free(priv->mii_bus);
252 +out_unregister_netdev:
253 + unregister_netdev(dev);
256 /* turn off mdc clock */
257 enet_writel(priv, 0, ENET_MIISC_REG);
258 @@ -1916,6 +1912,7 @@ static int bcm_enet_remove(struct platfo
259 enet_writel(priv, 0, ENET_MIISC_REG);
262 + phy_disconnect(dev->phydev);
263 mdiobus_unregister(priv->mii_bus);
264 mdiobus_free(priv->mii_bus);