[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