wcn36xx: Set default BTLE coexistence config
Subsystems (3)
PATCH v4 1/1 wcn36xx: Set default BTLE coexistence config
From: Eyal Ilsar <[email protected]>
If the value for the firmware configuration parameters BTC_STATIC_LEN_LE_BT and BTC_STATIC_LEN_LE_WLAN are not set the duty cycle between BT and WLAN is such that if BT (including BLE) is active WLAN gets 0 bandwidth. When tuning these parameters having a too high value for WLAN means that BLE performance degrades.
The "sweet" point of roughly half of the maximal values was empirically found to achieve a balance between BLE and Wi-Fi coexistence performance.
--- drivers/net/wireless/ath/wcn36xx/smd.c | 2 ++ 1 file changed, 2 insertions(+)
drivers/net/wireless/ath/wcn36xx/smd.c +2 -0
@@ -73,6 +73,8 @@ static struct wcn36xx_cfg_val wcn36xx_cfg_vals[] = {WCN36XX_CFG_VAL (TX_PWR_CTRL_ENABLE ,1 ),WCN36XX_CFG_VAL (ENABLE_CLOSE_LOOP ,1 ),WCN36XX_CFG_VAL (ENABLE_LPWR_IMG_TRANSITION ,0 ), +WCN36XX_CFG_VAL (BTC_STATIC_LEN_LE_BT ,120000 ), +WCN36XX_CFG_VAL (BTC_STATIC_LEN_LE_WLAN ,30000 ),WCN36XX_CFG_VAL (MAX_ASSOC_LIMIT ,10 ), WCN36XX_CFG_VAL(ENABLE_MCC_ADAPTIVE_SCHEDULER, 0), };
Signature · 2 lines
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
Quoted 11 lines · Ramon Fried
From: Eyal Ilsar <[email protected]>
If the value for the firmware configuration parameters
BTC_STATIC_LEN_LE_BTandBTC_STATIC_LEN_LE_WLANare not set the duty cycle between BT and WLAN is such that if BT (including BLE) is active WLAN gets 0 bandwidth. When tuning these parameters having a too high value for WLAN means that BLE performance degrades.
The "sweet" point of roughly half of the maximal values was empirically found to achieve a balance between BLE and Wi-Fi coexistence performance.
Quoted 2 lines
Signed-off-by Eyal Ilsar <[email protected]>Signed-off-by Ramon Fried <[email protected]>
Looks good,
Regards,
Bjorn
Quoted 21 lines
--- drivers/net/wireless/ath/wcn36xx/smd.c | 2 ++ 1 file changed, 2 insertions(+)drivers/net/wireless/ath/wcn36xx/smd.c +2
-0@@ -73,6 +73,8 @@ static struct wcn36xx_cfg_val wcn36xx_cfg_vals[] = {WCN36XX_CFG_VAL (TX_PWR_CTRL_ENABLE ,1 ),WCN36XX_CFG_VAL (ENABLE_CLOSE_LOOP ,1 ),WCN36XX_CFG_VAL (ENABLE_LPWR_IMG_TRANSITION ,0 ), +WCN36XX_CFG_VAL (BTC_STATIC_LEN_LE_BT ,120000 ), +WCN36XX_CFG_VAL (BTC_STATIC_LEN_LE_WLAN ,30000 ),WCN36XX_CFG_VAL (MAX_ASSOC_LIMIT ,10 ), WCN36XX_CFG_VAL(ENABLE_MCC_ADAPTIVE_SCHEDULER, 0), };Signature · 2 lines
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
Quoted 9 lines · Ramon Fried
If the value for the firmware configuration parameters
BTC_STATIC_LEN_LE_BTandBTC_STATIC_LEN_LE_WLANare not set the duty cycle between BT and WLAN is such that if BT (including BLE) is active WLAN gets 0 bandwidth. When tuning these parameters having a too high value for WLAN means that BLE performance degrades.
The "sweet" point of roughly half of the maximal values was empirically found to achieve a balance between BLE and Wi-Fi coexistence performance.
Quoted 4 lines
Signed-off-by Eyal Ilsar <[email protected]>Signed-off-by Ramon Fried <[email protected]>Acked-by Bjorn Andersson <[email protected]>Signed-off-by Kalle Valo <[email protected]>
Patch applied to ath-next branch of ath.git, thanks.
4119b6160a35 wcn36xx: set default BTLE coexistence config
Signature · 3 lines
https://patchwork.kernel.org/patch/10060833/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches