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

Krzysztof Kozlowski posted 1 patch 3 years, 7 months ago
There is a newer version of this series
.../devicetree/bindings/memory-controllers/fsl/imx8m-ddrc.yaml | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
[PATCH v2] 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 enfoerce real OPP table nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

---

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 445e46feda69..fc2e7de10331 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 v2] dt-bindings: memory-controllers: fsl,imx8m-ddrc: restrict opp-table to objects
Posted by Peng Fan 3 years, 7 months ago
> Subject: [PATCH v2] dt-bindings: memory-controllers: fsl,imx8m-ddrc:
> restrict opp-table to objects
> 
> Simple 'opp-table:true' accepts a boolean property as opp-table, so restrict
> it to object to properly enfoerce real OPP table nodes.

enfoerce->enforce?


> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Acked-by: Peng Fan <peng.fan@nxp.com>
> 
> ---
> 
> 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 445e46feda69..fc2e7de10331 100644
> --- a/Documentation/devicetree/bindings/memory-controllers/fsl/imx8m-
> ddrc.yaml
> +++ b/Documentation/devicetree/bindings/memory-controllers/fsl/imx8m-
> ddr
> +++ c.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 v2] dt-bindings: memory-controllers: fsl,imx8m-ddrc: restrict opp-table to objects
Posted by Krzysztof Kozlowski 3 years, 7 months ago
On 18/08/2022 04:29, Peng Fan wrote:
>> Subject: [PATCH v2] dt-bindings: memory-controllers: fsl,imx8m-ddrc:
>> restrict opp-table to objects
>>
>> Simple 'opp-table:true' accepts a boolean property as opp-table, so restrict
>> it to object to properly enfoerce real OPP table nodes.
> 
> enfoerce->enforce?

Thanks, indeed.


Best regards,
Krzysztof