From 8c207396ff49a385228003f5c7f17107ebfa7fdf Mon Sep 17 00:00:00 2001 From: Vignesh Viswanathan Date: Wed, 12 Sep 2018 12:54:21 +0530 Subject: [PATCH] qcacld-3.0: Set gAutoBmpsTimerValue to 30 seconds by default Currently gAutoBmpsTimerValue is set to defaut value of 5 seconds so that after 5 seconds from association completion, the device would go to power save. However, this would lead to issues if an ADDTS req is received from the supplicant. The ADDTS req would be sent to the AP but the AP would not send the ADDTS response immeidately and would buffer it as the device is in power sleep. Ideal way is for the framework to control the device power save where the framework should push the device to power save after connection and should disable power save before sending an ADDTS request. However, to avoid cases where the framework fails to send the power cfg ops to the host after connection leading the FW to the stuck in full power mode, setting the gAutoBmpsTimerValue to 30 seconds by default. Change-Id: Idc8af26174a56cd1afee3c7a4353b4d7ebeac2d7 CRs-Fixed: 2313913 --- core/hdd/inc/wlan_hdd_cfg.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/core/hdd/inc/wlan_hdd_cfg.h b/core/hdd/inc/wlan_hdd_cfg.h index 08925691e9..f18b3995bb 100644 --- a/core/hdd/inc/wlan_hdd_cfg.h +++ b/core/hdd/inc/wlan_hdd_cfg.h @@ -9729,7 +9729,7 @@ enum hdd_wext_control { * gAutoBmpsTimerValue - Set Auto BMPS Timer value * @Min: 0 * @Max: 120 - * @Default: 5 + * @Default: 30 * * This ini is used to set Auto BMPS Timer value in seconds * @@ -9744,7 +9744,7 @@ enum hdd_wext_control { #define CFG_AUTO_PS_ENABLE_TIMER_NAME "gAutoBmpsTimerValue" #define CFG_AUTO_PS_ENABLE_TIMER_MIN (0) #define CFG_AUTO_PS_ENABLE_TIMER_MAX (120) -#define CFG_AUTO_PS_ENABLE_TIMER_DEFAULT (5) +#define CFG_AUTO_PS_ENABLE_TIMER_DEFAULT (30) #ifdef WLAN_ICMP_DISABLE_PS /*