diff options
author | Johannes Berg <johannes.berg@intel.com> | 2014-10-08 03:57:29 -0400 |
---|---|---|
committer | Emmanuel Grumbach <emmanuel.grumbach@intel.com> | 2014-10-29 06:52:28 -0400 |
commit | f85e9d19d451b9cc3b0381a9422320f9b630441c (patch) | |
tree | 4eb31471974db6dcbb5286fd18c31e2810911d62 /drivers/net/wireless/iwlwifi | |
parent | ee00aed175e64c570e9da7052e39cdcc4ec822af (diff) |
iwlwifi: mvm: improve MCS rate warning
The warning gives no information about the frame, and presents
the flags so that one might think they're the frame. Clarify
and add more information.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Diffstat (limited to 'drivers/net/wireless/iwlwifi')
-rw-r--r-- | drivers/net/wireless/iwlwifi/mvm/tx.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/net/wireless/iwlwifi/mvm/tx.c b/drivers/net/wireless/iwlwifi/mvm/tx.c index c6a517c771df..8d848735cdb8 100644 --- a/drivers/net/wireless/iwlwifi/mvm/tx.c +++ b/drivers/net/wireless/iwlwifi/mvm/tx.c | |||
@@ -189,8 +189,10 @@ static void iwl_mvm_set_tx_cmd_rate(struct iwl_mvm *mvm, | |||
189 | 189 | ||
190 | /* HT rate doesn't make sense for a non data frame */ | 190 | /* HT rate doesn't make sense for a non data frame */ |
191 | WARN_ONCE(info->control.rates[0].flags & IEEE80211_TX_RC_MCS, | 191 | WARN_ONCE(info->control.rates[0].flags & IEEE80211_TX_RC_MCS, |
192 | "Got an HT rate for a non data frame 0x%x\n", | 192 | "Got an HT rate (flags:0x%x/mcs:%d) for a non data frame (fc:0x%x)\n", |
193 | info->control.rates[0].flags); | 193 | info->control.rates[0].flags, |
194 | info->control.rates[0].idx, | ||
195 | le16_to_cpu(fc)); | ||
194 | 196 | ||
195 | rate_idx = info->control.rates[0].idx; | 197 | rate_idx = info->control.rates[0].idx; |
196 | /* if the rate isn't a well known legacy rate, take the lowest one */ | 198 | /* if the rate isn't a well known legacy rate, take the lowest one */ |