On Thu, Jan 5, 2017 at 10:28 AM, Alan Tull <atull@xxxxxxxxxx> wrote: > On Wed, Jan 4, 2017 at 6:21 PM, Dinh Nguyen <dinguyen@xxxxxxxxxx> wrote: >> From: Alan Tull <atull@xxxxxxxxxxxxxxxxxxxxx> >> >> Add h2f and lwh2f bridges. >> Add base FPGA Region to support DT overlays for FPGA programming. >> Add l3regs. >> >> Signed-off-by: Alan Tull <atull@xxxxxxxxxxxxxxxxxxxxx> >> Signed-off-by: Dinh Nguyen <dinguyen@xxxxxxxxxx> >> --- >> arch/arm/boot/dts/socfpga.dtsi | 31 +++++++++++++++++++++++++++++++ >> 1 file changed, 31 insertions(+) >> >> diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi >> index de29172..dccc281 100644 >> --- a/arch/arm/boot/dts/socfpga.dtsi >> +++ b/arch/arm/boot/dts/socfpga.dtsi >> @@ -93,6 +93,16 @@ >> }; >> }; >> >> + base_fpga_region { >> + compatible = "fpga-region"; >> + fpga-mgr = <&fpgamgr0>; >> + fpga-bridges = <&fpga_bridge0>, <&fpga_bridge1>; > > Hi Dinh, > > We want to get rid of the 'fpga-bridges' line. > >> + >> + #address-cells = <0x1>; >> + #size-cells = <0x1>; >> + ranges = <0 0xff200000 0x100000>; > > Should get rid of the ranges line here too. The 'fpga-bridges' and > 'ranges' line can be added in the overlay. > > Alan > >> + }; >> + >> can0: can@ffc00000 { >> compatible = "bosch,d_can"; >> reg = <0xffc00000 0x1000>; >> @@ -513,6 +523,22 @@ >> }; >> }; >> >> + fpga_bridge0: fpga_bridge@ff400000 { >> + compatible = "altr,socfpga-lwhps2fpga-bridge"; >> + reg = <0xff400000 0x100000>; >> + resets = <&rst LWHPS2FPGA_RESET>; >> + reset-names = "lwhps2fpga"; The driver doesn't need 'reset-names' here or below for fpga_bridge1. >> + clocks = <&l4_main_clk>; >> + }; >> + >> + fpga_bridge1: fpga_bridge@ff500000 { >> + compatible = "altr,socfpga-hps2fpga-bridge"; >> + reg = <0xff500000 0x10000>; >> + resets = <&rst HPS2FPGA_RESET>; >> + reset-names = "hps2fpga"; >> + clocks = <&l4_main_clk>; >> + }; >> + >> fpgamgr0: fpgamgr@ff706000 { >> compatible = "altr,socfpga-fpga-mgr"; >> reg = <0xff706000 0x1000 >> @@ -694,6 +720,11 @@ >> arm,prefetch-offset = <7>; >> }; >> >> + l3regs@0xff800000 { >> + compatible = "altr,l3regs", "syscon"; >> + reg = <0xff800000 0x1000>; >> + }; >> + >> mmc: dwmmc0@ff704000 { >> compatible = "altr,socfpga-dw-mshc"; >> reg = <0xff704000 0x1000>; >> -- >> 2.7.4 >> >> -- >> 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 -- 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