diff options
author | wangweidong <wangweidong1@huawei.com> | 2016-04-14 03:43:52 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-04-15 20:10:00 -0400 |
commit | f48256efededaa87f475c0d6330d83f853cb064a (patch) | |
tree | 9db641e98629cd41e1084a2d46417f720a06e035 /drivers/net/phy | |
parent | 4e811b1e11d5bc9e25ca1a30ce6654add2b571e3 (diff) |
phy: make some bits preserved while setup forced mode
When tested the PHY SGMII Loopback:
1.set the LOOPBACK bit,
2.set the autoneg to AUTONEG_DISABLE, it calls the
genphy_setup_forced which will clear the bit.
The BMCR_LOOPBACK bit should be preserved.
As Florian pointed out that other bits should be preserved too.
So I make the BMCR_ISOLATE and BMCR_PDOWN as well.
Signed-off-by: Weidong Wang <wangweidong1@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/phy')
-rw-r--r-- | drivers/net/phy/phy_device.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/phy/phy_device.c b/drivers/net/phy/phy_device.c index e551f3a89cfd..10e39c2fbf81 100644 --- a/drivers/net/phy/phy_device.c +++ b/drivers/net/phy/phy_device.c | |||
@@ -1123,8 +1123,9 @@ static int genphy_config_advert(struct phy_device *phydev) | |||
1123 | */ | 1123 | */ |
1124 | int genphy_setup_forced(struct phy_device *phydev) | 1124 | int genphy_setup_forced(struct phy_device *phydev) |
1125 | { | 1125 | { |
1126 | int ctl = 0; | 1126 | int ctl = phy_read(phydev, MII_BMCR); |
1127 | 1127 | ||
1128 | ctl &= BMCR_LOOPBACK | BMCR_ISOLATE | BMCR_PDOWN; | ||
1128 | phydev->pause = 0; | 1129 | phydev->pause = 0; |
1129 | phydev->asym_pause = 0; | 1130 | phydev->asym_pause = 0; |
1130 | 1131 | ||