Messages in this thread Patches in this message | | | From | Jiang Wang <> | Subject | RE: [PATCH 2/2] e100: power off PHY after reset when interface is down | Date | Fri, 16 Dec 2011 02:29:14 +0000 |
| |
Modified the patch.
From a74d3114934a82d17a43eae7b5bb1488d2c1b6c8 Mon Sep 17 00:00:00 2001 From: Jiang Wang <Jiang.Wang@riverbed.com> Date: Thu, 15 Dec 2011 17:44:20 -0800 Subject: [PATCH 2/2] e100: power off PHY after reset when interface is down
PHYs supported by e100 re-starts auto-negotiation after writing to BMCR_RESET bit. This patch powers down PHY when the interface is down and reset is issued.
Signed-off-by: Jiang Wang <Jiang.Wang@riverbed.com> --- drivers/net/ethernet/intel/e100.c | 11 +++++++++++ 1 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/drivers/net/ethernet/intel/e100.c b/drivers/net/ethernet/intel/e100.c index ca994da..2764cdd 100644 --- a/drivers/net/ethernet/intel/e100.c +++ b/drivers/net/ethernet/intel/e100.c @@ -589,6 +589,7 @@ struct nic { multicast_all = (1 << 2), wol_magic = (1 << 3), ich_10h_workaround = (1 << 4), + powered_off = (1 << 5), } flags ____cacheline_aligned; enum mac mac; @@ -1454,6 +1455,8 @@ static int e100_phy_init(struct nic *nic) phy_data &= ~BMCR_PDOWN; mdio_write(nic->netdev, nic->mii.phy_id, MII_BMCR, phy_data); + nic->flags &= ~powered_off; + /* Get phy ID */ id_lo = mdio_read(netdev, nic->mii.phy_id, MII_PHYSID1); id_hi = mdio_read(netdev, nic->mii.phy_id, MII_PHYSID2); @@ -2273,6 +2276,7 @@ static void e100_down(struct nic *nic) phy_data = mdio_read(nic->netdev, nic->mii.phy_id, MII_BMCR); phy_data |= BMCR_PDOWN; mdio_write(nic->netdev, nic->mii.phy_id, MII_BMCR, phy_data); + nic->flags |= powered_off; } free_irq(nic->pdev->irq, nic->netdev); @@ -2383,6 +2387,13 @@ static int e100_set_settings(struct net_device *netdev, struct ethtool_cmd *cmd) err = mii_ethtool_sset(&nic->mii, cmd); e100_exec_cb(nic, NULL, e100_configure); + if (nic->flags & powered_off) { + uint16_t phy_data; + phy_data = mdio_read(nic->netdev, nic->mii.phy_id, MII_BMCR); + phy_data |= BMCR_PDOWN; + mdio_write(nic->netdev, nic->mii.phy_id, MII_BMCR, phy_data); + } + return err; } -- 1.7.1
------------------------------------- Jiang Wang Member of Technical Staff Riverbed Technology Tel: (408) 522-5109 Email: Jiang.Wang@riverbed.com www.riverbed.com
-----Original Message----- From: Jiang Wang [mailto:Jiang.Wang@riverbed.com] Sent: Tuesday, December 13, 2011 6:50 PM To: Jeff Kirsher; Jesse Brandeburg; Bruce Allan; Carolyn Wyborny; Don Skidmore; Greg Rose; Peter P Waskiewicz Jr; Alex Duyck; John Ronciak; e1000-devel@lists.sourceforge.net; netdev@vger.kernel.org; linux-kernel@vger.kernel.org Cc: Prasanna Panchamukhi; Chaitanya Lala; Francis St. Amant; miles.ito@intel.com; Jiang Wang Subject: [PATCH 2/2] e100: power off PHY after reset when interface is down PHYs supported by e100 re-starts auto-negotiation after writing to BMCR_RESET bit. This patch powers down PHY when the interface is down and reset is issued.
Signed-off-by: Jiang Wang <Jiang.Wang@riverbed.com> --- drivers/net/ethernet/intel/e100.c | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/drivers/net/ethernet/intel/e100.c b/drivers/net/ethernet/intel/e100.c index 9824e0a..b8e4910 100644 --- a/drivers/net/ethernet/intel/e100.c +++ b/drivers/net/ethernet/intel/e100.c @@ -2386,6 +2386,13 @@ static int e100_set_settings(struct net_device *netdev, struct ethtool_cmd *cmd) err = mii_ethtool_sset(&nic->mii, cmd); e100_exec_cb(nic, NULL, e100_configure); + if (!netif_running(netdev) && !(nic->flags & wol_magic)) { + uint16_t phy_data; + phy_data = mdio_read(nic->netdev, nic->mii.phy_id, MII_BMCR); + phy_data |= BMCR_PDOWN; + mdio_write(nic->netdev, nic->mii.phy_id, MII_BMCR, phy_data); + } + return err; } -- 1.7.1
| |