diff options
author | Joe Perches <joe@perches.com> | 2014-10-27 01:25:03 -0400 |
---|---|---|
committer | Emmanuel Grumbach <emmanuel.grumbach@intel.com> | 2014-10-29 06:53:07 -0400 |
commit | 50f6635afe565a0e1c5ab78f040294fe1dc41de0 (patch) | |
tree | b8d932e23d842018f5c915cfaa9a068c8db3a926 /drivers/net | |
parent | 7ae83d00825b39347b9a591703cd7cc3163523fb (diff) |
iwlwifi: dvm: Fix probable mask then right shift defect
Precedence of & and >> is not the same and is not left to right.
shift has higher precedence and should be done after the mask.
Add parentheses around the mask.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/wireless/iwlwifi/dvm/lib.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/wireless/iwlwifi/dvm/lib.c b/drivers/net/wireless/iwlwifi/dvm/lib.c index 2191621d69c1..02e4ede2b042 100644 --- a/drivers/net/wireless/iwlwifi/dvm/lib.c +++ b/drivers/net/wireless/iwlwifi/dvm/lib.c | |||
@@ -418,8 +418,8 @@ void iwlagn_bt_adjust_rssi_monitor(struct iwl_priv *priv, bool rssi_ena) | |||
418 | 418 | ||
419 | static bool iwlagn_bt_traffic_is_sco(struct iwl_bt_uart_msg *uart_msg) | 419 | static bool iwlagn_bt_traffic_is_sco(struct iwl_bt_uart_msg *uart_msg) |
420 | { | 420 | { |
421 | return BT_UART_MSG_FRAME3SCOESCO_MSK & uart_msg->frame3 >> | 421 | return (BT_UART_MSG_FRAME3SCOESCO_MSK & uart_msg->frame3) >> |
422 | BT_UART_MSG_FRAME3SCOESCO_POS; | 422 | BT_UART_MSG_FRAME3SCOESCO_POS; |
423 | } | 423 | } |
424 | 424 | ||
425 | static void iwlagn_bt_traffic_change_work(struct work_struct *work) | 425 | static void iwlagn_bt_traffic_change_work(struct work_struct *work) |