Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-31 Thread Steve Muckle
On Tue, May 31, 2016 at 11:00:11AM +0530, Viresh Kumar wrote: > On 30-05-16, 08:31, Steve Muckle wrote: > > My goal here was to have the system operate in this case in a manner > > that is obviously not optimized (running at fmax), so the platform owner > > realizes that the cpufreq driver doesn't

Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-31 Thread Steve Muckle
On Tue, May 31, 2016 at 11:00:11AM +0530, Viresh Kumar wrote: > On 30-05-16, 08:31, Steve Muckle wrote: > > My goal here was to have the system operate in this case in a manner > > that is obviously not optimized (running at fmax), so the platform owner > > realizes that the cpufreq driver doesn't

Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-31 Thread Steve Muckle
On Tue, May 31, 2016 at 04:44:51PM +0530, Viresh Kumar wrote: > On 25-05-16, 19:52, Steve Muckle wrote: > > +unsigned int cpufreq_driver_resolve_freq(struct cpufreq_policy *policy, > > +unsigned int target_freq) > > +{ > > + struct cpufreq_frequency_table

Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-31 Thread Steve Muckle
On Tue, May 31, 2016 at 04:44:51PM +0530, Viresh Kumar wrote: > On 25-05-16, 19:52, Steve Muckle wrote: > > +unsigned int cpufreq_driver_resolve_freq(struct cpufreq_policy *policy, > > +unsigned int target_freq) > > +{ > > + struct cpufreq_frequency_table

Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-31 Thread Viresh Kumar
On 25-05-16, 19:52, Steve Muckle wrote: > +unsigned int cpufreq_driver_resolve_freq(struct cpufreq_policy *policy, > + unsigned int target_freq) > +{ > + struct cpufreq_frequency_table *freq_table; > + int index, retval; > + > +

Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-31 Thread Viresh Kumar
On 25-05-16, 19:52, Steve Muckle wrote: > +unsigned int cpufreq_driver_resolve_freq(struct cpufreq_policy *policy, > + unsigned int target_freq) > +{ > + struct cpufreq_frequency_table *freq_table; > + int index, retval; > + > +

Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-30 Thread Viresh Kumar
On 30-05-16, 08:31, Steve Muckle wrote: > My goal here was to have the system operate in this case in a manner > that is obviously not optimized (running at fmax), so the platform owner > realizes that the cpufreq driver doesn't fully support the schedutil > governor. > > I was originally going

Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-30 Thread Viresh Kumar
On 30-05-16, 08:31, Steve Muckle wrote: > My goal here was to have the system operate in this case in a manner > that is obviously not optimized (running at fmax), so the platform owner > realizes that the cpufreq driver doesn't fully support the schedutil > governor. > > I was originally going

Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-30 Thread Steve Muckle
On Thu, May 26, 2016 at 11:55:14AM +0530, Viresh Kumar wrote: > On 25-05-16, 19:52, Steve Muckle wrote: > > Cpufreq governors may need to know what a particular target frequency > > maps to in the driver without necessarily wanting to set the frequency. > > Support this operation via a new cpufreq

Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-30 Thread Steve Muckle
On Thu, May 26, 2016 at 11:55:14AM +0530, Viresh Kumar wrote: > On 25-05-16, 19:52, Steve Muckle wrote: > > Cpufreq governors may need to know what a particular target frequency > > maps to in the driver without necessarily wanting to set the frequency. > > Support this operation via a new cpufreq

Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-26 Thread Viresh Kumar
On 25-05-16, 19:52, Steve Muckle wrote: > Cpufreq governors may need to know what a particular target frequency > maps to in the driver without necessarily wanting to set the frequency. > Support this operation via a new cpufreq API, > cpufreq_driver_resolve_freq(). > > The above API will call a

Re: [PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-26 Thread Viresh Kumar
On 25-05-16, 19:52, Steve Muckle wrote: > Cpufreq governors may need to know what a particular target frequency > maps to in the driver without necessarily wanting to set the frequency. > Support this operation via a new cpufreq API, > cpufreq_driver_resolve_freq(). > > The above API will call a

[PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-25 Thread Steve Muckle
Cpufreq governors may need to know what a particular target frequency maps to in the driver without necessarily wanting to set the frequency. Support this operation via a new cpufreq API, cpufreq_driver_resolve_freq(). The above API will call a new cpufreq driver callback, resolve_freq(), if it

[PATCH v2 1/3] cpufreq: add resolve_freq driver callback

2016-05-25 Thread Steve Muckle
Cpufreq governors may need to know what a particular target frequency maps to in the driver without necessarily wanting to set the frequency. Support this operation via a new cpufreq API, cpufreq_driver_resolve_freq(). The above API will call a new cpufreq driver callback, resolve_freq(), if it