diff options
| author | Erik Stromdahl <erik.stromdahl@gmail.com> | 2018-05-06 15:25:00 +0200 |
|---|---|---|
| committer | Kalle Valo <kvalo@codeaurora.org> | 2018-05-12 11:03:08 +0200 |
| commit | e3148cc5fecf60dcbd07e5c9cae987976d25cb17 (patch) | |
| tree | ed0f7884929fe2a73bc81f1001d00a2d485c626a /drivers/net/wireless/ath/ath10k/wmi.c | |
| parent | ath10k: snoc: Remove owner assignment from platform_driver (diff) | |
| download | linux-e3148cc5fecf60dcbd07e5c9cae987976d25cb17.tar.xz linux-e3148cc5fecf60dcbd07e5c9cae987976d25cb17.zip | |
ath10k: fix return value check in wake_tx_q op
ath10k_mac_tx_push_txq returns either a postive integer (length) on
success or a negative error code on error.
The "if (ret) break;" statement will thus always break out of the loop
immediately after ath10k_mac_tx_push_txq has returned (making the loop
pointless).
A side effect of this fix is that we will iterate the queue until
ath10k_mac_tx_push_txq returns -ENOENT. This will make sure the queue is
not added back to ar->txqs when it is empty. This could potentially
improve performance somewhat (I have seen a small improvement with SDIO
devices).
Signed-off-by: Erik Stromdahl <erik.stromdahl@gmail.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Diffstat (limited to 'drivers/net/wireless/ath/ath10k/wmi.c')
0 files changed, 0 insertions, 0 deletions