diff options
| author | Sara Sharon <sara.sharon@intel.com> | 2015-08-20 14:12:58 +0300 |
|---|---|---|
| committer | Luciano Coelho <luciano.coelho@intel.com> | 2015-08-28 13:26:32 +0300 |
| commit | 62e004fe941d56cb70ea9148bb79cf6698751248 (patch) | |
| tree | 8bcfbfc3c8405d268d64381af04816f9ddcc0ccf /drivers/net/wireless/iwlwifi/mvm/rx.c | |
| parent | dbb04b0d29f884dbb6d0a52dd7877445d2e776f1 (diff) | |
| download | linux-62e004fe941d56cb70ea9148bb79cf6698751248.tar.xz | |
iwlwifi: mvm: fix statistics variables type
When receiving statistics notification there is a field of average energy.
This is defines as signed 8 bit, while FW refers to it as unsigned.
when the energy is higher than 127 this causes in iwl_mvm_stat_iterator
a wrong computation of the signal
int sig = -stats->general.beacon_filter_average_energy;
resulting in incorrect CQM event (change from low to high).
While at it - update the rest of the fields to the correct type.
Signed-off-by: Sara Sharon <sara.sharon@intel.com>
Signed-off-by: Luciano Coelho <luciano.coelho@intel.com>
Diffstat (limited to 'drivers/net/wireless/iwlwifi/mvm/rx.c')
| -rw-r--r-- | drivers/net/wireless/iwlwifi/mvm/rx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/iwlwifi/mvm/rx.c b/drivers/net/wireless/iwlwifi/mvm/rx.c index c37c10a423ce..2ad814df0b3c 100644 --- a/drivers/net/wireless/iwlwifi/mvm/rx.c +++ b/drivers/net/wireless/iwlwifi/mvm/rx.c @@ -459,7 +459,7 @@ static void iwl_mvm_update_rx_statistics(struct iwl_mvm *mvm, struct iwl_mvm_stat_data { struct iwl_mvm *mvm; __le32 mac_id; - __s8 beacon_filter_average_energy; + u8 beacon_filter_average_energy; struct mvm_statistics_general_v8 *general; }; |