OK with me, I tested this on various bge and bnx last time it was around.

On 2012/07/04 04:53, Brad Smith wrote:
> This was part of the bnx(4) jumbo diff but it should be commited
> separately as it pertains to bge(4) as well.
> 
> Correct the jumbo setup code within brgphy(4). The code had a mistake
> and was actually disabling the jumbo settings for the PHY.
> 
> 
> Index: brgphy.c
> ===================================================================
> RCS file: /home/cvs/src/sys/dev/mii/brgphy.c,v
> retrieving revision 1.94
> diff -u -p -r1.94 brgphy.c
> --- brgphy.c  19 Dec 2011 13:21:01 -0000      1.94
> +++ brgphy.c  4 Jul 2012 08:50:18 -0000
> @@ -1128,12 +1134,12 @@ brgphy_jumbo_settings(struct mii_softc *
>               PHY_WRITE(sc, BRGPHY_MII_AUXCTL, 0x7);
>               val = PHY_READ(sc, BRGPHY_MII_AUXCTL);
>               PHY_WRITE(sc, BRGPHY_MII_AUXCTL,
> -                     val & ~(BRGPHY_AUXCTL_LONG_PKT | 0x7));
> +                     val | BRGPHY_AUXCTL_LONG_PKT);
>       }
>  
>       val = PHY_READ(sc, BRGPHY_MII_PHY_EXTCTL);
>       PHY_WRITE(sc, BRGPHY_MII_PHY_EXTCTL,
> -             val & ~BRGPHY_PHY_EXTCTL_HIGH_LA);
> +             val | BRGPHY_PHY_EXTCTL_HIGH_LA);
>  }
>  
>  void
> 
> -- 
> This message has been scanned for viruses and
> dangerous content by MailScanner, and is
> believed to be clean.

Reply via email to