From 6e3981004bc9090a5886de6f0b668af66f9463a6 Mon Sep 17 00:00:00 2001 From: Dmitry Tarnyagin Date: Tue, 4 Oct 2011 15:36:04 +0200 Subject: 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 Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/32978 Reviewed-by: QABUILD Reviewed-by: QATEST Reviewed-by: Janusz DZIEDZIC Tested-by: Bartosz MARKOWSKI Reviewed-by: Bartosz MARKOWSKI Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/33536 Reviewed-by: Philippe LANGLAIS --- drivers/staging/cw1200/txrx.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) 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) { -- cgit v1.2.3