In article <[EMAIL PROTECTED]> (at Tue, 27 Dec 2005 02:35:52 -0200), Arnaldo 
Carvalho de Melo <[EMAIL PROTECTED]> says:

> diff --git a/drivers/net/sundance.c b/drivers/net/sundance.c
> index 0ab9c38..8cdeb5c 100644
> --- a/drivers/net/sundance.c
> +++ b/drivers/net/sundance.c
> @@ -633,9 +633,13 @@ static int __devinit sundance_probe1 (st
>  
>       np->phys[0] = 1;                /* Default setting */
>       np->mii_preamble_required++;
> +     /*
> +      * It seems some phys doesn't deal well with address 0 being accessed
> +      * first, so leave address zero to the end of the loop (32 & 31).
> +      */
>       for (phy = 1; phy <= 32 && phy_idx < MII_CNT; phy++) {
> -             int mii_status = mdio_read(dev, phy, MII_BMSR);
>               int phyx = phy & 0x1f;
> +             int mii_status = mdio_read(dev, phyx, MII_BMSR);
>               if (mii_status != 0xffff  &&  mii_status != 0x0000) {
>                       np->phys[phy_idx++] = phyx;
>                       np->mii_if.advertising = mdio_read(dev, phyx, 
> MII_ADVERTISE);

Difference changeset, maybe?

--yoshfuji
-
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

Reply via email to