On Thu, Jan 22, 2009 at 04:04,  <[email protected]> wrote:
> Modified: trunk/drivers/net/bfin_mac.c (6031 => 6032)
>
>       /* search for connect PHY device */
> -     for (i = 0; i < PHY_MAX_ADDR; i++) {
> +     for (i = PHY_MAX_ADDR - 1; i >= 0; i--) {

why did you change it from searching 0...31 to 31...0 ?  the vast
majority of boards out there will be sitting at address 0, not 31.
this incurs unnecessary overhead and looks like a hack rather than a
real fix.
-mike
_______________________________________________
Linux-kernel-commits mailing list
[email protected]
https://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits

Reply via email to