mirror of https://gitee.com/openkylin/linux.git
ath10k: replace hardcoded constant with define
The hardcoded values used in ath10k_mac_tx_push_pending and ath10k_mac_op_wake_tx_queue set an upper limit of how many packets that can be consumed from the TX queue. HTC_HOST_MAX_MSG_PER_TX_BUNDLE is a proper name for this constant, as the value effectively limits the number of messages that can be consumed in one step. Thus, the value is an upper limit of the number of messages that can be added to a TX message bundle. Signed-off-by: Erik Stromdahl <erik.stromdahl@gmail.com> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
This commit is contained in:
parent
ab687de953
commit
e4568eac04
|
@ -51,6 +51,7 @@ struct ath10k;
|
||||||
*/
|
*/
|
||||||
|
|
||||||
#define HTC_HOST_MAX_MSG_PER_RX_BUNDLE 8
|
#define HTC_HOST_MAX_MSG_PER_RX_BUNDLE 8
|
||||||
|
#define HTC_HOST_MAX_MSG_PER_TX_BUNDLE 16
|
||||||
|
|
||||||
enum ath10k_htc_tx_flags {
|
enum ath10k_htc_tx_flags {
|
||||||
ATH10K_HTC_FLAG_NEED_CREDIT_UPDATE = 0x01,
|
ATH10K_HTC_FLAG_NEED_CREDIT_UPDATE = 0x01,
|
||||||
|
|
|
@ -4026,7 +4026,7 @@ void ath10k_mac_tx_push_pending(struct ath10k *ar)
|
||||||
drv_priv);
|
drv_priv);
|
||||||
|
|
||||||
/* Prevent aggressive sta/tid taking over tx queue */
|
/* Prevent aggressive sta/tid taking over tx queue */
|
||||||
max = 16;
|
max = HTC_HOST_MAX_MSG_PER_TX_BUNDLE;
|
||||||
ret = 0;
|
ret = 0;
|
||||||
while (ath10k_mac_tx_can_push(hw, txq) && max--) {
|
while (ath10k_mac_tx_can_push(hw, txq) && max--) {
|
||||||
ret = ath10k_mac_tx_push_txq(hw, txq);
|
ret = ath10k_mac_tx_push_txq(hw, txq);
|
||||||
|
@ -4288,7 +4288,7 @@ static void ath10k_mac_op_wake_tx_queue(struct ieee80211_hw *hw,
|
||||||
struct ieee80211_txq *f_txq;
|
struct ieee80211_txq *f_txq;
|
||||||
struct ath10k_txq *f_artxq;
|
struct ath10k_txq *f_artxq;
|
||||||
int ret = 0;
|
int ret = 0;
|
||||||
int max = 16;
|
int max = HTC_HOST_MAX_MSG_PER_TX_BUNDLE;
|
||||||
|
|
||||||
spin_lock_bh(&ar->txqs_lock);
|
spin_lock_bh(&ar->txqs_lock);
|
||||||
if (list_empty(&artxq->list))
|
if (list_empty(&artxq->list))
|
||||||
|
|
Loading…
Reference in New Issue