Received-SPF: pass (google.com: domain of linux-wireless+bounces-6399-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Precedence: bulk
MIME-Version: 1.0
References: <20240412162510.29483-1-robimarko@gmail.com> <20240416132124.GA2143558-robh@kernel.org>
In-Reply-To: <20240416132124.GA2143558-robh@kernel.org>
From: Robert Marko <robimarko@gmail.com>
Date: Tue, 16 Apr 2024 15:22:34 +0200
Message-ID: <CAOX2RU5tWt-dEA4it1G+LeNawwPqkwdG=vBBgnmdD6CNc4iywA@mail.gmail.com>
Subject: Re: [PATCH 1/2] dt-bindings: net: wireless: ath11k: add
 ieee80211-freq-limit property
To: Rob Herring <robh@kernel.org>
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
Content-Type: text/plain; charset="UTF-8"

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