Hi Sergei,
On Tue, Nov 30, 2010 at 17:06:25, Sergei Shtylyov wrote:
> > @@ -886,6 +916,19 @@ static struct cpufreq_frequency_table
> > da850_freq_table[] = {
> > };
> >
> > #ifdef CONFIG_REGULATOR
> > +static int da850_set_voltage(unsigned int index);
> > +static int da850_regulator_init(void);
> > +#endif
> > +
> > +static struct davinci_cpufreq_config cpufreq_info = {
> > + .freq_table =&da850_freq_table[0],
>
> & and [0] are not necessary.
Sure. Will remove.
>
> > @@ -928,12 +963,27 @@ static struct platform_device da850_cpufreq_device = {
> > .id = -1,
> > };
> >
> > +static unsigned int da850_max_speed = DA850_DEFAULT_SPEED;
> > +
> > +void __init da850_set_max_speed(unsigned int max)
> > +{
> > + da850_max_speed = max;
>
> Why not simply make this variable 'extern'?
Will make it extern.
Thanks,
Sekhar
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source