Hi! On Tue, Oct 27, 2015 at 05:09:11PM -0500, atull@xxxxxxxxxxxxxxxxxxxxx wrote: > From: Alan Tull <atull@xxxxxxxxxxxxxxxxxxxxx> > > New bindings document for simple fpga bus. > > Signed-off-by: Alan Tull <atull@xxxxxxxxxxxxxxxxxxxxx> > --- > v9: initial version added to this patchset > v10: s/fpga/FPGA/g > replace DT overlay example with slightly more complicated example > move to staging/simple-fpga-bus > v11: No change in this patch for v11 of the patch set > v12: Moved out of staging. > Changed to use FPGA bridges framework instead of resets > for bridges. > --- > .../devicetree/bindings/fpga/simple-fpga-bus.txt | 81 ++++++++++++++++++++ > 1 file changed, 81 insertions(+) > create mode 100644 Documentation/devicetree/bindings/fpga/simple-fpga-bus.txt > > diff --git a/Documentation/devicetree/bindings/fpga/simple-fpga-bus.txt b/Documentation/devicetree/bindings/fpga/simple-fpga-bus.txt > new file mode 100644 > index 0000000..2e742f7 > --- /dev/null > +++ b/Documentation/devicetree/bindings/fpga/simple-fpga-bus.txt > @@ -0,0 +1,81 @@ > +Simple FPGA Bus > +=============== > + > +A Simple FPGA Bus is a bus that handles configuring an FPGA and its bridges > +before populating the devices below its node. All this happens when a device > +tree overlay is added to the live tree. This document describes that device > +tree overlay. > + This is not really true, is it? The driver should work without applying the overlay, e.g. the bootloader might have already done it. > +Required properties: > +- compatible : should contain "simple-fpga-bus" > +- #address-cells, #size-cells, ranges: must be present to handle address space > + mapping for children. > + > +Optional properties: > +- fpga-mgr : should contain a phandle to a FPGA manager. > +- fpga-firmware : should contain the name of a FPGA image file located on the > + firmware search path. > +- partial-reconfig : boolean property should be defined if partial > + reconfiguration of the FPGA is to be done, otherwise full reconfiguration > + is done. > +- fpga-bridges : should contain a list of bridges that the bus will disable > + before programming the FPGA and then enable after the FPGA has been ^^^ ??? > + > +Example: > + > +/dts-v1/; > +/plugin/; > +/ { > + fragment@0 { > + target-path="/soc"; > + __overlay__ { > + #address-cells = <1>; > + #size-cells = <1>; > + > + bridge@0xff200000 { ^^ > + compatible = "simple-fpga-bus"; > + reg = <0xc0000000 0x20000000>, > + <0xff200000 0x00200000>; > + reg-names = "axi_h2f", "axi_h2f_lw"; > + > + #address-cells = <0x2>; > + #size-cells = <0x1>; > + > + ranges = <0x00000000 0x00000000 0xc0000000 0x00010000>, > + <0x00000001 0x00020000 0xff220000 0x00000008>, > + <0x00000001 0x00010040 0xff210040 0x00000020>; > + > + clocks = <0x2 0x2>; > + clock-names = "h2f_lw_axi_clock", "f2h_sdram0_clock"; > + > + fpga-mgr = <&hps_0_fpgamgr>; > + fpga-firmware = "soc_system.rbf"; > + > + fpga-bridges = <&hps_fpgabridge0>, <&hps_fpgabridge1>, <&hps_fpgabridge2>; > + > + onchip_memory2_0: memory@0x000000000 { ^^ > + device_type = "memory"; > + compatible = "ALTR,onchipmem-15.1"; > + reg = <0x00000000 0x00000000 0x00010000>; > + }; > + > + jtag_uart: serial@0x100020000 { ^^ > + compatible = "altr,juart-15.1", "altr,juart-1.0"; > + reg = <0x00000001 0x00020000 0x00000008>; > + interrupt-parent = <&intc>; > + interrupts = <0 42 4>; > + }; > + > + led_pio: gpio@0x100010040 { ^^ No 0x, please. > + compatible = "altr,pio-15.1", "altr,pio-1.0"; > + reg = <0x00000001 0x00010040 0x00000020>; > + altr,gpio-bank-width = <4>; > + resetvalue = <0>; > + #gpio-cells = <2>; > + gpio-controller; > + }; > + }; > + }; > + }; > +}; > + Regards, Steffen -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- 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