| Message ID | 20191113233558.4040259-1-bjorn.andersson@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | b70b3a36ec33a2c8d3292f3b33fe2047a8f57b9a |
| Delegated to: | Kalle Valo |
| Headers | show |
| Series | ath10k: qmi: Sleep for a while before assigning MSA memory | expand |
Bjorn Andersson <bjorn.andersson@linaro.org> wrote: > Unless we sleep for a while before transitioning the MSA memory to WLAN > the MPSS.AT.4.0.c2-01184-SDM845_GEN_PACK-1 firmware triggers a security > violation fairly reliably. Unforutnately recovering from this failure > always results in the entire system freezing. > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > Signed-off-by: Kalle Valo <kvalo@codeaurora.org> Patch applied to ath-next branch of ath.git, thanks. b70b3a36ec33 ath10k: qmi: Sleep for a while before assigning MSA memory
diff --git a/drivers/net/wireless/ath/ath10k/qmi.c b/drivers/net/wireless/ath/ath10k/qmi.c index 637f83ef65f8..a0ba07b85362 100644 --- a/drivers/net/wireless/ath/ath10k/qmi.c +++ b/drivers/net/wireless/ath/ath10k/qmi.c @@ -773,6 +773,13 @@ static void ath10k_qmi_event_server_arrive(struct ath10k_qmi *qmi) if (ret) return; + /* + * HACK: sleep for a while inbetween receiving the msa info response + * and the XPU update to prevent SDM845 from crashing due to a security + * violation, when running MPSS.AT.4.0.c2-01184-SDM845_GEN_PACK-1. + */ + msleep(20); + ret = ath10k_qmi_setup_msa_permissions(qmi); if (ret) return;
Unless we sleep for a while before transitioning the MSA memory to WLAN the MPSS.AT.4.0.c2-01184-SDM845_GEN_PACK-1 firmware triggers a security violation fairly reliably. Unforutnately recovering from this failure always results in the entire system freezing. Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> --- drivers/net/wireless/ath/ath10k/qmi.c | 7 +++++++ 1 file changed, 7 insertions(+)