[PATCH] dt-bindings: memory-controller: qca,ath79-ddr-controller: Drop consumer from example

Rob Herring (Arm) posted 1 patch 1 year, 1 month ago
.../memory-controllers/qca,ath79-ddr-controller.yaml       | 7 -------
1 file changed, 7 deletions(-)
[PATCH] dt-bindings: memory-controller: qca,ath79-ddr-controller: Drop consumer from example
Posted by Rob Herring (Arm) 1 year, 1 month ago
Normal practice is examples only show what the binding document defines
and doesn't include consumers in a provider example (or vice-versa). The
"qca,ddr-wb-channel-interrupts" and "qca,ddr-wb-channels" properties are
also not yet documented by a schema, so avoid (not yet enabled) warnings
on them by dropping the interrupt-controller node from the example.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 .../memory-controllers/qca,ath79-ddr-controller.yaml       | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/Documentation/devicetree/bindings/memory-controllers/qca,ath79-ddr-controller.yaml b/Documentation/devicetree/bindings/memory-controllers/qca,ath79-ddr-controller.yaml
index 0c511ab906bf..8b937f90a1fb 100644
--- a/Documentation/devicetree/bindings/memory-controllers/qca,ath79-ddr-controller.yaml
+++ b/Documentation/devicetree/bindings/memory-controllers/qca,ath79-ddr-controller.yaml
@@ -52,10 +52,3 @@ examples:
 
         #qca,ddr-wb-channel-cells = <1>;
     };
-
-    interrupt-controller {
-        // ...
-        qca,ddr-wb-channel-interrupts = <2>, <3>, <4>, <5>;
-        qca,ddr-wb-channels = <&ddr_ctrl 3>, <&ddr_ctrl 2>,
-                              <&ddr_ctrl 0>, <&ddr_ctrl 1>;
-    };
-- 
2.45.2
Re: [PATCH] dt-bindings: memory-controller: qca,ath79-ddr-controller: Drop consumer from example
Posted by Rob Herring (Arm) 1 year, 1 month ago
On Fri, 03 Jan 2025 15:24:48 -0600, Rob Herring (Arm) wrote:
> Normal practice is examples only show what the binding document defines
> and doesn't include consumers in a provider example (or vice-versa). The
> "qca,ddr-wb-channel-interrupts" and "qca,ddr-wb-channels" properties are
> also not yet documented by a schema, so avoid (not yet enabled) warnings
> on them by dropping the interrupt-controller node from the example.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../memory-controllers/qca,ath79-ddr-controller.yaml       | 7 -------
>  1 file changed, 7 deletions(-)
> 

Applied, thanks!
Re: [PATCH] dt-bindings: memory-controller: qca,ath79-ddr-controller: Drop consumer from example
Posted by Krzysztof Kozlowski 1 year, 1 month ago
On 03/01/2025 22:24, Rob Herring (Arm) wrote:
> Normal practice is examples only show what the binding document defines
> and doesn't include consumers in a provider example (or vice-versa). The
> "qca,ddr-wb-channel-interrupts" and "qca,ddr-wb-channels" properties are
> also not yet documented by a schema, so avoid (not yet enabled) warnings
> on them by dropping the interrupt-controller node from the example.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../memory-controllers/qca,ath79-ddr-controller.yaml       | 7 -------
>  1 file changed, 7 deletions(-)

It is too late in the cycle for me to pick it up. I will take it after
the merge window... or alternatively you can take it via DT for upcoming
merge window:

Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>

Best regards,
Krzysztof