diff mbox series

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

Message ID a3075482150d342f71ec235badacec32cdd6c553.1686300243.git.chunkeey@gmail.com (mailing list archive)
State Changes Requested
Delegated to: Kalle Valo
Headers show
Series [v1,1/2] dt-bindings: net: ath11k: add ieee80211-freq-limit property | expand

Commit Message

Christian Lamparter June 9, 2023, 8:44 a.m. UTC
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(+)

Comments

Conor Dooley June 9, 2023, 3:49 p.m. UTC | #1
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.
Rob Herring June 9, 2023, 4:51 p.m. UTC | #2
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
>
Christian Lamparter June 9, 2023, 5:11 p.m. UTC | #3
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
diff mbox series

Patch

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: |