diff mbox series

[V2,1/2] dt-bindings: memory: snps, ddrc-3.80a compatible also need interrupts

Message ID 20220321075131.17811-2-sherry.sun@nxp.com (mailing list archive)
State New, archived
Headers show
Series add ddr controller node to support EDAC on imx8mp | expand

Commit Message

Sherry Sun March 21, 2022, 7:51 a.m. UTC
For the snps,ddrc-3.80a compatible, the interrupts property is also
required, also order the compatibles by name (s goes before x).

Signed-off-by: Sherry Sun <sherry.sun@nxp.com>
---
 .../bindings/memory-controllers/synopsys,ddrc-ecc.yaml      | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Krzysztof Kozlowski March 21, 2022, 8:10 a.m. UTC | #1
On 21/03/2022 08:51, Sherry Sun wrote:
> For the snps,ddrc-3.80a compatible, the interrupts property is also
> required, also order the compatibles by name (s goes before x).
> 
> Signed-off-by: Sherry Sun <sherry.sun@nxp.com>
> ---
>  .../bindings/memory-controllers/synopsys,ddrc-ecc.yaml      | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 

Fixes: a9e6b3819b36 ("dt-bindings: memory: Add entry for version 3.80a")


Looks good, I will take it after merge window. Somehow I was not cc-ed
on previous memory controllers patch. :(

Best regards,
Krzysztof
Krzysztof Kozlowski April 4, 2022, 5:08 p.m. UTC | #2
On Mon, 21 Mar 2022 15:51:30 +0800, Sherry Sun wrote:
> For the snps,ddrc-3.80a compatible, the interrupts property is also
> required, also order the compatibles by name (s goes before x).
> 
> 

Applied, thanks!

[1/2] dt-bindings: memory: snps,ddrc-3.80a compatible also need interrupts
      commit: 4f9f45d0eb0e7d449bc9294459df79b9c66edfac

Best regards,
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/memory-controllers/synopsys,ddrc-ecc.yaml b/Documentation/devicetree/bindings/memory-controllers/synopsys,ddrc-ecc.yaml
index fb7ae38a9c86..e3bc6ebce090 100644
--- a/Documentation/devicetree/bindings/memory-controllers/synopsys,ddrc-ecc.yaml
+++ b/Documentation/devicetree/bindings/memory-controllers/synopsys,ddrc-ecc.yaml
@@ -24,9 +24,9 @@  description: |
 properties:
   compatible:
     enum:
+      - snps,ddrc-3.80a
       - xlnx,zynq-ddrc-a05
       - xlnx,zynqmp-ddrc-2.40a
-      - snps,ddrc-3.80a
 
   interrupts:
     maxItems: 1
@@ -43,7 +43,9 @@  allOf:
       properties:
         compatible:
           contains:
-            const: xlnx,zynqmp-ddrc-2.40a
+            enum:
+              - snps,ddrc-3.80a
+              - xlnx,zynqmp-ddrc-2.40a
     then:
       required:
         - interrupts