On Mon, Dec 19, 2005 at 10:09:32AM -0500, John W. Linville wrote: > > > @@ -633,7 +643,7 @@ static int __devinit sundance_probe1 (st > > > > > > np->phys[0] = 1; /* Default setting */ > > > np->mii_preamble_required++; > > > - for (phy = 1; phy <= 32 && phy_idx < MII_CNT; phy++) { > > > + for (phy = 0; phy < 32 && phy_idx < MII_CNT; phy++) { > > > int mii_status = mdio_read(dev, phy, MII_BMSR); > > > int phyx = phy & 0x1f; > > > if (mii_status != 0xffff && mii_status != 0x0000) { > > > > (Your PHY is at address 0?) Can you add some debug here to see what > > happens in both cases (f.e. print the returned MII_BMSR values for > > both 'start at 0' and 'start at 1')? Presumably there's something > > about starting at 1 that gets your hardware confused, I'd like to know > > what that is.. > > How about if you just ditch that hunk?
Sorry, I should have mentioned: Arnaldo told me (on IRC) that it doesn't detect the transceiver without this hunk. --L - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html