diff options
author | Dmitry Tarnyagin <dmitry.tarnyagin@stericsson.com> | 2011-10-04 15:36:04 +0200 |
---|---|---|
committer | Philippe Langlais <philippe.langlais@stericsson.com> | 2011-12-06 11:00:34 +0100 |
commit | 7c6c20b6471e9a4b981b8bc820fb3693c93b3f36 (patch) | |
tree | 96341a20290f6710a8690f55886ee59afecd6ac4 /drivers | |
parent | f6070378982ec4ad526eff193b8aff1ef455ce6e (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>
Diffstat (limited to 'drivers')
-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) { |