Received-SPF: pass (google.com: domain of linux-wireless+bounces-6398-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Tue, 16 Apr 2024 08:21:24 -0500
From: Rob Herring <robh@kernel.org>
To: Robert Marko <robimarko@gmail.com>
Cc: kvalo@kernel.org, davem@davemloft.net, edumazet@google.com,
	kuba@kernel.org, pabeni@redhat.com,
	krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
	jjohnson@kernel.org, linux-wireless@vger.kernel.org,
	netdev@vger.kernel.org, devicetree@vger.kernel.org,
	ath11k@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] dt-bindings: net: wireless: ath11k: add
 ieee80211-freq-limit property
Message-ID: <20240416132124.GA2143558-robh@kernel.org>
References: <20240412162510.29483-1-robimarko@gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20240412162510.29483-1-robimarko@gmail.com>

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
>