On Tue, 21 Feb 2017 15:51:18 +0100 Michal Simek michal.simek@xxxxxxxxxx wrote: ... >>> What's the hw configuration? >> >> I used Spartan-6 for testing. But the driver might work with other >> Xilinx FPGAs, so I didn't add exact hw description. > >ok. I expect this is spartan-6 next to another SoC and you use spi for >that another SoC and you connect. yes, it is MVEBU SoC, I'm using orion_spi driver. The connection is unidirectional SPI. >I think it will be good to extend that example and add also spi device >node to see where this should be placed. You mean the example in the binding description? I can add there something like: Usage example for full FPGA configuration: fpga-region0 { compatible = "fpga-region"; fpga-mgr = <&fpga_mgr_spi>; #address-cells = <0x1>; #size-cells = <0x1>; }; &spi1 { status = "okay"; fpga_mgr_spi: fpga-mgr@0 { compatible = "xlnx,fpga-slave-serial"; reg = <0>; spi-max-frequency = <60000000>; spi-cpha; done-gpios = <&gpio0 7 GPIO_ACTIVE_HIGH>; prog_b-gpios = <&gpio0 26 GPIO_ACTIVE_LOW>; }; }; Thanks, Anatolij -- 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