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

Reply via email to