Thanks for the quick response..

Please see my comments inline....

-----Original Message-----
From: Michal Simek [mailto:michal.simek at xilinx.com]
Sent: Tuesday, July 31, 2018 5:48 PM
To: Nava kishore Manne <redacted>; robh+dt at kernel.org;
mark.rutland at arm.com; Michal Simek [off-list ref]; Soren Brinkmann
[off-list ref]; atull at opensource.altera.com;
moritz.fischer at ettus.com; devicetree at vger.kernel.org; linux-arm-
kernel at lists.infradead.org; linux-kernel at vger.kernel.org; Appana Durga
Kedareswara Rao [off-list ref]; chinnikishore369 at gmail.com
Cc: Nava kishore Manne <redacted>
Subject: Re: [RFC PATCH 1/2] ARM: dt: fpga: Added binding docs for Xilinx
zynqmp FPGA manager

On 1.8.2018 12:04, Nava kishore Manne wrote:

quoted

New bindings document for zynqmp fpga manager.

Signed-off-by: Nava kishore Manne <redacted>
---
 .../devicetree/bindings/fpga/xlnx,zynqmp-pcap-fpga.txt   | 9 +++++++++
 1 file changed, 9 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/fpga/xlnx,zynqmp-

pcap-fpga.txt

quoted

diff --git a/Documentation/devicetree/bindings/fpga/xlnx,zynqmp-pcap-

fpga.txt b/Documentation/devicetree/bindings/fpga/xlnx,zynqmp-pcap-
fpga.txt

quoted

new file mode 100644
index 000000000000..0eac747747f5

--- /dev/null
+++ b/Documentation/devicetree/bindings/fpga/xlnx,zynqmp-pcap-fpga.txt

@@ -0,0 +1,9 @@
+Xilinx Zynqmp FPGA Manager

This should be ZynqMP.

Will fix in the next version..

quoted

+
+Required properties:
+- compatible:		should contain "xlnx,zynqmp-pcap-fpga"
+
+Example:
+	pcap: pcap at 00 {

no reason for address if there is no any below.

 Will fix in the next version....

Regards,
Navakishore.