Skip to content

Commit

Permalink
PHY probe not working properly for ibm_emac (PPC4xx)
Browse files Browse the repository at this point in the history
I have a system with AMCC PowerPC 405EP and PHY Intel LXT971A.  Linux
2.6.18.3 is not able to detect the PHY ID correctly.  The PHY ID
detected is 0, but should be 0x1d.

This is because phy_read() (__emac_mdio_read() resp.) from
drivers/net/ibm_emac/ibm_emac_core.c might return -ETIMEDOUT or
-EREMOTEIO on error.  This is ignored inside the

int mii_phy_probe(struct mii_phy *phy, int address)
from drivers/net/ibm_emac/ibm_emac_phy.c

as the return value is assigned to an u32 variable.

Cc: Jeff Garzik <jeff@garzik.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
  • Loading branch information
Hynek Petrak authored and Jeff Garzik committed Dec 26, 2006
1 parent ebf5112 commit 3e65bb9
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion drivers/net/ibm_emac/ibm_emac_phy.c
Original file line number Diff line number Diff line change
Expand Up @@ -309,7 +309,7 @@ int mii_phy_probe(struct mii_phy *phy, int address)
{
struct mii_phy_def *def;
int i;
u32 id;
int id;

phy->autoneg = AUTONEG_DISABLE;
phy->advertising = 0;
Expand All @@ -324,6 +324,8 @@ int mii_phy_probe(struct mii_phy *phy, int address)

/* Read ID and find matching entry */
id = (phy_read(phy, MII_PHYSID1) << 16) | phy_read(phy, MII_PHYSID2);
if (id < 0)
return -ENODEV;
for (i = 0; (def = mii_phy_table[i]) != NULL; i++)
if ((id & def->phy_id_mask) == def->phy_id)
break;
Expand Down

0 comments on commit 3e65bb9

Please sign in to comment.