[PATCH 1/3] doc: dt: fpga: Added Documentation for Xilinx Zynq FPGA manager.
moritz.fischer at ettus.com
Mon Oct 12 13:41:42 PDT 2015
Hi Michal, Josh,
On Mon, Oct 12, 2015 at 2:31 AM, Michal Simek <michal.simek at xilinx.com> wrote:
> 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 <moritz.fischer at ettus.com>
>>> .../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.
>> 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.
Alright, will correct typo, and mark it as optional unless someone
feels strongly against having the optional.
Thanks for the feedback,
More information about the linux-arm-kernel