diff options
Diffstat (limited to 'drivers/net/wireless/mwifiex/cmdevt.c')
| -rw-r--r-- | drivers/net/wireless/mwifiex/cmdevt.c | 39 | 
1 files changed, 27 insertions, 12 deletions
diff --git a/drivers/net/wireless/mwifiex/cmdevt.c b/drivers/net/wireless/mwifiex/cmdevt.c index b5c8b962ce1..74db0d24a57 100644 --- a/drivers/net/wireless/mwifiex/cmdevt.c +++ b/drivers/net/wireless/mwifiex/cmdevt.c @@ -153,7 +153,7 @@ static int mwifiex_dnld_cmd_to_fw(struct mwifiex_private *priv,  			" or cmd size is 0, not sending\n");  		if (cmd_node->wait_q_enabled)  			adapter->cmd_wait_q.status = -1; -		mwifiex_insert_cmd_to_free_q(adapter, cmd_node); +		mwifiex_recycle_cmd_node(adapter, cmd_node);  		return -1;  	} @@ -167,7 +167,7 @@ static int mwifiex_dnld_cmd_to_fw(struct mwifiex_private *priv,  			"DNLD_CMD: FW in reset state, ignore cmd %#x\n",  			cmd_code);  		mwifiex_complete_cmd(adapter, cmd_node); -		mwifiex_insert_cmd_to_free_q(adapter, cmd_node); +		mwifiex_recycle_cmd_node(adapter, cmd_node);  		return -1;  	} @@ -228,7 +228,7 @@ static int mwifiex_dnld_cmd_to_fw(struct mwifiex_private *priv,  			adapter->cmd_sent = false;  		if (cmd_node->wait_q_enabled)  			adapter->cmd_wait_q.status = -1; -		mwifiex_insert_cmd_to_free_q(adapter, adapter->curr_cmd); +		mwifiex_recycle_cmd_node(adapter, adapter->curr_cmd);  		spin_lock_irqsave(&adapter->mwifiex_cmd_lock, flags);  		adapter->curr_cmd = NULL; @@ -250,7 +250,7 @@ static int mwifiex_dnld_cmd_to_fw(struct mwifiex_private *priv,  	/* Setup the timer after transmit command */  	mod_timer(&adapter->cmd_timer, -		  jiffies + (MWIFIEX_TIMER_10S * HZ) / 1000); +		  jiffies + msecs_to_jiffies(MWIFIEX_TIMER_10S));  	return 0;  } @@ -632,6 +632,20 @@ mwifiex_insert_cmd_to_free_q(struct mwifiex_adapter *adapter,  	spin_unlock_irqrestore(&adapter->cmd_free_q_lock, flags);  } +/* This function reuses a command node. */ +void mwifiex_recycle_cmd_node(struct mwifiex_adapter *adapter, +			      struct cmd_ctrl_node *cmd_node) +{ +	struct host_cmd_ds_command *host_cmd = (void *)cmd_node->cmd_skb->data; + +	mwifiex_insert_cmd_to_free_q(adapter, cmd_node); + +	atomic_dec(&adapter->cmd_pending); +	dev_dbg(adapter->dev, "cmd: FREE_CMD: cmd=%#x, cmd_pending=%d\n", +		le16_to_cpu(host_cmd->command), +		atomic_read(&adapter->cmd_pending)); +} +  /*   * This function queues a command to the command pending queue.   * @@ -673,7 +687,9 @@ mwifiex_insert_cmd_to_pending_q(struct mwifiex_adapter *adapter,  		list_add(&cmd_node->list, &adapter->cmd_pending_q);  	spin_unlock_irqrestore(&adapter->cmd_pending_q_lock, flags); -	dev_dbg(adapter->dev, "cmd: QUEUE_CMD: cmd=%#x is queued\n", command); +	atomic_inc(&adapter->cmd_pending); +	dev_dbg(adapter->dev, "cmd: QUEUE_CMD: cmd=%#x, cmd_pending=%d\n", +		command, atomic_read(&adapter->cmd_pending));  }  /* @@ -783,7 +799,7 @@ int mwifiex_process_cmdresp(struct mwifiex_adapter *adapter)  	if (adapter->curr_cmd->cmd_flag & CMD_F_CANCELED) {  		dev_err(adapter->dev, "CMD_RESP: %#x been canceled\n",  			le16_to_cpu(resp->command)); -		mwifiex_insert_cmd_to_free_q(adapter, adapter->curr_cmd); +		mwifiex_recycle_cmd_node(adapter, adapter->curr_cmd);  		spin_lock_irqsave(&adapter->mwifiex_cmd_lock, flags);  		adapter->curr_cmd = NULL;  		spin_unlock_irqrestore(&adapter->mwifiex_cmd_lock, flags); @@ -833,7 +849,7 @@ int mwifiex_process_cmdresp(struct mwifiex_adapter *adapter)  		if (adapter->curr_cmd->wait_q_enabled)  			adapter->cmd_wait_q.status = -1; -		mwifiex_insert_cmd_to_free_q(adapter, adapter->curr_cmd); +		mwifiex_recycle_cmd_node(adapter, adapter->curr_cmd);  		spin_lock_irqsave(&adapter->mwifiex_cmd_lock, flags);  		adapter->curr_cmd = NULL;  		spin_unlock_irqrestore(&adapter->mwifiex_cmd_lock, flags); @@ -865,8 +881,7 @@ int mwifiex_process_cmdresp(struct mwifiex_adapter *adapter)  		if (adapter->curr_cmd->wait_q_enabled)  			adapter->cmd_wait_q.status = ret; -		/* Clean up and put current command back to cmd_free_q */ -		mwifiex_insert_cmd_to_free_q(adapter, adapter->curr_cmd); +		mwifiex_recycle_cmd_node(adapter, adapter->curr_cmd);  		spin_lock_irqsave(&adapter->mwifiex_cmd_lock, flags);  		adapter->curr_cmd = NULL; @@ -993,7 +1008,7 @@ mwifiex_cancel_all_pending_cmd(struct mwifiex_adapter *adapter)  			mwifiex_complete_cmd(adapter, cmd_node);  			cmd_node->wait_q_enabled = false;  		} -		mwifiex_insert_cmd_to_free_q(adapter, cmd_node); +		mwifiex_recycle_cmd_node(adapter, cmd_node);  		spin_lock_irqsave(&adapter->cmd_pending_q_lock, flags);  	}  	spin_unlock_irqrestore(&adapter->cmd_pending_q_lock, flags); @@ -1040,7 +1055,7 @@ mwifiex_cancel_pending_ioctl(struct mwifiex_adapter *adapter)  		cmd_node = adapter->curr_cmd;  		cmd_node->wait_q_enabled = false;  		cmd_node->cmd_flag |= CMD_F_CANCELED; -		mwifiex_insert_cmd_to_free_q(adapter, cmd_node); +		mwifiex_recycle_cmd_node(adapter, cmd_node);  		mwifiex_complete_cmd(adapter, adapter->curr_cmd);  		adapter->curr_cmd = NULL;  		spin_unlock_irqrestore(&adapter->mwifiex_cmd_lock, cmd_flags); @@ -1149,7 +1164,7 @@ int mwifiex_ret_802_11_hs_cfg(struct mwifiex_private *priv,  			phs_cfg->params.hs_config.gpio,  			phs_cfg->params.hs_config.gap);  	} -	if (conditions != HOST_SLEEP_CFG_CANCEL) { +	if (conditions != HS_CFG_CANCEL) {  		adapter->is_hs_configured = true;  		if (adapter->iface_type == MWIFIEX_USB ||  		    adapter->iface_type == MWIFIEX_PCIE)  |