On Sat, Nov 03, 2012 at 21:24:14, Shilimkar, Santosh wrote: > On Friday 02 November 2012 06:02 PM, Vaibhav Bedia wrote: > > Signed-off-by: Vaibhav Bedia <vaibhav.bedia@xxxxxx> > > --- > > arch/arm/boot/dts/am33xx.dtsi | 11 +++++++++++ > > 1 files changed, 11 insertions(+), 0 deletions(-) > > > > diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi > > index bb31bff..e2cbf24 100644 > > --- a/arch/arm/boot/dts/am33xx.dtsi > > +++ b/arch/arm/boot/dts/am33xx.dtsi > > @@ -210,5 +210,16 @@ > > interrupt-parent = <&intc>; > > interrupts = <91>; > > }; > > + > > + ocmcram: ocmcram@40300000 { > > + compatible = "ti,ocmcram"; > > + ti,hwmods = "ocmcram"; > > + ti,no_idle_on_suspend; > > + }; > Whats the intention behind adding OCMRAM ? > Sorry if I missed any comments from the cover letter ? > We need a mechanism to ensure that the clock to OCMC is kept running during boot and that it doesn't get disabled as part of the suspend sequence. Since the hwmod data for OCMC is already present and we have the no_idle_on_suspend flag for hwmod entries we get the desired behavior. This could also have been done via the clock tree but looks like we want to avoid adding leaf nodes in the clock data, hence the hwmod + DT approach. Regards, Vaibhav -- 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