Signed-off-by: hpey...@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);

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to