[U-Boot] [PATCH] PPC temac phy probe unsigned int underflow fix.

Wolfgang Denk wd at denx.de
Sun Sep 2 17:43:40 CEST 2012


Dear hpeyerl,

In message <574be49434caea48b4f5ed19536735b3 at mail.beer.org> you wrote:
> Signed-off-by: hpeyerl at beer.org (Herb Peyerl)
> 
> Apologies for the non-git formatted drive-by patch.
> 
> If, during phy detection, your phy does not respond, then 
> ll_temac_phy_addr() will underflow due to the declaration of local 
> variable 'phy' as an unsigned int.
> 
> 
> --- drivers/net/xilinx_ll_temac_mdio.c  11 Jul 2012 00:52:14 -0000      
> 1.1.1.1
> +++ drivers/net/xilinx_ll_temac_mdio.c  2 Aug 2012 16:23:48 -0000
> @@ -137,7 +137,7 @@
>   {
>          struct temac_reg *regs = (struct temac_reg *)bus->priv;
>          unsigned short val;
> -       unsigned int phy;
> +       int phy;
> 
>          for (phy = PHY_MAX_ADDR; phy >= 0; phy--) {
>                  val = ll_temac_local_mdio_read(regs, phy, 0, 
> PHY_DETECT_REG);

Sorry, this does not apply as your patch is white-space mangled (TABs
have been replaced by spaces, long lines wrapped).

Pleas resubmit after fixing you mailer configuration (or rather use
git send-email instead).

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Include the success of others in your dreams for your own success.


More information about the U-Boot mailing list