[PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property

[PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property

[Christian Lamparter]

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

[PATCH v1 2/2] wifi: ath11k: add support DT ieee80211-freq-limit

[Christian Lamparter]

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

Re: [PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property

[Conor Dooley]

[-- 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 --]

Re: [PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property

[Rob Herring]

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
> 

Re: [PATCH v1 1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property

[Christian Lamparter]

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