From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Subject: Re: [PATCH 1/3] doc: dt: fpga: Added Documentation for Xilinx Zynq FPGA manager. Date: Mon, 12 Oct 2015 11:31:28 +0200 Message-ID: <561B7DF0.6090601@xilinx.com> References: <1444344307-22509-1-git-send-email-moritz.fischer@ettus.com> <1444344307-22509-2-git-send-email-moritz.fischer@ettus.com> <20151009160423.GL10631@jcartwri.amer.corp.natinst.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20151009160423.GL10631@jcartwri.amer.corp.natinst.com> Sender: linux-kernel-owner@vger.kernel.org To: Josh Cartwright , Moritz Fischer Cc: michal.simek@xilinx.com, soren.brinkmann@xilinx.com, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, linux@arm.linux.org.uk, dinguyen@opensource.altera.com, atull@opensource.altera.com, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, gregkh@linuxfoundation.org List-Id: devicetree@vger.kernel.org On 10/09/2015 06:04 PM, Josh Cartwright wrote: > On Fri, Oct 09, 2015 at 12:45:05AM +0200, Moritz Fischer wrote: >> Signed-off-by: Moritz Fischer >> --- >> .../bindings/fpga/xilinx-zynq-fpga-mgr.txt | 26 ++++++++++++++++++++++ >> 1 file changed, 26 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/fpga/xilinx-zynq-fpga-mgr.txt >> >> diff --git a/Documentation/devicetree/bindings/fpga/xilinx-zynq-fpga-mgr.txt b/Documentation/devicetree/bindings/fpga/xilinx-zynq-fpga-mgr.txt >> new file mode 100644 >> index 0000000..82ffda8 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/fpga/xilinx-zynq-fpga-mgr.txt >> @@ -0,0 +1,26 @@ >> +Xilinx Zynq FPGA Manager >> + >> +Required properties: >> +- compatible: should contain "xlnx,zynq-devcfg-1.0" >> +- reg: base address and size for memory mapped io >> +- interrupt parent: interrupt source phandle > > I think you mean 'interrupt-parent', with the hyphen. yes. > > Actually, this isn't really a 'required' property of this node, as it > could be specified in a parent node. yes. Normally interrupt-parent is not listed as required property in the binding doc. Some docs listed it as optional property. Thanks, Michal