]> err.no Git - linux-2.6/commitdiff
[PATCH] enp2611: use 'dev' in link status timer
authorLennert Buytenhek <buytenh@wantstofly.org>
Wed, 23 Nov 2005 11:49:35 +0000 (12:49 +0100)
committerJeff Garzik <jgarzik@pobox.com>
Thu, 1 Dec 2005 07:25:26 +0000 (02:25 -0500)
We assign nds[i] to a local variable 'dev', which we never use
afterwards.  Use the local variable instead.

Signed-off-by: Lennert Buytenhek <buytenh@wantstofly.org>
Signed-off-by: Jeff Garzik <jgarzik@pobox.com>
drivers/net/ixp2000/enp2611.c

index f0cc23d423ef143a3d8192aad47007617032d892..bc29dbeec31811c05f80b266bc63aa8bc5d19447 100644 (file)
@@ -151,12 +151,12 @@ static void enp2611_check_link_status(unsigned long __dummy)
                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);
                }