On Fri, Sep 07, 2012 at 18:29:36, Porter, Matt wrote: > On Thu, Sep 06, 2012 at 12:30:15PM +0530, Philip, Avinash wrote: > > Add McSPI data node to AM33XX device tree file. The McSPI module (and so > > as the driver) is reused from OMAP4. > > > > Signed-off-by: Philip, Avinash <avinashphilip@xxxxxx> > > --- > > Resenting patch because ARM & OMAP mailing list was not copied. > > > > :100644 100644 bb31bff... 6b469bd... M arch/arm/boot/dts/am33xx.dtsi > > arch/arm/boot/dts/am33xx.dtsi | 25 +++++++++++++++++++++++++ > > 1 files changed, 25 insertions(+), 0 deletions(-) > > > > diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi > > index bb31bff..6b469bd 100644 > > --- a/arch/arm/boot/dts/am33xx.dtsi > > +++ b/arch/arm/boot/dts/am33xx.dtsi > > @@ -210,5 +210,30 @@ > > interrupt-parent = <&intc>; > > interrupts = <91>; > > }; > > + > > + spi0: spi@48030000 { > > + compatible = "ti,omap4-mcspi"; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + reg = <0x4830000 0x400>; > > Please fix this typo, should be 0x48030000. > > With the typo fixed, it's working on bone for me: Thanks for review and testing. I didn't notice the issue as currently hwmod fills up reg resource data & hence I missed it. I will correct & re-send. Thanks Avinash > > Tested-by: Matt Porter <mporter@xxxxxx> > > -Matt > -- 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