Skip to contentSkip to conversation

dt-bindings: edac: dmc-520.yaml

PATCH In review Shiping Ji 3 participants 6 messages Jan 27, 2020 – Feb 19, 2020

Subsystems (2)
Device tree bindings EDAC-core
lore
Share & tools
lore

Shiping Ji

author
#0 lore raw

PATCH v11 1/2 dt-bindings: edac: dmc-520.yaml

This is the device tree bindings for new EDAC driver dmc520_edac.c.

From: Lei Wang <[email protected]>

Signed-off-by Lei Wang <[email protected]>
Signed-off-by Shiping Ji <[email protected]>
Reviewed-by James Morse <[email protected]>

---
Changes in v11:
- Fix issues reported by make dt_binding_check

---
 .../devicetree/bindings/edac/dmc-520.yaml     | 59 +++++++++++++++++++
 1 file changed, 59 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/edac/dmc-520.yaml
Documentation/devicetree/bindings/edac/dmc-520.yaml new file +59 -0
new file mode 100644
index 000000000000..9272d2bd8634
@@ -0,0 +1,59 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/edac/dmc-520.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: ARM DMC-520 EDAC bindings
+
+maintainers:
+  - Lei Wang <[email protected]>
+
+description: |+
+  DMC-520 node is defined to describe DRAM error detection and correction.
+
+  https://static.docs.arm.com/100000/0200/corelink_dmc520_trm_100000_0200_01_en.pdf
+
+properties:
+  compatible:
+    items:
+      - const: brcm,dmc-520
+      - const: arm,dmc-520
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    minItems: 1
+    maxItems: 10
+
+  interrupt-names:
+    minItems: 1
+    maxItems: 10
+    items:
+      enum:
+        - ram_ecc_errc
+        - ram_ecc_errd
+        - dram_ecc_errc
+        - dram_ecc_errd
+        - failed_access
+        - failed_prog
+        - link_err
+        - temperature_event
+        - arch_fsm
+        - phy_request
+
+required:
+  - compatible
+  - reg
+  - interrupts
+  - interrupt-names
+
+examples:
+  - |
+    dmc0: dmc@200000 {
+        compatible = "brcm,dmc-520", "arm,dmc-520";
+        reg = <0x200000 0x80000>;
+        interrupts = <0x0 0x349 0x4>, <0x0 0x34B 0x4>;
+        interrupt-names = "dram_ecc_errc", "dram_ecc_errd";
+    };
Signature · 1 line
2.17.1
maintainer Reviewed-by on v11 1/2
#1 lore raw
Quoted 8 lines · Shiping Ji

This is the device tree bindings for new EDAC driver dmc520_edac.c.

From: Lei Wang <[email protected]>

Signed-off-by Lei Wang <[email protected]>
Signed-off-by Shiping Ji <[email protected]>
Reviewed-by James Morse <[email protected]>
Quoted 9 lines

---
Changes in v11:
- Fix issues reported by make dt_binding_check

---
 .../devicetree/bindings/edac/dmc-520.yaml     | 59 +++++++++++++++++++
 1 file changed, 59 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/edac/dmc-520.yaml
Reviewed-by Rob Herring <[email protected]>
maintainer
#2 lore raw
In reply to Shiping Ji
Quoted 8 lines · Shiping Ji

This is the device tree bindings for new EDAC driver dmc520_edac.c.

From: Lei Wang <[email protected]>

Signed-off-by Lei Wang <[email protected]>
Signed-off-by Shiping Ji <[email protected]>
Reviewed-by James Morse <[email protected]>
Quoted 8 lines

---
Changes in v11:
- Fix issues reported by make dt_binding_check

---
 .../devicetree/bindings/edac/dmc-520.yaml     | 59 +++++++++++++++++++
 1 file changed, 59 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/edac/dmc-520.yaml

I have only this v11 patch 1/2 in my inbox and not the actual driver, i.e., patch 2/2.

For the driver, I have v10 here:

https://lkml.kernel.org/r/[email protected]

Did you send a v11 of the driver itself or should I have a look at v10?

Signature · 4 lines
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette
author
#3 lore raw
Quoted 8 lines · Borislav Petkov

I have only this v11 patch 1/2 in my inbox and not the actual driver, i.e., patch 2/2.

For the driver, I have v10 here:

https://lkml.kernel.org/r/[email protected]

Did you send a v11 of the driver itself or should I have a look at v10?

Actually I didn't send v11 of the driver since it will be identical to v10, sorry for keeping you waiting on this. Please review v10, thanks!

--
Best regards,
Shiping Ji

maintainer
#4 lore raw
Quoted 2 lines · Shiping Ji

Actually I didn't send v11 of the driver since it will be identical to v10, sorry for keeping you waiting on this. Please review v10, thanks!

Ok, both patches queued and will appear in linux-next soon.

Thanks for the effort and persistence!

:-)

Signature · 4 lines
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette
author
#5 lore raw
Quoted 9 lines · Borislav Petkov
Quoted 2 lines · Shiping Ji

Actually I didn't send v11 of the driver since it will be identical to v10, sorry for keeping you waiting on this. Please review v10, thanks!

Ok, both patches queued and will appear in linux-next soon.

Thanks for the effort and persistence!

:-)

Thanks a lot for the prompt action! This is great learning process for me :)

--
Best regards,
Shiping Ji