diff mbox

wcn36xx: Set SMD timeout to 10 seconds

Message ID 1461272982-7233-1-git-send-email-bjorn.andersson@linaro.org
State Accepted
Commit 8527f68849caadd0b7c560c613f2e18c9e1f8d47
Headers show

Commit Message

Bjorn Andersson April 21, 2016, 9:09 p.m. UTC
After booting the wireless subsystem and uploading the NV blob to the
WCNSS_CTRL service the remote continues to do things and will not start
servicing wlan-requests for another 2-5 seconds (measured).

The downstream code does not have any special handling for this case,
but has a timeout of 10 seconds for the communication layer. By
extending the wcn36xx timeout to match this we follows the same flow for
the boot procedure and can successfully configure WiFi as wlan0 is
registered.

Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>

---
 drivers/net/wireless/ath/wcn36xx/smd.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.5.0

Comments

John Stultz April 21, 2016, 9:11 p.m. UTC | #1
On Thu, Apr 21, 2016 at 2:09 PM, Bjorn Andersson
<bjorn.andersson@linaro.org> wrote:
> After booting the wireless subsystem and uploading the NV blob to the

> WCNSS_CTRL service the remote continues to do things and will not start

> servicing wlan-requests for another 2-5 seconds (measured).

>

> The downstream code does not have any special handling for this case,

> but has a timeout of 10 seconds for the communication layer. By

> extending the wcn36xx timeout to match this we follows the same flow for

> the boot procedure and can successfully configure WiFi as wlan0 is

> registered.

>

> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>


I've been using this with my nexus7 tree, and its avoided issues I was
seeing without it.

Tested-by: John Stultz <john.stultz@linaro.org>


thanks
-john
diff mbox

Patch

diff --git a/drivers/net/wireless/ath/wcn36xx/smd.h b/drivers/net/wireless/ath/wcn36xx/smd.h
index e6aadd273c46..6310560901f0 100644
--- a/drivers/net/wireless/ath/wcn36xx/smd.h
+++ b/drivers/net/wireless/ath/wcn36xx/smd.h
@@ -24,7 +24,7 @@ 
 
 #define WCN36XX_HAL_BUF_SIZE				4096
 
-#define HAL_MSG_TIMEOUT 500
+#define HAL_MSG_TIMEOUT 10000
 #define WCN36XX_SMSM_WLAN_TX_ENABLE			0x00000400
 #define WCN36XX_SMSM_WLAN_TX_RINGS_EMPTY		0x00000200
 /* The PNO version info be contained in the rsp msg */