From: Bjorn Andersson <bjorn.andersson@linaro.org> To: Amit Pundir <amit.pundir@linaro.org> Cc: Kalle Valo <kvalo@codeaurora.org>, David S Miller <davem@davemloft.net>, Jakub Kicinski <kuba@kernel.org>, Jeffrey Hugo <jeffrey.l.hugo@gmail.com>, John Stultz <john.stultz@linaro.org>, Sumit Semwal <sumit.semwal@linaro.org>, Konrad Dybcio <konradybcio@gmail.com>, ath10k@lists.infradead.org, linux-wireless@vger.kernel.org, netdev@vger.kernel.org, lkml <linux-kernel@vger.kernel.org> Subject: Re: [PATCH] ath10k: qmi: Skip host capability request for Xiaomi Poco F1 Date: Thu, 17 Sep 2020 11:05:13 -0500 [thread overview] Message-ID: <20200917160513.GO1893@yoga> (raw) In-Reply-To: <1600328501-8832-1-git-send-email-amit.pundir@linaro.org> On Thu 17 Sep 02:41 CDT 2020, Amit Pundir wrote: > Workaround to get WiFi working on Xiaomi Poco F1 (sdm845) > phone. We get a non-fatal QMI_ERR_MALFORMED_MSG_V01 error > message in ath10k_qmi_host_cap_send_sync(), but we can still > bring up WiFi services successfully on AOSP if we ignore it. > > We suspect either the host cap is not implemented or there > may be firmware specific issues. Firmware version is > QC_IMAGE_VERSION_STRING=WLAN.HL.2.0.c3-00257-QCAHLSWMTPLZ-1 > > qcom,snoc-host-cap-8bit-quirk didn't help. If I use this > quirk, then the host capability request does get accepted, > but we run into fatal "msa info req rejected" error and > WiFi interface doesn't come up. > What happens if you skip sending the host-cap message? I had one firmware version for which I implemented a "qcom,snoc-host-cap-skip-quirk". But testing showed that the link was pretty unusable - pushing any real amount of data would cause it to silently stop working - and I realized that I could use the linux-firmware wlanmdsp.mbn instead, which works great on all my devices... > Attempts are being made to debug the failure reasons but no > luck so far. Hence this device specific workaround instead > of checking for QMI_ERR_MALFORMED_MSG_V01 error message. > Tried ath10k/WCN3990/hw1.0/wlanmdsp.mbn from the upstream > linux-firmware project but it didn't help and neither did > building board-2.bin file from stock bdwlan* files. > "Didn't work" as in the wlanmdsp.mbn from linux-firmware failed to load or some laer problem? Regards, Bjorn > This workaround will be removed once we have a viable fix. > Thanks to postmarketOS guys for catching this. > > Signed-off-by: Amit Pundir <amit.pundir@linaro.org> > --- > Device-tree for Xiaomi Poco F1(Beryllium) got merged in > qcom/arm64-for-5.10 last week > https://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git/commit/?id=77809cf74a8c > > drivers/net/wireless/ath/ath10k/qmi.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/ath/ath10k/qmi.c b/drivers/net/wireless/ath/ath10k/qmi.c > index 0dee1353d395..37c5350eb8b1 100644 > --- a/drivers/net/wireless/ath/ath10k/qmi.c > +++ b/drivers/net/wireless/ath/ath10k/qmi.c > @@ -651,7 +651,8 @@ static int ath10k_qmi_host_cap_send_sync(struct ath10k_qmi *qmi) > > /* older FW didn't support this request, which is not fatal */ > if (resp.resp.result != QMI_RESULT_SUCCESS_V01 && > - resp.resp.error != QMI_ERR_NOT_SUPPORTED_V01) { > + resp.resp.error != QMI_ERR_NOT_SUPPORTED_V01 && > + !of_machine_is_compatible("xiaomi,beryllium")) { /* Xiaomi Poco F1 workaround */ > ath10k_err(ar, "host capability request rejected: %d\n", resp.resp.error); > ret = -EINVAL; > goto out; > -- > 2.7.4 >
WARNING: multiple messages have this Message-ID (diff)
From: Bjorn Andersson <bjorn.andersson@linaro.org> To: Amit Pundir <amit.pundir@linaro.org> Cc: lkml <linux-kernel@vger.kernel.org>, Jeffrey Hugo <jeffrey.l.hugo@gmail.com>, netdev@vger.kernel.org, linux-wireless@vger.kernel.org, Konrad Dybcio <konradybcio@gmail.com>, ath10k@lists.infradead.org, David S Miller <davem@davemloft.net>, John Stultz <john.stultz@linaro.org>, Jakub Kicinski <kuba@kernel.org>, Sumit Semwal <sumit.semwal@linaro.org>, Kalle Valo <kvalo@codeaurora.org> Subject: Re: [PATCH] ath10k: qmi: Skip host capability request for Xiaomi Poco F1 Date: Thu, 17 Sep 2020 11:05:13 -0500 [thread overview] Message-ID: <20200917160513.GO1893@yoga> (raw) In-Reply-To: <1600328501-8832-1-git-send-email-amit.pundir@linaro.org> On Thu 17 Sep 02:41 CDT 2020, Amit Pundir wrote: > Workaround to get WiFi working on Xiaomi Poco F1 (sdm845) > phone. We get a non-fatal QMI_ERR_MALFORMED_MSG_V01 error > message in ath10k_qmi_host_cap_send_sync(), but we can still > bring up WiFi services successfully on AOSP if we ignore it. > > We suspect either the host cap is not implemented or there > may be firmware specific issues. Firmware version is > QC_IMAGE_VERSION_STRING=WLAN.HL.2.0.c3-00257-QCAHLSWMTPLZ-1 > > qcom,snoc-host-cap-8bit-quirk didn't help. If I use this > quirk, then the host capability request does get accepted, > but we run into fatal "msa info req rejected" error and > WiFi interface doesn't come up. > What happens if you skip sending the host-cap message? I had one firmware version for which I implemented a "qcom,snoc-host-cap-skip-quirk". But testing showed that the link was pretty unusable - pushing any real amount of data would cause it to silently stop working - and I realized that I could use the linux-firmware wlanmdsp.mbn instead, which works great on all my devices... > Attempts are being made to debug the failure reasons but no > luck so far. Hence this device specific workaround instead > of checking for QMI_ERR_MALFORMED_MSG_V01 error message. > Tried ath10k/WCN3990/hw1.0/wlanmdsp.mbn from the upstream > linux-firmware project but it didn't help and neither did > building board-2.bin file from stock bdwlan* files. > "Didn't work" as in the wlanmdsp.mbn from linux-firmware failed to load or some laer problem? Regards, Bjorn > This workaround will be removed once we have a viable fix. > Thanks to postmarketOS guys for catching this. > > Signed-off-by: Amit Pundir <amit.pundir@linaro.org> > --- > Device-tree for Xiaomi Poco F1(Beryllium) got merged in > qcom/arm64-for-5.10 last week > https://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git/commit/?id=77809cf74a8c > > drivers/net/wireless/ath/ath10k/qmi.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/ath/ath10k/qmi.c b/drivers/net/wireless/ath/ath10k/qmi.c > index 0dee1353d395..37c5350eb8b1 100644 > --- a/drivers/net/wireless/ath/ath10k/qmi.c > +++ b/drivers/net/wireless/ath/ath10k/qmi.c > @@ -651,7 +651,8 @@ static int ath10k_qmi_host_cap_send_sync(struct ath10k_qmi *qmi) > > /* older FW didn't support this request, which is not fatal */ > if (resp.resp.result != QMI_RESULT_SUCCESS_V01 && > - resp.resp.error != QMI_ERR_NOT_SUPPORTED_V01) { > + resp.resp.error != QMI_ERR_NOT_SUPPORTED_V01 && > + !of_machine_is_compatible("xiaomi,beryllium")) { /* Xiaomi Poco F1 workaround */ > ath10k_err(ar, "host capability request rejected: %d\n", resp.resp.error); > ret = -EINVAL; > goto out; > -- > 2.7.4 > _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k
next prev parent reply other threads:[~2020-09-17 16:23 UTC|newest] Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-09-17 7:41 [PATCH] ath10k: qmi: Skip host capability request for Xiaomi Poco F1 Amit Pundir 2020-09-17 7:41 ` Amit Pundir 2020-09-17 16:05 ` Bjorn Andersson [this message] 2020-09-17 16:05 ` Bjorn Andersson 2020-09-21 10:38 ` Amit Pundir 2020-09-21 10:38 ` Amit Pundir 2020-09-25 15:22 ` Bjorn Andersson 2020-09-25 15:22 ` Bjorn Andersson 2020-09-24 16:31 ` Kalle Valo 2020-09-24 16:31 ` Kalle Valo 2020-09-25 15:27 ` Bjorn Andersson 2020-09-25 15:27 ` Bjorn Andersson 2020-09-25 18:32 ` Amit Pundir 2020-09-25 18:32 ` Amit Pundir 2020-09-29 8:29 ` Kalle Valo 2020-09-29 8:29 ` Kalle Valo
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20200917160513.GO1893@yoga \ --to=bjorn.andersson@linaro.org \ --cc=amit.pundir@linaro.org \ --cc=ath10k@lists.infradead.org \ --cc=davem@davemloft.net \ --cc=jeffrey.l.hugo@gmail.com \ --cc=john.stultz@linaro.org \ --cc=konradybcio@gmail.com \ --cc=kuba@kernel.org \ --cc=kvalo@codeaurora.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-wireless@vger.kernel.org \ --cc=netdev@vger.kernel.org \ --cc=sumit.semwal@linaro.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.