* [PATCH 0/1] wcn36xx: Revert firmware link monitoring
@ 2021-10-22 14:10 Bryan O'Donoghue
2021-10-22 14:10 ` [PATCH 1/1] Revert "wcn36xx: Enable firmware link monitoring" Bryan O'Donoghue
2021-10-25 8:52 ` [PATCH 0/1] wcn36xx: Revert firmware link monitoring Kalle Valo
0 siblings, 2 replies; 7+ messages in thread
From: Bryan O'Donoghue @ 2021-10-22 14:10 UTC (permalink / raw)
To: kvalo, linux-wireless, wcn36xx
Cc: loic.poulain, benl, daniel.thompson, bryan.odonoghue
We can get this working with
- Secure APs PS on/off
- Open APs PS on
by setting firmware feature bit WLANACTIVE_OFFLOAD
Open APs PS off though is non-functional even with the above bit enabled
and three quaters of a wheel is useless.
So unfortunately for now zap it off.
Bryan O'Donoghue (1):
Revert "wcn36xx: Enable firmware link monitoring"
drivers/net/wireless/ath/wcn36xx/main.c | 1 -
1 file changed, 1 deletion(-)
--
2.33.0
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/1] Revert "wcn36xx: Enable firmware link monitoring"
2021-10-22 14:10 [PATCH 0/1] wcn36xx: Revert firmware link monitoring Bryan O'Donoghue
@ 2021-10-22 14:10 ` Bryan O'Donoghue
2021-10-22 15:05 ` Loic Poulain
2021-10-25 8:55 ` Kalle Valo
2021-10-25 8:52 ` [PATCH 0/1] wcn36xx: Revert firmware link monitoring Kalle Valo
1 sibling, 2 replies; 7+ messages in thread
From: Bryan O'Donoghue @ 2021-10-22 14:10 UTC (permalink / raw)
To: kvalo, linux-wireless, wcn36xx
Cc: loic.poulain, benl, daniel.thompson, bryan.odonoghue
This reverts commit c973fdad79f6eaf247d48b5fc77733e989eb01e1.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Signed-off-by: Benjamin Li <benl@squareup.com>
---
drivers/net/wireless/ath/wcn36xx/main.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/wireless/ath/wcn36xx/main.c b/drivers/net/wireless/ath/wcn36xx/main.c
index dd8810fd76a3d..fd8b2753da7dd 100644
--- a/drivers/net/wireless/ath/wcn36xx/main.c
+++ b/drivers/net/wireless/ath/wcn36xx/main.c
@@ -1334,7 +1334,6 @@ static int wcn36xx_init_ieee80211(struct wcn36xx *wcn)
ieee80211_hw_set(wcn->hw, HAS_RATE_CONTROL);
ieee80211_hw_set(wcn->hw, SINGLE_SCAN_ON_ALL_BANDS);
ieee80211_hw_set(wcn->hw, REPORTS_TX_ACK_STATUS);
- ieee80211_hw_set(wcn->hw, CONNECTION_MONITOR);
wcn->hw->wiphy->interface_modes = BIT(NL80211_IFTYPE_STATION) |
BIT(NL80211_IFTYPE_AP) |
--
2.33.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/1] Revert "wcn36xx: Enable firmware link monitoring"
2021-10-22 14:10 ` [PATCH 1/1] Revert "wcn36xx: Enable firmware link monitoring" Bryan O'Donoghue
@ 2021-10-22 15:05 ` Loic Poulain
2021-10-25 8:54 ` Kalle Valo
2021-10-25 8:55 ` Kalle Valo
1 sibling, 1 reply; 7+ messages in thread
From: Loic Poulain @ 2021-10-22 15:05 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: Kalle Valo, linux-wireless, wcn36xx, Benjamin Li, Daniel Thompson
On Fri, 22 Oct 2021 at 16:08, Bryan O'Donoghue
<bryan.odonoghue@linaro.org> wrote:
>
> This reverts commit c973fdad79f6eaf247d48b5fc77733e989eb01e1.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> Signed-off-by: Benjamin Li <benl@squareup.com>
Should it get a 'Fixes' tag?
Regards,
Loic
> ---
> drivers/net/wireless/ath/wcn36xx/main.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/net/wireless/ath/wcn36xx/main.c b/drivers/net/wireless/ath/wcn36xx/main.c
> index dd8810fd76a3d..fd8b2753da7dd 100644
> --- a/drivers/net/wireless/ath/wcn36xx/main.c
> +++ b/drivers/net/wireless/ath/wcn36xx/main.c
> @@ -1334,7 +1334,6 @@ static int wcn36xx_init_ieee80211(struct wcn36xx *wcn)
> ieee80211_hw_set(wcn->hw, HAS_RATE_CONTROL);
> ieee80211_hw_set(wcn->hw, SINGLE_SCAN_ON_ALL_BANDS);
> ieee80211_hw_set(wcn->hw, REPORTS_TX_ACK_STATUS);
> - ieee80211_hw_set(wcn->hw, CONNECTION_MONITOR);
>
> wcn->hw->wiphy->interface_modes = BIT(NL80211_IFTYPE_STATION) |
> BIT(NL80211_IFTYPE_AP) |
> --
> 2.33.0
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/1] wcn36xx: Revert firmware link monitoring
2021-10-22 14:10 [PATCH 0/1] wcn36xx: Revert firmware link monitoring Bryan O'Donoghue
2021-10-22 14:10 ` [PATCH 1/1] Revert "wcn36xx: Enable firmware link monitoring" Bryan O'Donoghue
@ 2021-10-25 8:52 ` Kalle Valo
2021-10-25 9:07 ` Bryan O'Donoghue
1 sibling, 1 reply; 7+ messages in thread
From: Kalle Valo @ 2021-10-25 8:52 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: linux-wireless, wcn36xx, loic.poulain, benl, daniel.thompson
Bryan O'Donoghue <bryan.odonoghue@linaro.org> writes:
> We can get this working with
>
> - Secure APs PS on/off
> - Open APs PS on
>
> by setting firmware feature bit WLANACTIVE_OFFLOAD
>
> Open APs PS off though is non-functional even with the above bit enabled
> and three quaters of a wheel is useless.
>
> So unfortunately for now zap it off.
Avoid using acronyms as much as possible, PS meaning power save might
not be clear for everyone.
And please try to be more specific, I'm not sure what power save feature
you are meaning here. (802.11 protocol power save, BMPS or what?)
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/1] Revert "wcn36xx: Enable firmware link monitoring"
2021-10-22 15:05 ` Loic Poulain
@ 2021-10-25 8:54 ` Kalle Valo
0 siblings, 0 replies; 7+ messages in thread
From: Kalle Valo @ 2021-10-25 8:54 UTC (permalink / raw)
To: Loic Poulain
Cc: Bryan O'Donoghue, linux-wireless, wcn36xx, Benjamin Li,
Daniel Thompson
Loic Poulain <loic.poulain@linaro.org> writes:
> On Fri, 22 Oct 2021 at 16:08, Bryan O'Donoghue
> <bryan.odonoghue@linaro.org> wrote:
>>
>> This reverts commit c973fdad79f6eaf247d48b5fc77733e989eb01e1.
>>
>> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
>> Signed-off-by: Benjamin Li <benl@squareup.com>
>
> Should it get a 'Fixes' tag?
But this patch is not really fixing anything, right? I wonder if there's
patch 2 missing?
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/1] Revert "wcn36xx: Enable firmware link monitoring"
2021-10-22 14:10 ` [PATCH 1/1] Revert "wcn36xx: Enable firmware link monitoring" Bryan O'Donoghue
2021-10-22 15:05 ` Loic Poulain
@ 2021-10-25 8:55 ` Kalle Valo
1 sibling, 0 replies; 7+ messages in thread
From: Kalle Valo @ 2021-10-25 8:55 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: linux-wireless, wcn36xx, loic.poulain, benl, daniel.thompson
Bryan O'Donoghue <bryan.odonoghue@linaro.org> writes:
> This reverts commit c973fdad79f6eaf247d48b5fc77733e989eb01e1.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
The commit log should answer to question "why?". Please move the
explanation from the cover email to the commit log.
> Signed-off-by: Benjamin Li <benl@squareup.com>
Why Benjamin's s-o-b?
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/1] wcn36xx: Revert firmware link monitoring
2021-10-25 8:52 ` [PATCH 0/1] wcn36xx: Revert firmware link monitoring Kalle Valo
@ 2021-10-25 9:07 ` Bryan O'Donoghue
0 siblings, 0 replies; 7+ messages in thread
From: Bryan O'Donoghue @ 2021-10-25 9:07 UTC (permalink / raw)
To: Kalle Valo; +Cc: linux-wireless, wcn36xx, loic.poulain, benl, daniel.thompson
On 25/10/2021 09:52, Kalle Valo wrote:
> Avoid using acronyms as much as possible, PS meaning power save might
> not be clear for everyone.
>
> And please try to be more specific, I'm not sure what power save feature
> you are meaning here. (802.11 protocol power save, BMPS or what?)
Yep - the issue is on an Open AP when we enter BMPS link monitoring
stops working.
I'll make that clear in a resend
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-10-25 9:05 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-22 14:10 [PATCH 0/1] wcn36xx: Revert firmware link monitoring Bryan O'Donoghue
2021-10-22 14:10 ` [PATCH 1/1] Revert "wcn36xx: Enable firmware link monitoring" Bryan O'Donoghue
2021-10-22 15:05 ` Loic Poulain
2021-10-25 8:54 ` Kalle Valo
2021-10-25 8:55 ` Kalle Valo
2021-10-25 8:52 ` [PATCH 0/1] wcn36xx: Revert firmware link monitoring Kalle Valo
2021-10-25 9:07 ` Bryan O'Donoghue
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).