Hi Romain, On mer., nov. 30 2016, Romain Perier <romain.perier@xxxxxxxxxxxxxxxxxx> wrote: > Armada 3700 SoC has an SPI Controller, this commit adds the definition > of the SPI device node at the SoC level. > > Signed-off-by: Romain Perier <romain.perier@xxxxxxxxxxxxxxxxxx> > --- > > Changes in v2: > - Removed properties max-frequency and clock-frequency, it is no > longer required and not used by the DT-bindings. > > arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi > index e9bd587..63c2002 100644 > --- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi > @@ -98,6 +98,17 @@ > /* 32M internal register @ 0xd000_0000 */ > ranges = <0x0 0x0 0xd0000000 0x2000000>; > > + spi0: spi@10600 { > + compatible = "marvell,armada-3700-spi"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x10600 0x5d>; The last register is at the offset 0x1065C but according tot he datasheet the range address associated to this IP is from 0x10600 to 0x10FFF. In the first case the size of the register set should be 0x60 (each register is 32-bits). But I prefer that we register the full range so a size of 0xA00. Thanks, Gregory > + clocks = <&nb_periph_clk 7>; > + interrupts = <GIC_SPI 0 IRQ_TYPE_LEVEL_HIGH>; > + num-cs = <4>; > + status = "disabled"; > + }; > + > uart0: serial@12000 { > compatible = "marvell,armada-3700-uart"; > reg = <0x12000 0x400>; > -- > 2.9.3 > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- 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