summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWolfgang Denk <wd@denx.de>2011-09-28 21:02:43 +0200
committerWolfgang Denk <wd@denx.de>2011-09-28 21:02:43 +0200
commit0dcfb0fcb80d6c6615ce1a78ae78c858a9b1f236 (patch)
tree6fdf00f645dc245e46452fff6eec49fcffe8f25d
parentff4dea038371eb5ba9f9fa63a07748247647312a (diff)
downloadu-boot-imx-0dcfb0fcb80d6c6615ce1a78ae78c858a9b1f236.zip
u-boot-imx-0dcfb0fcb80d6c6615ce1a78ae78c858a9b1f236.tar.gz
u-boot-imx-0dcfb0fcb80d6c6615ce1a78ae78c858a9b1f236.tar.bz2
Revert "phylib: remove a couple of redundant code lines"
This reverts commit 041c542219af7f31c372d89b4c7c6f4c8064a8ce. The lines removed by this commit weren't redundant. The logic is (and probably should be better commented): Find the intersection of the advertised capabilities of both sides of the link (lpa). From that intersection, find the highest capability we can run at (that will be the negotiated link). Now imagine that the intersection (lpa) is (LPA_100HALF | LPA_10FULL). The code will now set phydev->speed to 100, and phydev->duplex to 1, but this link does not support 100FULL. Kudos to Andy Fleming <afleming@gmail.com> for binging this to attention and for the explanation. Signed-off-by: Wolfgang Denk <wd@denx.de>
-rw-r--r--drivers/net/phy/phy.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c
index 833a051..8da7688 100644
--- a/drivers/net/phy/phy.c
+++ b/drivers/net/phy/phy.c
@@ -318,10 +318,13 @@ static int genphy_parse_link(struct phy_device *phydev)
lpa = phy_read(phydev, MDIO_DEVAD_NONE, MII_ADVERTISE);
lpa &= phy_read(phydev, MDIO_DEVAD_NONE, MII_LPA);
- if (lpa & (LPA_100FULL | LPA_100HALF))
+ if (lpa & (LPA_100FULL | LPA_100HALF)) {
phydev->speed = SPEED_100;
- if (lpa & (LPA_100FULL | LPA_10FULL))
+ if (lpa & LPA_100FULL)
+ phydev->duplex = DUPLEX_FULL;
+
+ } else if (lpa & LPA_10FULL)
phydev->duplex = DUPLEX_FULL;
} else {
u32 bmcr = phy_read(phydev, MDIO_DEVAD_NONE, MII_BMCR);