Soeren, On Mon, Oct 12, 2015 at 9:33 AM, Sören Brinkmann <soren.brinkmann@xxxxxxxxxx> wrote: > 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. I kinda prefer the named one, so I'll probaly adapt the implementation, any feelings either way anyone? >someone > Sören Thanks, Moritz -- 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