* [v5,2/4] dt: bindings: Document ZynqMP DDRC in Synopsys documentation
@ 2018-09-06 15:21 Manish Narani
0 siblings, 0 replies; 3+ messages in thread
From: Manish Narani @ 2018-09-06 15:21 UTC (permalink / raw)
To: Rob Herring
Cc: mark.rutland, Michal Simek, bp, mchehab, leoyang.li,
amit.kucheria, olof, Srinivas Goud, Anirudha Sarangi,
linux-kernel, devicetree, linux-arm-kernel, linux-edac
Hi Rob,
> -----Original Message-----
> From: Rob Herring [mailto:robh@kernel.org]
> Sent: Tuesday, September 4, 2018 7:46 PM
> To: Manish Narani <MNARANI@xilinx.com>
> Cc: mark.rutland@arm.com; Michal Simek <michals@xilinx.com>;
> bp@alien8.de; mchehab@kernel.org; leoyang.li@nxp.com;
> amit.kucheria@linaro.org; olof@lixom.net; Srinivas Goud <sgoud@xilinx.com>;
> Anirudha Sarangi <anirudh@xilinx.com>; linux-kernel@vger.kernel.org;
> devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
> edac@vger.kernel.org
> Subject: Re: [PATCH v5 2/4] dt: bindings: Document ZynqMP DDRC in Synopsys
> documentation
>
> On Fri, Aug 31, 2018 at 06:57:48PM +0530, Manish Narani wrote:
> > Add information of ZynqMP DDRC which reports the single bit errors
> > that are corrected and the double bit errors that are detected.
> >
> > Signed-off-by: Manish Narani <manish.narani@xilinx.com>
> > ---
> > .../bindings/memory-controllers/synopsys.txt | 27 ++++++++++++++++++-
> ---
> > 1 file changed, 22 insertions(+), 5 deletions(-)
>
> Please add acks/reviewed-bys when posting new versions.
Sure. I will include it in v6.
Thanks,
Manish Narani
^ permalink raw reply [flat|nested] 3+ messages in thread
* [v5,2/4] dt: bindings: Document ZynqMP DDRC in Synopsys documentation
@ 2018-09-04 14:16 Rob Herring
0 siblings, 0 replies; 3+ messages in thread
From: Rob Herring @ 2018-09-04 14:16 UTC (permalink / raw)
To: Manish Narani
Cc: mark.rutland, michal.simek, bp, mchehab, leoyang.li,
amit.kucheria, olof, sgoud, anirudh, linux-kernel, devicetree,
linux-arm-kernel, linux-edac
On Fri, Aug 31, 2018 at 06:57:48PM +0530, Manish Narani wrote:
> Add information of ZynqMP DDRC which reports the single bit errors that
> are corrected and the double bit errors that are detected.
>
> Signed-off-by: Manish Narani <manish.narani@xilinx.com>
> ---
> .../bindings/memory-controllers/synopsys.txt | 27 ++++++++++++++++++----
> 1 file changed, 22 insertions(+), 5 deletions(-)
Please add acks/reviewed-bys when posting new versions.
Rob
^ permalink raw reply [flat|nested] 3+ messages in thread
* [v5,2/4] dt: bindings: Document ZynqMP DDRC in Synopsys documentation
@ 2018-08-31 13:27 Manish Narani
0 siblings, 0 replies; 3+ messages in thread
From: Manish Narani @ 2018-08-31 13:27 UTC (permalink / raw)
To: robh+dt, mark.rutland, michal.simek, bp, mchehab, manish.narani,
leoyang.li, amit.kucheria, olof
Cc: sgoud, anirudh, linux-kernel, devicetree, linux-arm-kernel, linux-edac
Add information of ZynqMP DDRC which reports the single bit errors that
are corrected and the double bit errors that are detected.
Signed-off-by: Manish Narani <manish.narani@xilinx.com>
---
.../bindings/memory-controllers/synopsys.txt | 27 ++++++++++++++++++----
1 file changed, 22 insertions(+), 5 deletions(-)
diff --git a/Documentation/devicetree/bindings/memory-controllers/synopsys.txt b/Documentation/devicetree/bindings/memory-controllers/synopsys.txt
index a43d26d..9d32762 100644
--- a/Documentation/devicetree/bindings/memory-controllers/synopsys.txt
+++ b/Documentation/devicetree/bindings/memory-controllers/synopsys.txt
@@ -1,15 +1,32 @@
Binding for Synopsys IntelliDDR Multi Protocol Memory Controller
-This controller has an optional ECC support in half-bus width (16-bit)
-configuration. The ECC controller corrects one bit error and detects
-two bit errors.
+The ZynqMP DDR ECC controller has an optional ECC support in 64-bit and 32-bit
+bus width configurations.
+
+The Zynq DDR ECC controller has an optional ECC support in half-bus width
+(16-bit) configuration.
+
+These both ECC controllers correct single bit ECC errors and detect double bit
+ECC errors.
Required properties:
- - compatible: Should be 'xlnx,zynq-ddrc-a05'
- - reg: Base address and size of the controllers memory area
+ - compatible: One of:
+ - 'xlnx,zynq-ddrc-a05' : Zynq DDR ECC controller
+ - 'xlnx,zynqmp-ddrc-2.40a' : ZynqMP DDR ECC controller
+ - reg: Should contain DDR controller registers location and length.
+
+Required properties for "xlnx,zynqmp-ddrc-2.40a":
+ - interrupts: Property with a value describing the interrupt number.
Example:
memory-controller@f8006000 {
compatible = "xlnx,zynq-ddrc-a05";
reg = <0xf8006000 0x1000>;
};
+
+ mc: memory-controller@fd070000 {
+ compatible = "xlnx,zynqmp-ddrc-2.40a";
+ reg = <0x0 0xfd070000 0x0 0x30000>;
+ interrupt-parent = <&gic>;
+ interrupts = <0 112 4>;
+ };
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-09-06 15:21 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-06 15:21 [v5,2/4] dt: bindings: Document ZynqMP DDRC in Synopsys documentation Manish Narani
-- strict thread matches above, loose matches on Subject: below --
2018-09-04 14:16 Rob Herring
2018-08-31 13:27 Manish Narani
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).