wifi: iwlwifi: mvm: d3: fix WoWLAN command version lookup

After moving from commands to notificaitons in the d3 resume flow,
removing the WOWLAN_GET_STATUSES and REPLY_OFFLOADS_QUERY_CMD causes
the return of the default value when looking up their version.
Returning zero here results in the driver sending the not supported
NON_QOS_TX_COUNTER_CMD.

Signed-off-by: Yedidya Benshimol <yedidya.ben.shimol@intel.com>
Reviewed-by: Gregory Greenman <gregory.greenman@intel.com>
Signed-off-by: Miri Korenblit <miriam.rachel.korenblit@intel.com>
Link: https://msgid.link/20240510170500.8cabfd580614.If3a0db9851f56041f8f5360959354abd5379224a@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>

diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/d3.c b/drivers/net/wireless/intel/iwlwifi/mvm/d3.c
index 71e6b06481a93b6bf6f6eb14b1b1656510cdbc92..e37ae34acba60686b7991824202d9cf92cfd5b31 100644 (file)
--- a/drivers/net/wireless/intel/iwlwifi/mvm/d3.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/d3.c
@@ -2341,7 +2341,8 @@ static bool iwl_mvm_setup_connection_keep(struct iwl_mvm *mvm,
 
 out:
        if (iwl_fw_lookup_notif_ver(mvm->fw, LONG_GROUP,
-                                   WOWLAN_GET_STATUSES, 0) < 10) {
+                                   WOWLAN_GET_STATUSES,
+                                   IWL_FW_CMD_VER_UNKNOWN) < 10) {
                mvmvif->seqno_valid = true;
                /* +0x10 because the set API expects next-to-use, not last-used */
                mvmvif->seqno = status->non_qos_seq_ctr + 0x10;