amd-xgbe: Stop the PHY before releasing interrupts
authorLendacky, Thomas <Thomas.Lendacky@amd.com>
Tue, 28 Feb 2017 21:02:51 +0000 (15:02 -0600)
committerDavid S. Miller <davem@davemloft.net>
Thu, 2 Mar 2017 20:57:53 +0000 (12:57 -0800)
Some configurations require the use of the hardware's MDIO support to
communicate with external PHYs. The MDIO commands indicate completion
through the device interrupt. When bringing down the device the interrupts
were released before stopping the external PHY, resulting in MDIO command
timeouts. Move the stopping of the PHY to before the releasing of the
interrupts.

Signed-off-by: Tom Lendacky <thomas.lendacky@amd.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/amd/xgbe/xgbe-drv.c

index 3aa457c8ca21d30f768eaf57b2f94eedeaadf40d..248f60d171a5a0ce76744a95e2d59039939e6538 100644 (file)
@@ -1131,12 +1131,12 @@ static void xgbe_stop(struct xgbe_prv_data *pdata)
        hw_if->disable_tx(pdata);
        hw_if->disable_rx(pdata);
 
+       phy_if->phy_stop(pdata);
+
        xgbe_free_irqs(pdata);
 
        xgbe_napi_disable(pdata, 1);
 
-       phy_if->phy_stop(pdata);
-
        hw_if->exit(pdata);
 
        channel = pdata->channel;