We assign nds[i] to a local variable 'dev', which we never use afterwards. Use the local variable instead.
Signed-off-by: Lennert Buytenhek <[EMAIL PROTECTED]> diff -urN linux-2.6.15-rc1.upstream.orig/drivers/net/ixp2000/enp2611.c linux-2.6.15-rc1.upstream/drivers/net/ixp2000/enp2611.c --- linux-2.6.15-rc1.upstream.orig/drivers/net/ixp2000/enp2611.c 2005-11-22 12:45:15.000000000 +0100 +++ linux-2.6.15-rc1.upstream/drivers/net/ixp2000/enp2611.c 2005-11-22 12:45:44.000000000 +0100 @@ -151,12 +151,12 @@ dev = nds[i]; status = pm3386_is_link_up(i); - if (status && !netif_carrier_ok(nds[i])) { + if (status && !netif_carrier_ok(dev)) { pm3386_enable_tx(i); caleb_enable_tx(i); - netif_carrier_on(nds[i]); - } else if (!status && netif_carrier_ok(nds[i])) { - netif_carrier_off(nds[i]); + netif_carrier_on(dev); + } else if (!status && netif_carrier_ok(dev)) { + netif_carrier_off(dev); caleb_disable_tx(i); pm3386_disable_tx(i); } - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html