* [PATCH 1/2] dt-bindings: net: wireless: ath11k: add ieee80211-freq-limit property
@ 2024-04-12 16:24 Robert Marko
2024-04-12 16:24 ` [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit property to limit channels Robert Marko
2024-04-16 13:21 ` [PATCH 1/2] dt-bindings: net: wireless: ath11k: add ieee80211-freq-limit property Rob Herring
0 siblings, 2 replies; 9+ messages in thread
From: Robert Marko @ 2024-04-12 16:24 UTC (permalink / raw
To: kvalo, davem, edumazet, kuba, pabeni, robh,
krzysztof.kozlowski+dt, conor+dt, jjohnson, linux-wireless,
netdev, devicetree, ath11k, linux-kernel
Cc: Robert Marko
This is an existing optional property that ieee80211.yaml/cfg80211
provides. It's useful to further restrict supported frequencies
for a specified device through device-tree.
Signed-off-by: Robert Marko <robimarko@gmail.com>
---
.../devicetree/bindings/net/wireless/qcom,ath11k.yaml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
index 672282cdfc2f..907bbb646614 100644
--- a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
+++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
@@ -55,6 +55,8 @@ properties:
phandle to a node describing reserved memory (System RAM memory)
used by ath11k firmware (see bindings/reserved-memory/reserved-memory.txt)
+ ieee80211-freq-limit: true
+
iommus:
minItems: 1
maxItems: 2
@@ -88,6 +90,7 @@ required:
additionalProperties: false
allOf:
+ - $ref: ieee80211.yaml#
- if:
properties:
compatible:
--
2.44.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit property to limit channels
2024-04-12 16:24 [PATCH 1/2] dt-bindings: net: wireless: ath11k: add ieee80211-freq-limit property Robert Marko
@ 2024-04-12 16:24 ` Robert Marko
2024-04-12 19:52 ` Christian Lamparter
2024-04-16 13:21 ` [PATCH 1/2] dt-bindings: net: wireless: ath11k: add ieee80211-freq-limit property Rob Herring
1 sibling, 1 reply; 9+ messages in thread
From: Robert Marko @ 2024-04-12 16:24 UTC (permalink / raw
To: kvalo, davem, edumazet, kuba, pabeni, robh,
krzysztof.kozlowski+dt, conor+dt, jjohnson, linux-wireless,
netdev, devicetree, ath11k, linux-kernel
Cc: Robert Marko
The common DT property can be used to limit the available channels
but ath11k has to manually call wiphy_read_of_freq_limits().
Signed-off-by: Robert Marko <robimarko@gmail.com>
---
drivers/net/wireless/ath/ath11k/mac.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c
index c32be587000d..59bde128d351 100644
--- a/drivers/net/wireless/ath/ath11k/mac.c
+++ b/drivers/net/wireless/ath/ath11k/mac.c
@@ -10124,6 +10124,7 @@ static int __ath11k_mac_register(struct ath11k *ar)
if (ret)
goto err;
+ wiphy_read_of_freq_limits(ar->hw->wiphy);
ath11k_mac_setup_ht_vht_cap(ar, cap, &ht_cap);
ath11k_mac_setup_he_cap(ar, cap);
--
2.44.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit property to limit channels
2024-04-12 16:24 ` [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit property to limit channels Robert Marko
@ 2024-04-12 19:52 ` Christian Lamparter
2024-04-12 20:21 ` Robert Marko
2024-04-12 20:53 ` Jeff Johnson
0 siblings, 2 replies; 9+ messages in thread
From: Christian Lamparter @ 2024-04-12 19:52 UTC (permalink / raw
To: Robert Marko, kvalo, davem, edumazet, kuba, pabeni, robh,
krzysztof.kozlowski+dt, conor+dt, jjohnson, linux-wireless,
netdev, devicetree, ath11k, linux-kernel
On 4/12/24 6:24 PM, Robert Marko wrote:
> The common DT property can be used to limit the available channels
> but ath11k has to manually call wiphy_read_of_freq_limits().
>
> Signed-off-by: Robert Marko <robimarko@gmail.com>
I've seen this before.
https://patchwork.kernel.org/project/linux-wireless/patch/ed266944c721de8dbf0fe35f387a3a71b2c84037.1686486468.git.chunkeey@gmail.com/
(dt-binding too. it has/had an ack)
https://patchwork.kernel.org/project/linux-wireless/patch/fc606d2550d047a53b4289235dd3c0fe23d5daac.1686486468.git.chunkeey@gmail.com/
sooo.... this is awkward.
> ---
> drivers/net/wireless/ath/ath11k/mac.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c
> index c32be587000d..59bde128d351 100644
> --- a/drivers/net/wireless/ath/ath11k/mac.c
> +++ b/drivers/net/wireless/ath/ath11k/mac.c
> @@ -10124,6 +10124,7 @@ static int __ath11k_mac_register(struct ath11k *ar)
> if (ret)
> goto err;
>
> + wiphy_read_of_freq_limits(ar->hw->wiphy);
> ath11k_mac_setup_ht_vht_cap(ar, cap, &ht_cap);
> ath11k_mac_setup_he_cap(ar, cap);
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit property to limit channels
2024-04-12 19:52 ` Christian Lamparter
@ 2024-04-12 20:21 ` Robert Marko
2024-04-12 20:53 ` Jeff Johnson
1 sibling, 0 replies; 9+ messages in thread
From: Robert Marko @ 2024-04-12 20:21 UTC (permalink / raw
To: Christian Lamparter
Cc: kvalo, davem, edumazet, kuba, pabeni, robh,
krzysztof.kozlowski+dt, conor+dt, jjohnson, linux-wireless,
netdev, devicetree, ath11k, linux-kernel
On Fri, 12 Apr 2024 at 21:52, Christian Lamparter
<christian.lamparter@isd.uni-stuttgart.de> wrote:
>
> On 4/12/24 6:24 PM, Robert Marko wrote:
> > The common DT property can be used to limit the available channels
> > but ath11k has to manually call wiphy_read_of_freq_limits().
> >
> > Signed-off-by: Robert Marko <robimarko@gmail.com>
>
> I've seen this before.
>
> https://patchwork.kernel.org/project/linux-wireless/patch/ed266944c721de8dbf0fe35f387a3a71b2c84037.1686486468.git.chunkeey@gmail.com/
>
> (dt-binding too. it has/had an ack)
> https://patchwork.kernel.org/project/linux-wireless/patch/fc606d2550d047a53b4289235dd3c0fe23d5daac.1686486468.git.chunkeey@gmail.com/
>
> sooo.... this is awkward.
Definitively,
I completely missed your patch set and we needed this in OpenWrt.
You were, first so your patch set should be applied.
Regards,
Robert
>
> > ---
> > drivers/net/wireless/ath/ath11k/mac.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c
> > index c32be587000d..59bde128d351 100644
> > --- a/drivers/net/wireless/ath/ath11k/mac.c
> > +++ b/drivers/net/wireless/ath/ath11k/mac.c
> > @@ -10124,6 +10124,7 @@ static int __ath11k_mac_register(struct ath11k *ar)
> > if (ret)
> > goto err;
> >
> > + wiphy_read_of_freq_limits(ar->hw->wiphy);
> > ath11k_mac_setup_ht_vht_cap(ar, cap, &ht_cap);
> > ath11k_mac_setup_he_cap(ar, cap);
> >
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit property to limit channels
2024-04-12 19:52 ` Christian Lamparter
2024-04-12 20:21 ` Robert Marko
@ 2024-04-12 20:53 ` Jeff Johnson
2024-04-15 16:06 ` Kalle Valo
1 sibling, 1 reply; 9+ messages in thread
From: Jeff Johnson @ 2024-04-12 20:53 UTC (permalink / raw
To: Christian Lamparter, Robert Marko, kvalo, davem, edumazet, kuba,
pabeni, robh, krzysztof.kozlowski+dt, conor+dt, jjohnson,
linux-wireless, netdev, devicetree, ath11k, linux-kernel
On 4/12/2024 12:52 PM, Christian Lamparter wrote:
> On 4/12/24 6:24 PM, Robert Marko wrote:
>> The common DT property can be used to limit the available channels
>> but ath11k has to manually call wiphy_read_of_freq_limits().
>>
>> Signed-off-by: Robert Marko <robimarko@gmail.com>
>
> I've seen this before.
>
> https://patchwork.kernel.org/project/linux-wireless/patch/ed266944c721de8dbf0fe35f387a3a71b2c84037.1686486468.git.chunkeey@gmail.com/
>
> (dt-binding too. it has/had an ack)
> https://patchwork.kernel.org/project/linux-wireless/patch/fc606d2550d047a53b4289235dd3c0fe23d5daac.1686486468.git.chunkeey@gmail.com/
>
> sooo.... this is awkward.
Patchwork indicates Changes Requested
Any idea what changes Kalle is looking for?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit property to limit channels
2024-04-12 20:53 ` Jeff Johnson
@ 2024-04-15 16:06 ` Kalle Valo
2024-04-15 16:08 ` Robert Marko
0 siblings, 1 reply; 9+ messages in thread
From: Kalle Valo @ 2024-04-15 16:06 UTC (permalink / raw
To: Jeff Johnson
Cc: Christian Lamparter, Robert Marko, davem, edumazet, kuba, pabeni,
robh, krzysztof.kozlowski+dt, conor+dt, jjohnson, linux-wireless,
netdev, devicetree, ath11k, linux-kernel
Jeff Johnson <quic_jjohnson@quicinc.com> writes:
> On 4/12/2024 12:52 PM, Christian Lamparter wrote:
>> On 4/12/24 6:24 PM, Robert Marko wrote:
>>> The common DT property can be used to limit the available channels
>>> but ath11k has to manually call wiphy_read_of_freq_limits().
>>>
>>> Signed-off-by: Robert Marko <robimarko@gmail.com>
>>
>> I've seen this before.
>>
>> https://patchwork.kernel.org/project/linux-wireless/patch/ed266944c721de8dbf0fe35f387a3a71b2c84037.1686486468.git.chunkeey@gmail.com/
>>
>> (dt-binding too. it has/had an ack)
>> https://patchwork.kernel.org/project/linux-wireless/patch/fc606d2550d047a53b4289235dd3c0fe23d5daac.1686486468.git.chunkeey@gmail.com/
>>
>> sooo.... this is awkward.
>
> Patchwork indicates Changes Requested
> Any idea what changes Kalle is looking for?
I can't remember anymore but most likely I assumed based on Krzysztof's
comments there will be v3 and missed that Conor already had acked it.
Sorry about that, I set Christian's patches to New state now so that
they are back in queue. And I'll drop Robert patches. Does this sound ok
to everyone?
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit property to limit channels
2024-04-15 16:06 ` Kalle Valo
@ 2024-04-15 16:08 ` Robert Marko
0 siblings, 0 replies; 9+ messages in thread
From: Robert Marko @ 2024-04-15 16:08 UTC (permalink / raw
To: Kalle Valo
Cc: Jeff Johnson, Christian Lamparter, davem, edumazet, kuba, pabeni,
robh, krzysztof.kozlowski+dt, conor+dt, jjohnson, linux-wireless,
netdev, devicetree, ath11k, linux-kernel
On Mon, 15 Apr 2024 at 18:06, Kalle Valo <kvalo@kernel.org> wrote:
>
> Jeff Johnson <quic_jjohnson@quicinc.com> writes:
>
> > On 4/12/2024 12:52 PM, Christian Lamparter wrote:
> >> On 4/12/24 6:24 PM, Robert Marko wrote:
> >>> The common DT property can be used to limit the available channels
> >>> but ath11k has to manually call wiphy_read_of_freq_limits().
> >>>
> >>> Signed-off-by: Robert Marko <robimarko@gmail.com>
> >>
> >> I've seen this before.
> >>
> >> https://patchwork.kernel.org/project/linux-wireless/patch/ed266944c721de8dbf0fe35f387a3a71b2c84037.1686486468.git.chunkeey@gmail.com/
> >>
> >> (dt-binding too. it has/had an ack)
> >> https://patchwork.kernel.org/project/linux-wireless/patch/fc606d2550d047a53b4289235dd3c0fe23d5daac.1686486468.git.chunkeey@gmail.com/
> >>
> >> sooo.... this is awkward.
> >
> > Patchwork indicates Changes Requested
> > Any idea what changes Kalle is looking for?
>
> I can't remember anymore but most likely I assumed based on Krzysztof's
> comments there will be v3 and missed that Conor already had acked it.
> Sorry about that, I set Christian's patches to New state now so that
> they are back in queue. And I'll drop Robert patches. Does this sound ok
> to everyone?
Sounds good to me, thanks for looking into it.
Regards,
Robert
>
> --
> https://patchwork.kernel.org/project/linux-wireless/list/
>
> https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] dt-bindings: net: wireless: ath11k: add ieee80211-freq-limit property
2024-04-12 16:24 [PATCH 1/2] dt-bindings: net: wireless: ath11k: add ieee80211-freq-limit property Robert Marko
2024-04-12 16:24 ` [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit property to limit channels Robert Marko
@ 2024-04-16 13:21 ` Rob Herring
2024-04-16 13:22 ` Robert Marko
1 sibling, 1 reply; 9+ messages in thread
From: Rob Herring @ 2024-04-16 13:21 UTC (permalink / raw
To: Robert Marko
Cc: kvalo, davem, edumazet, kuba, pabeni, krzysztof.kozlowski+dt,
conor+dt, jjohnson, linux-wireless, netdev, devicetree, ath11k,
linux-kernel
On Fri, Apr 12, 2024 at 06:24:08PM +0200, Robert Marko wrote:
> This is an existing optional property that ieee80211.yaml/cfg80211
> provides. It's useful to further restrict supported frequencies
> for a specified device through device-tree.
>
> Signed-off-by: Robert Marko <robimarko@gmail.com>
> ---
> .../devicetree/bindings/net/wireless/qcom,ath11k.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> index 672282cdfc2f..907bbb646614 100644
> --- a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> +++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> @@ -55,6 +55,8 @@ properties:
> phandle to a node describing reserved memory (System RAM memory)
> used by ath11k firmware (see bindings/reserved-memory/reserved-memory.txt)
>
> + ieee80211-freq-limit: true
> +
Drop this and change additionalProperties to unevaluatedProperties.
> iommus:
> minItems: 1
> maxItems: 2
> @@ -88,6 +90,7 @@ required:
> additionalProperties: false
>
> allOf:
> + - $ref: ieee80211.yaml#
> - if:
> properties:
> compatible:
> --
> 2.44.0
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] dt-bindings: net: wireless: ath11k: add ieee80211-freq-limit property
2024-04-16 13:21 ` [PATCH 1/2] dt-bindings: net: wireless: ath11k: add ieee80211-freq-limit property Rob Herring
@ 2024-04-16 13:22 ` Robert Marko
0 siblings, 0 replies; 9+ messages in thread
From: Robert Marko @ 2024-04-16 13:22 UTC (permalink / raw
To: Rob Herring
Cc: kvalo, davem, edumazet, kuba, pabeni, krzysztof.kozlowski+dt,
conor+dt, jjohnson, linux-wireless, netdev, devicetree, ath11k,
linux-kernel
On Tue, 16 Apr 2024 at 15:21, Rob Herring <robh@kernel.org> wrote:
>
> On Fri, Apr 12, 2024 at 06:24:08PM +0200, Robert Marko wrote:
> > This is an existing optional property that ieee80211.yaml/cfg80211
> > provides. It's useful to further restrict supported frequencies
> > for a specified device through device-tree.
> >
> > Signed-off-by: Robert Marko <robimarko@gmail.com>
> > ---
> > .../devicetree/bindings/net/wireless/qcom,ath11k.yaml | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> > index 672282cdfc2f..907bbb646614 100644
> > --- a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> > +++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> > @@ -55,6 +55,8 @@ properties:
> > phandle to a node describing reserved memory (System RAM memory)
> > used by ath11k firmware (see bindings/reserved-memory/reserved-memory.txt)
> >
> > + ieee80211-freq-limit: true
> > +
>
> Drop this and change additionalProperties to unevaluatedProperties.
Hi Rob,
This patch series has been dropped as Christian already proposed the
same before and it was reviewed.
Regards,
Robert
>
> > iommus:
> > minItems: 1
> > maxItems: 2
> > @@ -88,6 +90,7 @@ required:
> > additionalProperties: false
> >
> > allOf:
> > + - $ref: ieee80211.yaml#
> > - if:
> > properties:
> > compatible:
> > --
> > 2.44.0
> >
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2024-04-16 13:22 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-12 16:24 [PATCH 1/2] dt-bindings: net: wireless: ath11k: add ieee80211-freq-limit property Robert Marko
2024-04-12 16:24 ` [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit property to limit channels Robert Marko
2024-04-12 19:52 ` Christian Lamparter
2024-04-12 20:21 ` Robert Marko
2024-04-12 20:53 ` Jeff Johnson
2024-04-15 16:06 ` Kalle Valo
2024-04-15 16:08 ` Robert Marko
2024-04-16 13:21 ` [PATCH 1/2] dt-bindings: net: wireless: ath11k: add ieee80211-freq-limit property Rob Herring
2024-04-16 13:22 ` Robert Marko
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.