[v3] dt-bindings: memory-controllers: fsl,imx8m-ddrc: restrict opp-table to objects

[PATCH v3] dt-bindings: memory-controllers: fsl,imx8m-ddrc: restrict opp-table to objects

Posted by Krzysztof Kozlowski 3 years, 7 months ago

Simple 'opp-table:true' accepts a boolean property as opp-table, so
restrict it to object to properly enferce real OPP table nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Peng Fan <peng.fan@nxp.com>

---

Changes since v2:
1. Correct typo in msg (Peng).
2. Add Ack.

Changes since v1:
1. Correct typo in subject.
---
 .../devicetree/bindings/memory-controllers/fsl/imx8m-ddrc.yaml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/memory-controllers/fsl/imx8m-ddrc.yaml b/Documentation/devicetree/bindings/memory-controllers/fsl/imx8m-ddrc.yaml
index 2b39fce5f650..519b123116dc 100644
--- a/Documentation/devicetree/bindings/memory-controllers/fsl/imx8m-ddrc.yaml
+++ b/Documentation/devicetree/bindings/memory-controllers/fsl/imx8m-ddrc.yaml
@@ -47,7 +47,8 @@ properties:
       - const: apb
 
   operating-points-v2: true
-  opp-table: true
+  opp-table:
+    type: object
 
 required:
   - reg
-- 
2.34.1

Re: [PATCH v3] dt-bindings: memory-controllers: fsl,imx8m-ddrc: restrict opp-table to objects

Posted by Rob Herring 3 years, 7 months ago

On Thu, 18 Aug 2022 09:15:49 +0300, Krzysztof Kozlowski wrote:
> Simple 'opp-table:true' accepts a boolean property as opp-table, so
> restrict it to object to properly enferce real OPP table nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Acked-by: Peng Fan <peng.fan@nxp.com>
> 
> ---
> 
> Changes since v2:
> 1. Correct typo in msg (Peng).
> 2. Add Ack.
> 
> Changes since v1:
> 1. Correct typo in subject.
> ---
>  .../devicetree/bindings/memory-controllers/fsl/imx8m-ddrc.yaml | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 

Applied, thanks!