Hi Romain and Wolfram, On mar., nov. 08 2016, Romain Perier <romain.perier@xxxxxxxxxxxxxxxxxx> wrote: > The Armada 3700 has two i2c bus interface units, this commit adds the > definitions of the corresponding device nodes. It also enables the node > on the development board for this SoC. > > Signed-off-by: Romain Perier <romain.perier@xxxxxxxxxxxxxxxxxx> Acked-by: Gregory CLEMENT <gregory.clement@xxxxxxxxxxxxxxxxxx> This patch is OK for me but I would like to apply it myself on mvebu/dt. Indeed Thomas Petazzoni pointed that I made a typo on the peripheral clock label: nb_perih_clk instead of nb_periph_clk. I would like to fix it and by applying this patch I will avoid a merge conflict or worse a mismatch with the label name. Wolfram, once you will be OK with the patch 1 and 3, I will apply the patch 2. Thanks, Gregory > --- > arch/arm64/boot/dts/marvell/armada-3720-db.dts | 4 ++++ > arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 18 ++++++++++++++++++ > 2 files changed, 22 insertions(+) > > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-db.dts b/arch/arm64/boot/dts/marvell/armada-3720-db.dts > index 1372e9a6..16d84af 100644 > --- a/arch/arm64/boot/dts/marvell/armada-3720-db.dts > +++ b/arch/arm64/boot/dts/marvell/armada-3720-db.dts > @@ -62,6 +62,10 @@ > }; > }; > > +&i2c0 { > + status = "okay"; > +}; > + > /* CON3 */ > &sata { > status = "okay"; > diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi > index c476253..bf2d73d 100644 > --- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi > @@ -98,6 +98,24 @@ > /* 32M internal register @ 0xd000_0000 */ > ranges = <0x0 0x0 0xd0000000 0x2000000>; > > + i2c0: i2c@11000 { > + compatible = "marvell,armada-3700-i2c"; > + reg = <0x11000 0x24>; > + clocks = <&nb_perih_clk 10>; > + interrupts = <GIC_SPI 1 IRQ_TYPE_LEVEL_HIGH>; > + mrvl,i2c-fast-mode; > + status = "disabled"; > + }; > + > + i2c1: i2c@11080 { > + compatible = "marvell,armada-3700-i2c"; > + reg = <0x11080 0x24>; > + clocks = <&nb_perih_clk 9>; > + interrupts = <GIC_SPI 2 IRQ_TYPE_LEVEL_HIGH>; > + mrvl,i2c-fast-mode; > + 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