diff options
author | Tony Breeds <tony@bakeyournoodle.com> | 2009-04-09 18:29:10 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-04-11 05:48:03 -0400 |
commit | 8d1c3c0746098bee8ad116073120166347f21719 (patch) | |
tree | 3ef0d293667c0b750c9fe587a897be80c57ed685 | |
parent | 04193058c1005551af93f04a4b975fbd7f95cad5 (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>
-rw-r--r-- | drivers/net/ixgbe/ixgbe_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ixgbe/ixgbe_main.c b/drivers/net/ixgbe/ixgbe_main.c index 2856486e1655..813c5bc1a8fa 100644 --- a/drivers/net/ixgbe/ixgbe_main.c +++ b/drivers/net/ixgbe/ixgbe_main.c | |||
@@ -3917,7 +3917,7 @@ static void ixgbe_sfp_config_module_task(struct work_struct *work) | |||
3917 | } | 3917 | } |
3918 | hw->mac.ops.setup_sfp(hw); | 3918 | hw->mac.ops.setup_sfp(hw); |
3919 | 3919 | ||
3920 | if (!adapter->flags & IXGBE_FLAG_IN_SFP_LINK_TASK) | 3920 | if (!(adapter->flags & IXGBE_FLAG_IN_SFP_LINK_TASK)) |
3921 | /* This will also work for DA Twinax connections */ | 3921 | /* This will also work for DA Twinax connections */ |
3922 | schedule_work(&adapter->multispeed_fiber_task); | 3922 | schedule_work(&adapter->multispeed_fiber_task); |
3923 | adapter->flags &= ~IXGBE_FLAG_IN_SFP_MOD_TASK; | 3923 | adapter->flags &= ~IXGBE_FLAG_IN_SFP_MOD_TASK; |