From mboxrd@z Thu Jan 1 00:00:00 1970 From: "G, Manjunath Kondaiah" Subject: Re: [RFC/PATCH 12/14] dt: omap3: beagle board: set clock freq for i2c devices Date: Wed, 10 Aug 2011 22:15:09 +0530 Message-ID: <20110810164509.GA2705@manju-desktop> References: <1312897232-4792-1-git-send-email-manjugk@ti.com> <1312897232-4792-13-git-send-email-manjugk@ti.com> <4E427CD0.7060206@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <4E427CD0.7060206@ti.com> Sender: linux-omap-owner@vger.kernel.org To: "Cousson, Benoit" Cc: "devicetree-discuss@lists.ozlabs.org" , "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "grant.likely@secretlab.ca" List-Id: devicetree@vger.kernel.org On Wed, Aug 10, 2011 at 02:42:56PM +0200, Cousson, Benoit wrote: > On 8/9/2011 4:10 PM, G, Manjunath Kondaiah wrote: > > > >Update omap3 beagle dts file with required clock frequencies for the i2c > >client devices existing on beagle board. > > > >Beagle custom board dts file is cleaned up so that it can coexist with omap3 > >soc dts file. > > > >Signed-off-by: G, Manjunath Kondaiah > >--- > > arch/arm/boot/dts/omap3-beagle-nunchuck.dts | 11 +---------- > > arch/arm/boot/dts/omap3-beagle.dts | 18 +++++++++++++++--- > > 2 files changed, 16 insertions(+), 13 deletions(-) > > > >diff --git a/arch/arm/boot/dts/omap3-beagle-nunchuck.dts b/arch/arm/boot/dts/omap3-beagle-nunchuck.dts > >index 2607be5..324ff86 100644 > >--- a/arch/arm/boot/dts/omap3-beagle-nunchuck.dts > >+++ b/arch/arm/boot/dts/omap3-beagle-nunchuck.dts > >@@ -1,16 +1,7 @@ > > /include/ "omap3-beagle.dts" > > > > / { > >- i2c@48072000 { > >- compatible = "ti,omap3-i2c"; > >- reg =<0x48072000 0x80>; > >- #address-cells =<1>; > >- #size-cells =<0>; > >- > >- eeprom@50 { > >- compatible = "at,at24c01"; > >- reg =< 0x50>; > >- }; > > This change should probably not be there. > > >+ i2c@2 { > > joystick@52 { > > compatible = "sparkfun,wiichuck"; > > It looks like someone is having fun with a beagle connected to a Wii > nunchuck:-) It's custom beagle belong to grant :) > > > reg =< 0x52>; > >diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts > >index 4439466..49a5ac7 100644 > >--- a/arch/arm/boot/dts/omap3-beagle.dts > >+++ b/arch/arm/boot/dts/omap3-beagle.dts > >@@ -1,7 +1,19 @@ > >-/dts-v1/; > >-/include/ "skeleton.dtsi" > >+/include/ "omap3-soc.dtsi" > > There is no need for the "-soc" postfix, otherwise all the other > SoCs inside DTS directory should have it. ok. -M From mboxrd@z Thu Jan 1 00:00:00 1970 From: manjugk@ti.com (G, Manjunath Kondaiah) Date: Wed, 10 Aug 2011 22:15:09 +0530 Subject: [RFC/PATCH 12/14] dt: omap3: beagle board: set clock freq for i2c devices In-Reply-To: <4E427CD0.7060206@ti.com> References: <1312897232-4792-1-git-send-email-manjugk@ti.com> <1312897232-4792-13-git-send-email-manjugk@ti.com> <4E427CD0.7060206@ti.com> Message-ID: <20110810164509.GA2705@manju-desktop> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Aug 10, 2011 at 02:42:56PM +0200, Cousson, Benoit wrote: > On 8/9/2011 4:10 PM, G, Manjunath Kondaiah wrote: > > > >Update omap3 beagle dts file with required clock frequencies for the i2c > >client devices existing on beagle board. > > > >Beagle custom board dts file is cleaned up so that it can coexist with omap3 > >soc dts file. > > > >Signed-off-by: G, Manjunath Kondaiah > >--- > > arch/arm/boot/dts/omap3-beagle-nunchuck.dts | 11 +---------- > > arch/arm/boot/dts/omap3-beagle.dts | 18 +++++++++++++++--- > > 2 files changed, 16 insertions(+), 13 deletions(-) > > > >diff --git a/arch/arm/boot/dts/omap3-beagle-nunchuck.dts b/arch/arm/boot/dts/omap3-beagle-nunchuck.dts > >index 2607be5..324ff86 100644 > >--- a/arch/arm/boot/dts/omap3-beagle-nunchuck.dts > >+++ b/arch/arm/boot/dts/omap3-beagle-nunchuck.dts > >@@ -1,16 +1,7 @@ > > /include/ "omap3-beagle.dts" > > > > / { > >- i2c at 48072000 { > >- compatible = "ti,omap3-i2c"; > >- reg =<0x48072000 0x80>; > >- #address-cells =<1>; > >- #size-cells =<0>; > >- > >- eeprom at 50 { > >- compatible = "at,at24c01"; > >- reg =< 0x50>; > >- }; > > This change should probably not be there. > > >+ i2c at 2 { > > joystick at 52 { > > compatible = "sparkfun,wiichuck"; > > It looks like someone is having fun with a beagle connected to a Wii > nunchuck:-) It's custom beagle belong to grant :) > > > reg =< 0x52>; > >diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts > >index 4439466..49a5ac7 100644 > >--- a/arch/arm/boot/dts/omap3-beagle.dts > >+++ b/arch/arm/boot/dts/omap3-beagle.dts > >@@ -1,7 +1,19 @@ > >-/dts-v1/; > >-/include/ "skeleton.dtsi" > >+/include/ "omap3-soc.dtsi" > > There is no need for the "-soc" postfix, otherwise all the other > SoCs inside DTS directory should have it. ok. -M