Hello Vishwanath,

On Fri, 8 Jan 2010, Sripathy, Vishwanath wrote:

> > From: Paul Walmsley [mailto:[email protected]]
> > > On Thu, 7 Jan 2010, Vishwanath BS wrote:
> > >
> > > +static struct clk dpll4_ck_3630 __initdata = {
> > 
> > Marking these as __initdata will crash the system for DVFS changes after
> > the kernel boots.  Please fix.
> 
> Sorry I did not understand why it should crash the system for DVFS? 
> dpll4_ck_3630 is not used anywhere after it's values are copied into 
> dpll4_ck as part of clock_init. Same is the case with others.

Sorry, you're right - your code copies the entire structure and not just 
the pointer. 


- Paul
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to