Re: [PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-08-02 Thread Chanwoo Choi
Hi Matthias, On 2018년 08월 03일 09:24, Matthias Kaehlcke wrote: > On Fri, Aug 03, 2018 at 09:03:30AM +0900, Chanwoo Choi wrote: >> Hi Matthias, >> >> On 2018년 08월 03일 08:36, Matthias Kaehlcke wrote: >>> Hi Chanwoo, >>> >>> this patch and "PM / devfreq: Fix handling of min/max_freq == 0" >>> address

Re: [PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-08-02 Thread Chanwoo Choi
Hi Matthias, On 2018년 08월 03일 09:24, Matthias Kaehlcke wrote: > On Fri, Aug 03, 2018 at 09:03:30AM +0900, Chanwoo Choi wrote: >> Hi Matthias, >> >> On 2018년 08월 03일 08:36, Matthias Kaehlcke wrote: >>> Hi Chanwoo, >>> >>> this patch and "PM / devfreq: Fix handling of min/max_freq == 0" >>> address

Re: [PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-08-02 Thread Matthias Kaehlcke
On Fri, Aug 03, 2018 at 09:03:30AM +0900, Chanwoo Choi wrote: > Hi Matthias, > > On 2018년 08월 03일 08:36, Matthias Kaehlcke wrote: > > Hi Chanwoo, > > > > this patch and "PM / devfreq: Fix handling of min/max_freq == 0" > > address issues not directly related with the throttler. It seems it > >

Re: [PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-08-02 Thread Matthias Kaehlcke
On Fri, Aug 03, 2018 at 09:03:30AM +0900, Chanwoo Choi wrote: > Hi Matthias, > > On 2018년 08월 03일 08:36, Matthias Kaehlcke wrote: > > Hi Chanwoo, > > > > this patch and "PM / devfreq: Fix handling of min/max_freq == 0" > > address issues not directly related with the throttler. It seems it > >

Re: [PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-08-02 Thread Chanwoo Choi
Hi Matthias, On 2018년 08월 03일 08:36, Matthias Kaehlcke wrote: > Hi Chanwoo, > > this patch and "PM / devfreq: Fix handling of min/max_freq == 0" > address issues not directly related with the throttler. It seems it > could still take a while for the throttler to move forward, do you > want me to

Re: [PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-08-02 Thread Chanwoo Choi
Hi Matthias, On 2018년 08월 03일 08:36, Matthias Kaehlcke wrote: > Hi Chanwoo, > > this patch and "PM / devfreq: Fix handling of min/max_freq == 0" > address issues not directly related with the throttler. It seems it > could still take a while for the throttler to move forward, do you > want me to

Re: [PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-08-02 Thread Matthias Kaehlcke
Hi Chanwoo, this patch and "PM / devfreq: Fix handling of min/max_freq == 0" address issues not directly related with the throttler. It seems it could still take a while for the throttler to move forward, do you want me to spin out these two patches so that they can get merged independently from

Re: [PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-08-02 Thread Matthias Kaehlcke
Hi Chanwoo, this patch and "PM / devfreq: Fix handling of min/max_freq == 0" address issues not directly related with the throttler. It seems it could still take a while for the throttler to move forward, do you want me to spin out these two patches so that they can get merged independently from

Re: [PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-07-03 Thread Chanwoo Choi
Hi Matthias, On 2018년 07월 04일 08:46, Matthias Kaehlcke wrote: > Several governors use the user space limits df->min/max_freq to adjust > the target frequency. This is not necessary, since update_devfreq() > already takes care of this. Instead the governor can request the available > min/max

Re: [PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-07-03 Thread Chanwoo Choi
Hi Matthias, On 2018년 07월 04일 08:46, Matthias Kaehlcke wrote: > Several governors use the user space limits df->min/max_freq to adjust > the target frequency. This is not necessary, since update_devfreq() > already takes care of this. Instead the governor can request the available > min/max

[PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-07-03 Thread Matthias Kaehlcke
Several governors use the user space limits df->min/max_freq to adjust the target frequency. This is not necessary, since update_devfreq() already takes care of this. Instead the governor can request the available min/max frequency by setting the target frequency to DEVFREQ_MIN/MAX_FREQ and let

[PATCH v5 03/12] PM / devfreq: Don't adjust to user limits in governors

2018-07-03 Thread Matthias Kaehlcke
Several governors use the user space limits df->min/max_freq to adjust the target frequency. This is not necessary, since update_devfreq() already takes care of this. Instead the governor can request the available min/max frequency by setting the target frequency to DEVFREQ_MIN/MAX_FREQ and let