From: Andy Green <andy.green@linaro.org> To: Bjorn Andersson <bjorn@kryo.se> Cc: Kalle Valo <kvalo@codeaurora.org>, Eugene Krasnikov <k.eugene.e@gmail.com>, wcn36xx <wcn36xx@lists.infradead.org>, linux-wireless <linux-wireless@vger.kernel.org>, netdev <netdev@vger.kernel.org> Subject: Re: [PATCH 6/7] net: wireless: wcn36xx: remove powersaving for wcn3620 Date: Tue, 10 Feb 2015 05:07:28 +0800 [thread overview] Message-ID: <CAAfg0W6dPupGLgwDF5FtNqEFvC4Y+GvFBAmXWcUQwXr2y6KugQ@mail.gmail.com> (raw) In-Reply-To: <CAJAp7OhcFTuYfc7JOKW1SrVRbucuSWg5jcFaoVnSDgBxsyo2ng@mail.gmail.com> On 10 February 2015 at 01:54, Bjorn Andersson <bjorn@kryo.se> wrote: > On Sat, Jan 17, 2015 at 9:11 PM, Andy Green <andy.green@linaro.org> wrote: >> WCN3620 powersaving mode is not stable. Disable it if we're >> on a wcn3620 chip type. >> >> Signed-off-by: Andy Green <andy.green@linaro.org> >> --- >> drivers/net/wireless/ath/wcn36xx/main.c | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/drivers/net/wireless/ath/wcn36xx/main.c b/drivers/net/wireless/ath/wcn36xx/main.c >> index c4178c7..569d45b 100644 >> --- a/drivers/net/wireless/ath/wcn36xx/main.c >> +++ b/drivers/net/wireless/ath/wcn36xx/main.c >> @@ -924,6 +924,10 @@ static int wcn36xx_init_ieee80211(struct wcn36xx *wcn) >> IEEE80211_HW_AMPDU_AGGREGATION | >> IEEE80211_HW_TIMING_BEACON_ONLY; >> >> + /* 3620 powersaving currently unstable */ >> + if (wcn->chip_version == WCN36XX_CHIP_3620) >> + wcn->hw->flags &= ~IEEE80211_HW_SUPPORTS_PS; >> + > > Hi Andy, > > I have the same problem (Data Abort Exception in wcnss) on 3680, this > with my wip smd code and a hacked up remoteproc-pil. I haven't spent > any effort on debugging this, but it looks like it's something related > to what we have ported to mainline (or lack thereof) rather than a > 3620 specific issue. Makes sense since I only have 3620, I assumed it's related to that. However he can work for a while with powersaving on, but ping latencies are +600-800ms and he's shaky. Later I found something mac80211 generic scan preparation or post-scan code (for scan initiated by wpa_supplicant) is able to stop wlan traffic after a few goes even if the actual scan mode smd is not sent. At that point I think a nice solution would be a donation of time from guys who specialize in wcn for a living to come and hand out a pony or two... -Andy > Regards, > Bjorn
WARNING: multiple messages have this Message-ID (diff)
From: Andy Green <andy.green-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> To: Bjorn Andersson <bjorn-UYDU3/A3LUY@public.gmane.org> Cc: Kalle Valo <kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>, Eugene Krasnikov <k.eugene.e-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, wcn36xx <wcn36xx-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>, linux-wireless <linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, netdev <netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> Subject: Re: [PATCH 6/7] net: wireless: wcn36xx: remove powersaving for wcn3620 Date: Tue, 10 Feb 2015 05:07:28 +0800 [thread overview] Message-ID: <CAAfg0W6dPupGLgwDF5FtNqEFvC4Y+GvFBAmXWcUQwXr2y6KugQ@mail.gmail.com> (raw) In-Reply-To: <CAJAp7OhcFTuYfc7JOKW1SrVRbucuSWg5jcFaoVnSDgBxsyo2ng-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> On 10 February 2015 at 01:54, Bjorn Andersson <bjorn-UYDU3/A3LUY@public.gmane.org> wrote: > On Sat, Jan 17, 2015 at 9:11 PM, Andy Green <andy.green-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote: >> WCN3620 powersaving mode is not stable. Disable it if we're >> on a wcn3620 chip type. >> >> Signed-off-by: Andy Green <andy.green-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> >> --- >> drivers/net/wireless/ath/wcn36xx/main.c | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/drivers/net/wireless/ath/wcn36xx/main.c b/drivers/net/wireless/ath/wcn36xx/main.c >> index c4178c7..569d45b 100644 >> --- a/drivers/net/wireless/ath/wcn36xx/main.c >> +++ b/drivers/net/wireless/ath/wcn36xx/main.c >> @@ -924,6 +924,10 @@ static int wcn36xx_init_ieee80211(struct wcn36xx *wcn) >> IEEE80211_HW_AMPDU_AGGREGATION | >> IEEE80211_HW_TIMING_BEACON_ONLY; >> >> + /* 3620 powersaving currently unstable */ >> + if (wcn->chip_version == WCN36XX_CHIP_3620) >> + wcn->hw->flags &= ~IEEE80211_HW_SUPPORTS_PS; >> + > > Hi Andy, > > I have the same problem (Data Abort Exception in wcnss) on 3680, this > with my wip smd code and a hacked up remoteproc-pil. I haven't spent > any effort on debugging this, but it looks like it's something related > to what we have ported to mainline (or lack thereof) rather than a > 3620 specific issue. Makes sense since I only have 3620, I assumed it's related to that. However he can work for a while with powersaving on, but ping latencies are +600-800ms and he's shaky. Later I found something mac80211 generic scan preparation or post-scan code (for scan initiated by wpa_supplicant) is able to stop wlan traffic after a few goes even if the actual scan mode smd is not sent. At that point I think a nice solution would be a donation of time from guys who specialize in wcn for a living to come and hand out a pony or two... -Andy > Regards, > Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-02-09 21:07 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-01-18 5:10 [PATCH 0/7] net: wireless: wcn36xx: add basic wcn3620 support Andy Green 2015-01-18 5:10 ` Andy Green 2015-01-18 5:10 ` [PATCH 1/7] net: wireless: wcn36xx: add wcn3620 chip type definition Andy Green 2015-01-18 5:10 ` Andy Green 2015-01-18 14:17 ` Kalle Valo 2015-01-19 0:24 ` Andy Green 2015-01-23 15:39 ` Kalle Valo 2015-01-23 15:39 ` Kalle Valo 2015-01-18 5:10 ` [PATCH 2/7] net: wireless: wcn36xx: get chip type from platform ops Andy Green 2015-01-18 8:36 ` Pat Erley 2015-01-18 8:36 ` Pat Erley 2015-01-18 5:10 ` [PATCH 3/7] net: wireless: wcn36xx: use 3680 dxe regs for 3620 Andy Green 2015-01-18 5:11 ` [PATCH 4/7] net: wireless: wcn36xx: introduce WCN36XX_HAL_AVOID_FREQ_RANGE_IND Andy Green 2015-01-27 20:01 ` Eugene Krasnikov 2015-01-27 20:57 ` Andy Green 2015-01-18 5:11 ` [PATCH 5/7] net: wireless: wcn36xx: swallow two wcn3620 IND messages Andy Green 2015-01-18 11:47 ` Sergei Shtylyov 2015-01-18 11:47 ` Sergei Shtylyov 2015-01-18 12:59 ` Andy Green 2015-01-18 5:11 ` [PATCH 6/7] net: wireless: wcn36xx: remove powersaving for wcn3620 Andy Green 2015-02-09 17:54 ` Bjorn Andersson 2015-02-09 17:54 ` Bjorn Andersson 2015-02-09 21:07 ` Andy Green [this message] 2015-02-09 21:07 ` Andy Green [not found] ` <CAJAp7OjLDCzLYeb3LHNs1sOHWM64XZWzr46UQ2vNirH_2JibNg@mail.gmail.com> 2015-02-09 21:28 ` Andy Green 2015-02-09 21:28 ` Andy Green 2015-02-09 21:40 ` Bjorn Andersson 2015-02-09 22:01 ` Andy Green 2015-01-18 5:11 ` [PATCH 7/7] net: wireless: wcn36xx: handle new trigger_ba format Andy Green 2015-01-27 19:58 ` Eugene Krasnikov
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=CAAfg0W6dPupGLgwDF5FtNqEFvC4Y+GvFBAmXWcUQwXr2y6KugQ@mail.gmail.com \ --to=andy.green@linaro.org \ --cc=bjorn@kryo.se \ --cc=k.eugene.e@gmail.com \ --cc=kvalo@codeaurora.org \ --cc=linux-wireless@vger.kernel.org \ --cc=netdev@vger.kernel.org \ --cc=wcn36xx@lists.infradead.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.