On Thu, Jul 14, 2011 at 07:57:16AM +0900, Grant Likely wrote: > On Thu, Jul 14, 2011 at 7:06 AM, G, Manjunath Kondaiah <manjugk@xxxxxx> wrote: > > > > Add omap3 SoC file for handling omap3 SoC i2c controllers existing > > on l4-core bus. > > > > Out of three i2c controllers, first i2c controller is interfaced with > > PMIC on all the boards of OMAP3. The clock for i2c controllers are > > controlled through omap hwmod framework hence first i2c controller > > device registration through dt is disabled till hwmod dependencies > > are resolved. > > > > Signed-off-by: G, Manjunath Kondaiah <manjugk@xxxxxx> > > --- > > arch/arm/boot/dts/omap3-soc.dtsi | 65 ++++++++++++++++++++++++++++++++++++++ > > 1 files changed, 65 insertions(+), 0 deletions(-) > > create mode 100644 arch/arm/boot/dts/omap3-soc.dtsi > > > > diff --git a/arch/arm/boot/dts/omap3-soc.dtsi b/arch/arm/boot/dts/omap3-soc.dtsi > > new file mode 100644 > > index 0000000..f186a32 > > --- /dev/null > > +++ b/arch/arm/boot/dts/omap3-soc.dtsi > > @@ -0,0 +1,65 @@ > > +/* > > + * Device Tree Source for OMAP3 SoC > > + * > > + * Copyright (C) 2011 Texas Instruments Incorporated - http://www.ti.com/ > > + * > > + * This file is licensed under the terms of the GNU General Public License > > + * version 2. This program is licensed "as is" without any warranty of any > > + * kind, whether express or implied. > > + */ > > + > > +/dts-v1/; > > +/include/ "skeleton.dtsi" > > + > > +/ { > > + #address-cells = <1>; > > + #size-cells = <1>; > > + model = "ti,omap3"; > > You can drop the model property here since this doesn't define a board. ok > > > + compatible = "ti,omap3"; > > + > > + intc: interrupt-controller@0x48200000 { > > + compatible = "ti,omap3-intc", "arm,intc"; > > Which arm intc controller? For any new 'compatible' value you define, > the patch needs to include documentation for it in > Documentation/devicetree/bindings. for time being, I can drop this and introduce later with documentation. > > > + interrupt-controller; > > + #interrupt-cells = <1>; > > + reg = <0x48200000 0x1000>; > > + }; > > + > > + l4-core { > > + compatible = "ti,l4-core"; > > Probably should be "ti,omap3-l4-core"? ok. > > > + #address-cells = <1>; > > + #size-cells = <1>; > > + ranges = <0 0x48000000 0x1000000>; > > + > > + i2c@1 { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + compatible = "ti,omap3-i2c"; > > + reg = <0x70000 0x100>; > > + interrupts = < 88 >; > > + status = "disabled"; > > Drop the 'status' properties. I know the current tegra code does > this, but I'd prefer devices to be enabled by default and for boards > to explicitly disable them instead of the other way around. wanted to disable i2c1 by default. i will remove. -Manjunath -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
