aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/net
diff options
context:
space:
mode:
authorTony Breeds <tony@bakeyournoodle.com>2009-04-16 19:27:23 -0400
committerDavid S. Miller <davem@davemloft.net>2009-04-16 19:27:23 -0400
commit2238aff5bbd9d928b69e579b4c95842f97da95ba (patch)
tree8420e3b1888c56c61bfdd1482f468dcdcdbfc180 /drivers/net
parent3664090e199f10cb0282097faae8f8ca58c1e4ae (diff)
ixgbe: Be explict with what we are !'ing in ixgbe_sfp_config_module_task()
GCC warns: drivers/net/ixgbe/ixgbe_main.c: In function 'ixgbe_sfp_config_module_task': drivers/net/ixgbe/ixgbe_main.c:3920: warning: suggest parantheses around operand of '!' or change '&' to '&&' or '!' to '~' Which I think is right. Bracket to remove ambiguity. Signed-off-by: Tony Breeds <tony@bakeyournoodle.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r--drivers/net/ixgbe/ixgbe_main.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ixgbe/ixgbe_main.c b/drivers/net/ixgbe/ixgbe_main.c
index febde45cf9fa..49a903784566 100644
--- a/drivers/net/ixgbe/ixgbe_main.c
+++ b/drivers/net/ixgbe/ixgbe_main.c
@@ -3946,7 +3946,7 @@ static void ixgbe_sfp_config_module_task(struct work_struct *work)
3946 } 3946 }
3947 hw->mac.ops.setup_sfp(hw); 3947 hw->mac.ops.setup_sfp(hw);
3948 3948
3949 if (!adapter->flags & IXGBE_FLAG_IN_SFP_LINK_TASK) 3949 if (!(adapter->flags & IXGBE_FLAG_IN_SFP_LINK_TASK))
3950 /* This will also work for DA Twinax connections */ 3950 /* This will also work for DA Twinax connections */
3951 schedule_work(&adapter->multispeed_fiber_task); 3951 schedule_work(&adapter->multispeed_fiber_task);
3952 adapter->flags &= ~IXGBE_FLAG_IN_SFP_MOD_TASK; 3952 adapter->flags &= ~IXGBE_FLAG_IN_SFP_MOD_TASK;