2024-05-28 13:46:48

by Rob Herring (Arm)

[permalink] [raw]
Subject: [PATCH] dt-bindings: net: wireless: ath11k: Drop "qcom,ipq8074-wcss-pil" from example

Convention for examples is to only show what's covered by the binding,
so drop the provider "qcom,ipq8074-wcss-pil". It is also not documented
by a schema which caused a warning.

Signed-off-by: Rob Herring (Arm) <[email protected]>
---
.../devicetree/bindings/net/wireless/qcom,ath11k.yaml | 9 ---------
1 file changed, 9 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
index a2d55bf4c7a5..ff5763dc66a8 100644
--- a/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
+++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath11k.yaml
@@ -265,15 +265,6 @@ allOf:

examples:
- |
-
- q6v5_wcss: remoteproc@cd00000 {
- compatible = "qcom,ipq8074-wcss-pil";
- reg = <0xcd00000 0x4040>,
- <0x4ab000 0x20>;
- reg-names = "qdsp6",
- "rmb";
- };
-
wifi0: wifi@c000000 {
compatible = "qcom,ipq8074-wifi";
reg = <0xc000000 0x2000000>;
--
2.43.0



2024-05-28 16:03:02

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: net: wireless: ath11k: Drop "qcom,ipq8074-wcss-pil" from example

On 28/05/2024 15:46, Rob Herring (Arm) wrote:
> Convention for examples is to only show what's covered by the binding,
> so drop the provider "qcom,ipq8074-wcss-pil". It is also not documented
> by a schema which caused a warning.
>
> Signed-off-by: Rob Herring (Arm) <[email protected]>

Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof


2024-05-29 09:05:26

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: net: wireless: ath11k: Drop "qcom,ipq8074-wcss-pil" from example

"Rob Herring (Arm)" <[email protected]> wrote:

> Convention for examples is to only show what's covered by the binding,
> so drop the provider "qcom,ipq8074-wcss-pil". It is also not documented
> by a schema which caused a warning.
>
> Signed-off-by: Rob Herring (Arm) <[email protected]>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>

I'll take this to our ath.git tree, ok?

--
https://patchwork.kernel.org/project/linux-wireless/patch/[email protected]/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches


2024-05-30 15:50:47

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: net: wireless: ath11k: Drop "qcom,ipq8074-wcss-pil" from example

"Rob Herring (Arm)" <[email protected]> wrote:

> Convention for examples is to only show what's covered by the binding,
> so drop the provider "qcom,ipq8074-wcss-pil". It is also not documented
> by a schema which caused a warning.
>
> Signed-off-by: Rob Herring (Arm) <[email protected]>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>
> Signed-off-by: Kalle Valo <[email protected]>

Patch applied to ath-next branch of ath.git, thanks.

43e934360d1d dt-bindings: net: wireless: ath11k: Drop "qcom,ipq8074-wcss-pil" from example

--
https://patchwork.kernel.org/project/linux-wireless/patch/[email protected]/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches