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

Reply via email to