From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Porter Subject: Re: [RESEND PATCH 3/4] ARM: dts: bcm281xx: Add i2c busses Date: Thu, 17 Oct 2013 08:14:54 -0400 Message-ID: <20131017121454.GZ2721@beef> References: <1381960909-1735-1-git-send-email-tim.kryger@linaro.org> <1381960909-1735-4-git-send-email-tim.kryger@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-doc-owner@vger.kernel.org To: Christian Daudt Cc: Tim Kryger , Mark Rutland , "devicetree@vger.kernel.org" , Pawel Moll , Ian Campbell , Wolfram Sang , Stephen Warren , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, Rob Herring , linux-i2c@vger.kernel.org, Rob Landley , Grant Likely , "linux-arm-kernel@lists.infradead.org" List-Id: linux-i2c@vger.kernel.org On Wed, Oct 16, 2013 at 09:32:51PM -0700, Christian Daudt wrote: > Hi Tim, > This patch does not work as it relies on the yet-unmerged clock code > for bcm11351. This patch does not apply to -rc, and when tweaked it > does not compile as it references non-existent bsc_clks. > Has the clk patches been submitted yet ? Applying this will have to > wait until those have been accepted. Hi Christian, Tim mentioned the dependencies in the cover letter. Sebastian's series, for which the pull request is already sent, and then Tim's own fixed clock series that is pending some review. -Matt > On Wed, Oct 16, 2013 at 3:01 PM, Tim Kryger wrote: > > Add the DTS nodes for all the i2c busses in the SoC. > > > > Signed-off-by: Tim Kryger > > Reviewed-by: Christian Daudt > > Reviewed-by: Matt Porter > > Reviewed-by: Markus Mayer > > --- > > arch/arm/boot/dts/bcm11351.dtsi | 40 ++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 40 insertions(+) > > > > diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351.dtsi > > index 1246885..4bfd7e3 100644 > > --- a/arch/arm/boot/dts/bcm11351.dtsi > > +++ b/arch/arm/boot/dts/bcm11351.dtsi > > @@ -146,6 +146,46 @@ > > status = "disabled"; > > }; > > > > + i2c@3e016000 { > > + compatible = "brcm,bcm11351-i2c", "brcm,kona-i2c"; > > + reg = <0x3e016000 0x80>; > > + interrupts = ; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&bsc1_clk>; > > + status = "disabled"; > > + }; > > + > > + i2c@3e017000 { > > + compatible = "brcm,bcm11351-i2c", "brcm,kona-i2c"; > > + reg = <0x3e017000 0x80>; > > + interrupts = ; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&bsc2_clk>; > > + status = "disabled"; > > + }; > > + > > + i2c@3e018000 { > > + compatible = "brcm,bcm11351-i2c", "brcm,kona-i2c"; > > + reg = <0x3e018000 0x80>; > > + interrupts = ; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&bsc3_clk>; > > + status = "disabled"; > > + }; > > + > > + i2c@3500d000 { > > + compatible = "brcm,bcm11351-i2c", "brcm,kona-i2c"; > > + reg = <0x3500d000 0x80>; > > + interrupts = ; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&pmu_bsc_clk>; > > + status = "disabled"; > > + }; > > + > > clocks { > > bsc1_clk: bsc1 { > > compatible = "fixed-clock"; > > -- > > 1.8.0.1 > > > > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel