Messages in this thread | | | Date | Mon, 23 Nov 2015 17:53:33 +0900 | Subject | Re: Re: [PATCH 6/6] PM / devfreq: Set the min_freq and max_freq of devfreq device | From | Chanwoo Choi <> |
| |
On Mon, Nov 23, 2015 at 4:21 PM, MyungJoo Ham <myungjoo.ham@samsung.com> wrote: > [] >> > >> > The value 0 is used for min/max_freq to declare >> > that min/max_freq is deactivated. Therefore, it is not >> > required to do so; they are not intended to show the hardware >> > configuration as well. >> >> This case consider the devfreq device using OPP because devfreq_set_freq_table() >> get the number of OPP entry in OPP list before setting the min_freq/max_freq. >> If the devfreq device don't use the OPP entry, devfreq_set_freq_table() >> will return without any operation. >> >> IMHO, when devfreq device uses the OPP table including the frequency, >> min_freq/max_freq should show the correct value as CPUFREQ framework. >> > > The side effect of this patch shows up when opp_disable() and opp_enable() > are used.
Ah. You're right. I was not considering the the case of using opp_disable() and opp_enable(). I'll consider it again including the usage case of opp_diable/opp_enable.
Regards, Chanwoo Choi
| |