Dear Heiko Schocher, In message <1320819979-3861-1-git-send-email...@denx.de> you wrote: ... > + if (count < MAX_PHY) > + active_phy_addr[j++] = i; > + else { > + printf("%s: to much PHYs detected.\n", > + __func__); > + break; > + }
Coding Style: if the "else" branch uses braces, the "if" branch shall use braces, too. Also, please s/too much/too many/ in the printf() above. 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: w...@denx.de Be kind to unkind people - they need it the most. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot