Yes, it's a bug. Intention was 9.
Thanks.

-----Original Message-----
From: anamtsov@gmail.com [mailto:anamtsov@gmail.com] On Behalf Of
Arik Nemtsov
Sent: Monday, August 12, 2013 3:21 PM
To: Johannes Berg
Cc: linux-wireless@vger.kernel.org; Bondar, Alexander
Subject: Re: [PATCH 28/30] iwlwifi: mvm: Change beacon abort escape time
value

On Fri, Jul 26, 2013 at 11:28 AM, Johannes Berg [off-list ref]
wrote:

quoted

From: Alexander Bondar <redacted>

Set beacon abort escape timer values - 6 beacons in D0 state,
9 beacons in D3 and D0i3.

Signed-off-by: Alexander Bondar <redacted>
Reviewed-by: Emmanuel Grumbach <redacted>
Signed-off-by: Johannes Berg <redacted>
---
 drivers/net/wireless/iwlwifi/mvm/fw-api-power.h | 3 ++-
 drivers/net/wireless/iwlwifi/mvm/power.c        | 3 +++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/net/wireless/iwlwifi/mvm/fw-api-power.h

b/drivers/net/wireless/iwlwifi/mvm/fw-api-power.h
index 149347d..060e630 100644

--- a/drivers/net/wireless/iwlwifi/mvm/fw-api-power.h
+++ b/drivers/net/wireless/iwlwifi/mvm/fw-api-power.h

@@ -285,7 +285,8 @@ struct iwl_beacon_filter_cmd {  #define

IWL_BF_ESCAPE_TIMER_MAX 1024  #define IWL_BF_ESCAPE_TIMER_MIN

0

quoted

-#define IWL_BA_ESCAPE_TIMER_DEFAULT 3
+#define IWL_BA_ESCAPE_TIMER_DEFAULT 6 #define

IWL_BA_ESCAPE_TIMER_D3

quoted

+6

Did you mean 9 here?
(Sorry for the late review)

---------------------------------------------------------------------
A member of the Intel Corporation group of companies

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.