iwlwifi: wake from runtime suspend before sending sync commands

If a host command was queued while in runtime suspend, it would go out
before the D0I3_END_CMD was sent.  Sometimes it works, but sometimes
it fails, and it is obviously the wrong thing to do.

To fix this, have the opmode take a reference before sending a SYNC
command and make the pcie trans wait for the runtime state to become
active before actually queueing the command.

Signed-off-by: Luca Coelho <luciano.coelho@intel.com>

1 files changed, 7 insertions, 1 deletions

diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/utils.c b/drivers/net/wireless/intel/iwlwifi/mvm/utils.c
index f0ffd62f02d3..eb41d3bd8059 100644
--- a/drivers/net/wireless/intel/iwlwifi/mvm/utils.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/utils.c
@@ -90,11 +90,17 @@ int iwl_mvm_send_cmd(struct iwl_mvm *mvm, struct iwl_host_cmd *cmd)
          * the mutex, this ensures we don't try to send two
          * (or more) synchronous commands at a time.
          */
-        if (!(cmd->flags & CMD_ASYNC))
+        if (!(cmd->flags & CMD_ASYNC)) {
                 lockdep_assert_held(&mvm->mutex);
+                if (!(cmd->flags & CMD_SEND_IN_IDLE))
+                        iwl_mvm_ref(mvm, IWL_MVM_REF_SENDING_CMD);
+        }
 
         ret = iwl_trans_send_cmd(mvm->trans, cmd);
 
+        if (!(cmd->flags & (CMD_ASYNC | CMD_SEND_IN_IDLE)))
+                iwl_mvm_unref(mvm, IWL_MVM_REF_SENDING_CMD);
+
         /*
          * If the caller wants the SKB, then don't hide any problems, the
          * caller might access the response buffer which will be NULL if