| Message ID | 1461272982-7233-1-git-send-email-bjorn.andersson@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Delegated to: | Kalle Valo |
| Headers | show |
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 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Bjorn Andersson <bjorn.andersson@linaro.org> writes: > 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> Applied, thanks.
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 */
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(-)