"Nori, Sekhar" <nsek...@ti.com> writes:

[...]

>> > +#ifdef CONFIG_CPU_FREQ
>>
>> How about leave this without an #ifdef, and make da850_init() have
>> the #ifdef so it is always called on boot.
>>
>> BTW, is there a reason to not have the CPUfreq settings be the
>> default?
>>
>
> Hmm, I thought ifdefery inside the function will get
> frowned upon.

It would.  You can't win...  ;)

> There is no reason not to use the CPUFreq setting as
> default (except that it is not the hardware reset default)
>
> On that thought may be I can just get it out of #ifdef CPU_FREQ
> altogether and have it called all the time.

...unless you do this.  That would indeed be the preferred option.

Of course, the call will be well commented as to why you reparent the
clocks by default.  :)

Kevin


_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to