[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <MWHPR0201MB34814FE28937F4A6F3540C72C22E0@MWHPR0201MB3481.namprd02.prod.outlook.com>
Date: Tue, 31 Jul 2018 13:12:32 +0000
From: Nava kishore Manne <navam@...inx.com>
To: Michal Simek <michals@...inx.com>,
"robh+dt@...nel.org" <robh+dt@...nel.org>,
"mark.rutland@....com" <mark.rutland@....com>,
Michal Simek <michals@...inx.com>,
Soren Brinkmann <sorenb@...inx.com>,
"atull@...nsource.altera.com" <atull@...nsource.altera.com>,
"moritz.fischer@...us.com" <moritz.fischer@...us.com>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Appana Durga Kedareswara Rao <appanad@...inx.com>,
"chinnikishore369@...il.com" <chinnikishore369@...il.com>,
"atull@...nel.org" <atull@...nel.org>
Subject: RE: [RFC PATCH 1/2] ARM: dt: fpga: Added binding docs for Xilinx
zynqmp FPGA manager
Thanks for the quick response..
Please see my comments inline....
> -----Original Message-----
> From: Michal Simek [mailto:michal.simek@...inx.com]
> Sent: Tuesday, July 31, 2018 5:48 PM
> To: Nava kishore Manne <navam@...inx.com>; robh+dt@...nel.org;
> mark.rutland@....com; Michal Simek <michals@...inx.com>; Soren Brinkmann
> <sorenb@...inx.com>; atull@...nsource.altera.com;
> moritz.fischer@...us.com; devicetree@...r.kernel.org; linux-arm-
> kernel@...ts.infradead.org; linux-kernel@...r.kernel.org; Appana Durga
> Kedareswara Rao <appanad@...inx.com>; chinnikishore369@...il.com
> Cc: Nava kishore Manne <navam@...inx.com>
> 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:
> > New bindings document for zynqmp fpga manager.
> >
> > Signed-off-by: Nava kishore Manne <navam@...inx.com>
> > ---
> > .../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
> >
> > diff --git a/Documentation/devicetree/bindings/fpga/xlnx,zynqmp-pcap-
> fpga.txt b/Documentation/devicetree/bindings/fpga/xlnx,zynqmp-pcap-
> fpga.txt
> > 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..
> > +
> > +Required properties:
> > +- compatible: should contain "xlnx,zynqmp-pcap-fpga"
> > +
> > +Example:
> > + pcap: pcap@00 {
>
> no reason for address if there is no any below.
>
Will fix in the next version....
Regards,
Navakishore.
Powered by blists - more mailing lists