On Fri, 2015-10-09 at 12:45AM +0200, Moritz Fischer wrote: > Signed-off-by: Moritz Fischer <moritz.fischer@xxxxxxxxx> > --- > .../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 > +- interrupts: interrupt for the FPGA manager device > +- clocks: phandle for clocks required operation > +- syscon: phandle for access to SLCR registers > + > +Optional properties: > +- clock-names: names for clocks Is it optional? Currently, there is only one clock input, so a match without specifying a clock name should work making this optional. But in your implementation, you do specify a clock name in devm_clk_get(). I'm not entirely sure, but that call might fail if it doesn't find the corresponding clock-names property. I think, either we should make this required and list the required entries here. Or the implementation probably needs to drop the clock name when looking up its input clock. Sören -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html