Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760443AbbKTVWA (ORCPT ); Fri, 20 Nov 2015 16:22:00 -0500 Received: from mail-wm0-f47.google.com ([74.125.82.47]:38415 "EHLO mail-wm0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751895AbbKTVV6 (ORCPT ); Fri, 20 Nov 2015 16:21:58 -0500 Subject: Re: [PATCH 1/2] arm64: dts: berlin4ct: add I2C nodes for BG4CT To: Jisheng Zhang , robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, catalin.marinas@arm.com, will.deacon@arm.com References: <1448012821-7413-1-git-send-email-jszhang@marvell.com> <1448012821-7413-2-git-send-email-jszhang@marvell.com> Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org From: Sebastian Hesselbarth Message-ID: <564F8EF3.5010508@gmail.com> Date: Fri, 20 Nov 2015 22:21:55 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 In-Reply-To: <1448012821-7413-2-git-send-email-jszhang@marvell.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2756 Lines: 93 On 20.11.2015 10:47, Jisheng Zhang wrote: > The Marvell Berlin BG4CT SoC has 4 TWSI which are compatible with the > Synopsys DesignWare I2C driver. Add the corresponding nodes. > > Signed-off-by: Jisheng Zhang > --- > arch/arm64/boot/dts/marvell/berlin4ct.dtsi | 52 ++++++++++++++++++++++++++++++ > 1 file changed, 52 insertions(+) > > diff --git a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi > index cca4c41..39d0676 100644 > --- a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi > +++ b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi > @@ -232,6 +232,32 @@ > }; > }; > > + i2c0: i2c@1400 { > + compatible = "snps,designware-i2c"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x1400 0x100>; > + clocks = <&clk CLK_APBCORE>; This patch looks fine to me, except that clock node naming and clock indices may change. We should really postpone this series until we worked out clock. Sebastian > + i2c-sda-hold-time-ns = <35>; > + i2c-sda-falling-time-ns = <425>; > + i2c-scl-falling-time-ns = <205>; > + interrupts = <4>; > + status = "disabled"; > + }; > + > + i2c1: i2c@1800 { > + compatible = "snps,designware-i2c"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x1800 0x100>; > + clocks = <&clk CLK_APBCORE>; > + i2c-sda-hold-time-ns = <35>; > + i2c-sda-falling-time-ns = <425>; > + i2c-scl-falling-time-ns = <205>; > + interrupts = <5>; > + status = "disabled"; > + }; > + > aic: interrupt-controller@3800 { > compatible = "snps,dw-apb-ictl"; > reg = <0x3800 0x30>; > @@ -319,6 +345,32 @@ > }; > }; > > + i2c2: i2c@b000 { > + compatible = "snps,designware-i2c"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0xb000 0x100>; > + clocks = <&osc>; > + i2c-sda-hold-time-ns = <140>; > + i2c-sda-falling-time-ns = <500>; > + i2c-scl-falling-time-ns = <220>; > + interrupts = <6>; > + status = "disabled"; > + }; > + > + i2c3: i2c@c000 { > + compatible = "snps,designware-i2c"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0xc000 0x100>; > + clocks = <&osc>; > + i2c-sda-hold-time-ns = <140>; > + i2c-sda-falling-time-ns = <500>; > + i2c-scl-falling-time-ns = <220>; > + interrupts = <7>; > + status = "disabled"; > + }; > + > uart0: uart@d000 { > compatible = "snps,dw-apb-uart"; > reg = <0xd000 0x100>; > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/