diff options
| author | Dmitry Tarnyagin <dmitry.tarnyagin@stericsson.com> | 2011-10-04 15:36:04 +0200 |
|---|---|---|
| committer | Philippe LANGLAIS <philippe.langlais@stericsson.com> | 2011-10-13 10:21:14 +0200 |
| commit | 6e3981004bc9090a5886de6f0b668af66f9463a6 (patch) | |
| tree | 240fd4493f619fde6d74e8afb99aa67f54b01cf9 | |
| parent | cc9e650c1357a56636ec08e88b3271a5a93c6ecc (diff) | |
cw1200: Fix for incorrect retry statistics reporting.
In case of WSM_STATUS_RETRY_EXCEEDED TX status firmware reports
not actual number of retransmission attempts, but the number minus one.
Inaccurate reporting slightly affected rate control algorithm.
ST-Ericsson ID: 363511
Change-Id: I9ad216435b76e7e995d905deda4234bdf7ae3f3c
Signed-off-by: Dmitry Tarnyagin <dmitry.tarnyagin@stericsson.com>
Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/32978
Reviewed-by: QABUILD
Reviewed-by: QATEST
Reviewed-by: Janusz DZIEDZIC <janusz.dziedzic@tieto.com>
Tested-by: Bartosz MARKOWSKI <bartosz.markowski@tieto.com>
Reviewed-by: Bartosz MARKOWSKI <bartosz.markowski@tieto.com>
Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/33536
Reviewed-by: Philippe LANGLAIS <philippe.langlais@stericsson.com>
| -rw-r--r-- | drivers/staging/cw1200/txrx.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/staging/cw1200/txrx.c b/drivers/staging/cw1200/txrx.c index 92a5bf377b6..b0028ae81dc 100644 --- a/drivers/staging/cw1200/txrx.c +++ b/drivers/staging/cw1200/txrx.c @@ -741,7 +741,8 @@ void cw1200_tx_confirm_cb(struct cw1200_common *priv, struct sk_buff *skb; int tid = CW1200_MAX_TID; - txrx_printk(KERN_DEBUG "[TX] TX confirm.\n"); + txrx_printk(KERN_DEBUG "[TX] TX confirm: %d, %d.\n", + arg->status, arg->ackFailures); if (unlikely(priv->mode == NL80211_IFTYPE_UNSPECIFIED)) { /* STA is stopped. */ @@ -806,6 +807,8 @@ void cw1200_tx_confirm_cb(struct cw1200_common *priv, queue_work(priv->workqueue, &priv->tx_failure_work); } + if (tx_count) + ++tx_count; } for (i = 0; i < IEEE80211_TX_MAX_RATES; ++i) { |