Re: [RFC/PATCH 12/14] dt: omap3: beagle board: set clock freq for i2c devices

G, Manjunath Kondaiah
2011-08-10T16:45:28Z

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<manjugk@ti.com>
> >---
> > 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