2022-08-17 07:38:48

by Krzysztof Kozlowski

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.

Signed-off-by: Krzysztof Kozlowski <[email protected]>

---

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

2022-08-18 01:39:42

by Peng Fan

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

> 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 <[email protected]>

Acked-by: Peng Fan <[email protected]>
>
> ---
>
> 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

2022-08-18 06:55:24

by Krzysztof Kozlowski

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

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