* [PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property
@ 2023-06-09 8:44 ` Christian Lamparter
0 siblings, 0 replies; 10+ messages in thread
From: Christian Lamparter @ 2023-06-09 8:44 UTC (permalink / raw)
To: devicetree, linux-wireless, ath11k
Cc: kvalo, conor+dt, krzysztof.kozlowski+dt, robh+dt
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: Christian Lamparter <chunkeey@gmail.com>
---
Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
index 7d5f982a3d09..91bf69c2df01 100644
--- a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
+++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
@@ -58,6 +58,8 @@ properties:
minItems: 1
maxItems: 2
+ ieee80211-freq-limit: true
+
wifi-firmware:
type: object
description: |
--
2.40.1
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH v1 2/2] wifi: ath11k: add support DT ieee80211-freq-limit
2023-06-09 8:44 ` Christian Lamparter
@ 2023-06-09 8:44 ` Christian Lamparter
-1 siblings, 0 replies; 10+ messages in thread
From: Christian Lamparter @ 2023-06-09 8:44 UTC (permalink / raw)
To: devicetree, linux-wireless, ath11k
Cc: kvalo, conor+dt, krzysztof.kozlowski+dt, robh+dt
The common DT property can be used to limit the available
channels/frequencies. But ath11k has to manually call
wiphy_read_of_freq_limits().
Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
---
compile-tested only.
---
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 c947d1c8d8c1..7e43f2770f27 100644
--- a/drivers/net/wireless/ath/ath11k/mac.c
+++ b/drivers/net/wireless/ath/ath11k/mac.c
@@ -9414,6 +9414,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.40.1
--
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v1 2/2] wifi: ath11k: add support DT ieee80211-freq-limit
@ 2023-06-09 8:44 ` Christian Lamparter
0 siblings, 0 replies; 10+ messages in thread
From: Christian Lamparter @ 2023-06-09 8:44 UTC (permalink / raw)
To: devicetree, linux-wireless, ath11k
Cc: kvalo, conor+dt, krzysztof.kozlowski+dt, robh+dt
The common DT property can be used to limit the available
channels/frequencies. But ath11k has to manually call
wiphy_read_of_freq_limits().
Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
---
compile-tested only.
---
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 c947d1c8d8c1..7e43f2770f27 100644
--- a/drivers/net/wireless/ath/ath11k/mac.c
+++ b/drivers/net/wireless/ath/ath11k/mac.c
@@ -9414,6 +9414,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.40.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property
2023-06-09 8:44 ` Christian Lamparter
@ 2023-06-09 15:49 ` Conor Dooley
-1 siblings, 0 replies; 10+ messages in thread
From: Conor Dooley @ 2023-06-09 15:49 UTC (permalink / raw)
To: Christian Lamparter
Cc: devicetree, linux-wireless, ath11k, kvalo, conor+dt,
krzysztof.kozlowski+dt, robh+dt
[-- Attachment #1.1: Type: text/plain, Size: 391 bytes --]
On Fri, Jun 09, 2023 at 10:44:54AM +0200, Christian Lamparter 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: Christian Lamparter <chunkeey@gmail.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Cheers,
Conor.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
[-- Attachment #2: Type: text/plain, Size: 102 bytes --]
--
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property
@ 2023-06-09 15:49 ` Conor Dooley
0 siblings, 0 replies; 10+ messages in thread
From: Conor Dooley @ 2023-06-09 15:49 UTC (permalink / raw)
To: Christian Lamparter
Cc: devicetree, linux-wireless, ath11k, kvalo, conor+dt,
krzysztof.kozlowski+dt, robh+dt
[-- Attachment #1: Type: text/plain, Size: 391 bytes --]
On Fri, Jun 09, 2023 at 10:44:54AM +0200, Christian Lamparter 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: Christian Lamparter <chunkeey@gmail.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Cheers,
Conor.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property
2023-06-09 8:44 ` Christian Lamparter
@ 2023-06-09 16:51 ` Rob Herring
-1 siblings, 0 replies; 10+ messages in thread
From: Rob Herring @ 2023-06-09 16:51 UTC (permalink / raw)
To: Christian Lamparter
Cc: devicetree, linux-wireless, ath11k, kvalo, conor+dt,
krzysztof.kozlowski+dt
On Fri, Jun 09, 2023 at 10:44:54AM +0200, Christian Lamparter 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: Christian Lamparter <chunkeey@gmail.com>
> ---
> Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> index 7d5f982a3d09..91bf69c2df01 100644
> --- a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> +++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> @@ -58,6 +58,8 @@ properties:
> minItems: 1
> maxItems: 2
>
> + ieee80211-freq-limit: true
> +
You need a ref to ieee80211.yaml as well.
> wifi-firmware:
> type: object
> description: |
> --
> 2.40.1
>
--
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property
@ 2023-06-09 16:51 ` Rob Herring
0 siblings, 0 replies; 10+ messages in thread
From: Rob Herring @ 2023-06-09 16:51 UTC (permalink / raw)
To: Christian Lamparter
Cc: devicetree, linux-wireless, ath11k, kvalo, conor+dt,
krzysztof.kozlowski+dt
On Fri, Jun 09, 2023 at 10:44:54AM +0200, Christian Lamparter 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: Christian Lamparter <chunkeey@gmail.com>
> ---
> Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> index 7d5f982a3d09..91bf69c2df01 100644
> --- a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> +++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
> @@ -58,6 +58,8 @@ properties:
> minItems: 1
> maxItems: 2
>
> + ieee80211-freq-limit: true
> +
You need a ref to ieee80211.yaml as well.
> wifi-firmware:
> type: object
> description: |
> --
> 2.40.1
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property
2023-06-09 16:51 ` Rob Herring
@ 2023-06-09 17:11 ` Christian Lamparter
-1 siblings, 0 replies; 10+ messages in thread
From: Christian Lamparter @ 2023-06-09 17:11 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-wireless, ath11k, kvalo, conor+dt,
krzysztof.kozlowski+dt
On 6/9/23 18:51, Rob Herring wrote:
> On Fri, Jun 09, 2023 at 10:44:54AM +0200, Christian Lamparter 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: Christian Lamparter <chunkeey@gmail.com>
>> ---
>> Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
>> index 7d5f982a3d09..91bf69c2df01 100644
>> --- a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
>> +++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
>> @@ -58,6 +58,8 @@ properties:
>> minItems: 1
>> maxItems: 2
>>
>> + ieee80211-freq-limit: true
>> +
>
> You need a ref to ieee80211.yaml as well.
Yes. will fix that.
(Related to that: ath10k already supported ieee80211-freq-limit since 2018
but not in the binding. will post a separate patch shortly)
Regards,
Christian
--
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property
@ 2023-06-09 17:11 ` Christian Lamparter
0 siblings, 0 replies; 10+ messages in thread
From: Christian Lamparter @ 2023-06-09 17:11 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-wireless, ath11k, kvalo, conor+dt,
krzysztof.kozlowski+dt
On 6/9/23 18:51, Rob Herring wrote:
> On Fri, Jun 09, 2023 at 10:44:54AM +0200, Christian Lamparter 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: Christian Lamparter <chunkeey@gmail.com>
>> ---
>> Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
>> index 7d5f982a3d09..91bf69c2df01 100644
>> --- a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
>> +++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
>> @@ -58,6 +58,8 @@ properties:
>> minItems: 1
>> maxItems: 2
>>
>> + ieee80211-freq-limit: true
>> +
>
> You need a ref to ieee80211.yaml as well.
Yes. will fix that.
(Related to that: ath10k already supported ieee80211-freq-limit since 2018
but not in the binding. will post a separate patch shortly)
Regards,
Christian
^ permalink raw reply [flat|nested] 10+ messages in thread