Re: [PATCH 1/4] PM / devfreq: Add function to set max/min frequency

[Javi Merino <javi.merino@arm.com>]

On Sat, Jul 04, 2015 at 03:44:08AM +0100, MyungJoo Ham wrote:
> On Fri, Jul 3, 2015 at 9:58 PM, Javi Merino <javi.merino@arm.com> wrote:
> > From: Ørjan Eide <orjan.eide@arm.com>
> >
> > Factor out the logic to set minimum and maximum frequency for the device
> > so that it can be used by other parts of the kernel.
> >
> > Cc: MyungJoo Ham <myungjoo.ham@samsung.com>
> > Cc: Kyungmin Park <kyungmin.park@samsung.com>
> > Signed-off-by: Ørjan Eide <orjan.eide@arm.com>
> 
> Dear Javi,
> 
> Is there reasons to name the function "devfreq_qos_set_[min|max]"? not
> "devfreq_set_[min|max]"? The functions may be used by non-PM-qos
> components as well (e.g., thermal).

No, there are no reasons for those names.  I'll rename them to
devfreq_set_[min|max]

> And I like the idea of using devfreq devices as cooling devices. :)

Yes, we would like to be able to use them as passive cooling devices,
like what we do with cpufreq for the cpus. 

> ps. I will be on a trip starting hours later for 8 days and then in a
> military training for 3 days. Please expect that my responses during
> the next 11-12 days will be very slow.

No worries.  Thanks,
Javi