Skip to content

Commit

Permalink
[PATCH] enp2611: use 'dev' in link status timer
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
buytenh authored and jgarzik committed Dec 1, 2005
1 parent de287fd commit c44185d
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions drivers/net/ixp2000/enp2611.c
Original file line number Diff line number Diff line change
Expand Up @@ -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);
}
Expand Down

0 comments on commit c44185d

Please sign in to comment.