iwlwifi: mvm: fix WGDS table print in iwl_mvm_chub_update_mcc()
authorLuca Coelho <luciano.coelho@intel.com>
Sun, 24 Oct 2021 15:20:33 +0000 (18:20 +0300)
committerLuca Coelho <luciano.coelho@intel.com>
Thu, 28 Oct 2021 09:04:11 +0000 (12:04 +0300)
WGDS table index 0 means disabled, but we were erroneously checking
for < 0 to print that it is disabled.  Fix that and make the print
more readable by mentioning that it's either disabled or there was an
error.

Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
Link: https://lore.kernel.org/r/iwlwifi.20211024181719.98a5572bf0f8.I6c112ca80cf427f12b2c752899d293cb6437ba5f@changeid
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
drivers/net/wireless/intel/iwlwifi/mvm/nvm.c

index da705fcaf0fcc726760175dc421c240a8d9c5164..6d18a1fd649b90e52006f7869663541e7320360a 100644 (file)
@@ -583,8 +583,9 @@ void iwl_mvm_rx_chub_update_mcc(struct iwl_mvm *mvm,
                return;
 
        wgds_tbl_idx = iwl_mvm_get_sar_geo_profile(mvm);
-       if (wgds_tbl_idx < 0)
-               IWL_DEBUG_INFO(mvm, "SAR WGDS is disabled (%d)\n",
+       if (wgds_tbl_idx < 1)
+               IWL_DEBUG_INFO(mvm,
+                              "SAR WGDS is disabled or error received (%d)\n",
                               wgds_tbl_idx);
        else
                IWL_DEBUG_INFO(mvm, "SAR WGDS: geo profile %d is configured\n",