Hi Frank, Am Dienstag, 9. Juli 2024, 22:23:05 CEST schrieb Frank Li: > iMX9 memory controller is similar with other layerscape chips. But some > register layout has a little bit difference, so add new compatible string > 'nxp,imx9-memory-controller' for it. Is this controller the same for all i.MX9 SoC? E.g. i.MX91, i.MX93, i.MX95 and any future variants? Best regards, Alexander > imx9 need two 'reg', one for DDR controller and the other is ECC inject > engine register space. Keep the same restriction for other compatible > string. > > Signed-off-by: Frank Li <Frank.Li@xxxxxxx> > --- > .../bindings/memory-controllers/fsl/fsl,ddr.yaml | 31 +++++++++++++++++++++- > 1 file changed, 30 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/memory-controllers/fsl/fsl,ddr.yaml b/Documentation/devicetree/bindings/memory-controllers/fsl/fsl,ddr.yaml > index 84f778a99546b..e0786153eec73 100644 > --- a/Documentation/devicetree/bindings/memory-controllers/fsl/fsl,ddr.yaml > +++ b/Documentation/devicetree/bindings/memory-controllers/fsl/fsl,ddr.yaml > @@ -40,6 +40,7 @@ properties: > - fsl,p1021-memory-controller > - fsl,p2020-memory-controller > - fsl,qoriq-memory-controller > + - nxp,imx9-memory-controller > > interrupts: > maxItems: 1 > @@ -51,13 +52,41 @@ properties: > type: boolean > > reg: > - maxItems: 1 > + items: > + - description: Controller register space > + - description: Inject register space > + minItems: 1 > + > + reg-names: > + items: > + - const: ctrl > + - const: inject > + minItems: 1 > > required: > - compatible > - interrupts > - reg > > +allOf: > + - if: > + properties: > + compatible: > + contains: > + enum: > + - nxp,imx9-memory-controller > + then: > + properties: > + reg: > + minItems: 2 > + reg-names: > + minItems: 2 > + else: > + properties: > + reg: > + maxItems: 1 > + reg-names: false > + > additionalProperties: false > > examples: > > -- TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht München, HRB 105018 Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider http://www.tq-group.com/
