Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

2016-12-18 3:19 GMT+09:00 Tobias Jakobi : > Hey Chanwoo, > > > Chanwoo Choi wrote: >> 2016-12-18 0:13 GMT+09:00 Tobias Jakobi : >>> Hey guys, >>> >>> Chanwoo Choi wrote: Hi Lin, 2016-11-24 18:54 GMT+09:00 Chanwoo Choi

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

2016-12-18 3:19 GMT+09:00 Tobias Jakobi : > Hey Chanwoo, > > > Chanwoo Choi wrote: >> 2016-12-18 0:13 GMT+09:00 Tobias Jakobi : >>> Hey guys, >>> >>> Chanwoo Choi wrote: Hi Lin, 2016-11-24 18:54 GMT+09:00 Chanwoo Choi : > Hi Lin, > > On 2016년 11월 24일 18:28, Chanwoo Choi

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hey Chanwoo, Chanwoo Choi wrote: > 2016-12-18 0:13 GMT+09:00 Tobias Jakobi : >> Hey guys, >> >> Chanwoo Choi wrote: >>> Hi Lin, >>> >>> 2016-11-24 18:54 GMT+09:00 Chanwoo Choi : Hi Lin, On 2016년 11월 24일 18:28, Chanwoo Choi

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hey Chanwoo, Chanwoo Choi wrote: > 2016-12-18 0:13 GMT+09:00 Tobias Jakobi : >> Hey guys, >> >> Chanwoo Choi wrote: >>> Hi Lin, >>> >>> 2016-11-24 18:54 GMT+09:00 Chanwoo Choi : Hi Lin, On 2016년 11월 24일 18:28, Chanwoo Choi wrote: > Hi Lin, > > On 2016년 11월 24일 17:34,

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

2016-12-18 0:13 GMT+09:00 Tobias Jakobi : > Hey guys, > > Chanwoo Choi wrote: >> Hi Lin, >> >> 2016-11-24 18:54 GMT+09:00 Chanwoo Choi : >>> Hi Lin, >>> >>> On 2016년 11월 24일 18:28, Chanwoo Choi wrote: Hi Lin, On 2016년 11월 24일

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

2016-12-18 0:13 GMT+09:00 Tobias Jakobi : > Hey guys, > > Chanwoo Choi wrote: >> Hi Lin, >> >> 2016-11-24 18:54 GMT+09:00 Chanwoo Choi : >>> Hi Lin, >>> >>> On 2016년 11월 24일 18:28, Chanwoo Choi wrote: Hi Lin, On 2016년 11월 24일 17:34, hl wrote: > Hi Chanwoo Choi, > > >

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hey guys, Chanwoo Choi wrote: > Hi Lin, > > 2016-11-24 18:54 GMT+09:00 Chanwoo Choi : >> Hi Lin, >> >> On 2016년 11월 24일 18:28, Chanwoo Choi wrote: >>> Hi Lin, >>> >>> On 2016년 11월 24일 17:34, hl wrote: Hi Chanwoo Choi, On 2016年11月24日 16:16, Chanwoo Choi

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hey guys, Chanwoo Choi wrote: > Hi Lin, > > 2016-11-24 18:54 GMT+09:00 Chanwoo Choi : >> Hi Lin, >> >> On 2016년 11월 24일 18:28, Chanwoo Choi wrote: >>> Hi Lin, >>> >>> On 2016년 11월 24일 17:34, hl wrote: Hi Chanwoo Choi, On 2016年11月24日 16:16, Chanwoo Choi wrote: > Hi Lin,

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Lin, 2016-11-24 18:54 GMT+09:00 Chanwoo Choi : > Hi Lin, > > On 2016년 11월 24일 18:28, Chanwoo Choi wrote: >> Hi Lin, >> >> On 2016년 11월 24일 17:34, hl wrote: >>> Hi Chanwoo Choi, >>> >>> >>> On 2016年11月24日 16:16, Chanwoo Choi wrote: Hi Lin, On 2016년 11월 24일

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Lin, 2016-11-24 18:54 GMT+09:00 Chanwoo Choi : > Hi Lin, > > On 2016년 11월 24일 18:28, Chanwoo Choi wrote: >> Hi Lin, >> >> On 2016년 11월 24일 17:34, hl wrote: >>> Hi Chanwoo Choi, >>> >>> >>> On 2016年11月24日 16:16, Chanwoo Choi wrote: Hi Lin, On 2016년 11월 24일 16:34, hl wrote: >

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Lin, On 2016년 11월 24일 18:28, Chanwoo Choi wrote: > Hi Lin, > > On 2016년 11월 24일 17:34, hl wrote: >> Hi Chanwoo Choi, >> >> >> On 2016年11月24日 16:16, Chanwoo Choi wrote: >>> Hi Lin, >>> >>> On 2016년 11월 24일 16:34, hl wrote: Hi Chanwoo Choi, I think the

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Lin, On 2016년 11월 24일 18:28, Chanwoo Choi wrote: > Hi Lin, > > On 2016년 11월 24일 17:34, hl wrote: >> Hi Chanwoo Choi, >> >> >> On 2016年11月24日 16:16, Chanwoo Choi wrote: >>> Hi Lin, >>> >>> On 2016년 11월 24일 16:34, hl wrote: Hi Chanwoo Choi, I think the

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Lin, On 2016년 11월 24일 17:34, hl wrote: > Hi Chanwoo Choi, > > > On 2016年11月24日 16:16, Chanwoo Choi wrote: >> Hi Lin, >> >> On 2016년 11월 24일 16:34, hl wrote: >>> Hi Chanwoo Choi, >>> >>> I think the dev_pm_opp_get_suspend_opp() have implement most of >>> the funtion, all we need is just

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Lin, On 2016년 11월 24일 17:34, hl wrote: > Hi Chanwoo Choi, > > > On 2016年11月24日 16:16, Chanwoo Choi wrote: >> Hi Lin, >> >> On 2016년 11월 24일 16:34, hl wrote: >>> Hi Chanwoo Choi, >>> >>> I think the dev_pm_opp_get_suspend_opp() have implement most of >>> the funtion, all we need is just

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Chanwoo Choi, On 2016年11月24日 16:16, Chanwoo Choi wrote: Hi Lin, On 2016년 11월 24일 16:34, hl wrote: Hi Chanwoo Choi, I think the dev_pm_opp_get_suspend_opp() have implement most of the funtion, all we need is just define the node in dts, like following: _opp_table { opp06 {

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Chanwoo Choi, On 2016年11月24日 16:16, Chanwoo Choi wrote: Hi Lin, On 2016년 11월 24일 16:34, hl wrote: Hi Chanwoo Choi, I think the dev_pm_opp_get_suspend_opp() have implement most of the funtion, all we need is just define the node in dts, like following: _opp_table { opp06 {

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Lin, On 2016년 11월 24일 16:34, hl wrote: > Hi Chanwoo Choi, > > I think the dev_pm_opp_get_suspend_opp() have implement most of > the funtion, all we need is just define the node in dts, like following: > > _opp_table { > opp06 { > opp-suspend; > }; > }; Two approaches use

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Lin, On 2016년 11월 24일 16:34, hl wrote: > Hi Chanwoo Choi, > > I think the dev_pm_opp_get_suspend_opp() have implement most of > the funtion, all we need is just define the node in dts, like following: > > _opp_table { > opp06 { > opp-suspend; > }; > }; Two approaches use

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Chanwoo Choi, I think the dev_pm_opp_get_suspend_opp() have implement most of the funtion, all we need is just define the node in dts, like following: _opp_table { opp06 { opp-suspend; }; }; so i think my way semm more simple. On 2016年11月24日 15:10, Chanwoo Choi wrote:

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

Hi Chanwoo Choi, I think the dev_pm_opp_get_suspend_opp() have implement most of the funtion, all we need is just define the node in dts, like following: _opp_table { opp06 { opp-suspend; }; }; so i think my way semm more simple. On 2016年11月24日 15:10, Chanwoo Choi wrote:

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

+ Tobias Jakobi, Hi Lin, We need to discuss how to support the suspend-opp of devfreq device. Now, there are two patch thread for suspend-opp of devfreq. The Lin's approach modify the devfreq_suspend_device() to support suspend-opp. The Tobias's approach[1] add new devfreq_suspend() and then

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

+ Tobias Jakobi, Hi Lin, We need to discuss how to support the suspend-opp of devfreq device. Now, there are two patch thread for suspend-opp of devfreq. The Lin's approach modify the devfreq_suspend_device() to support suspend-opp. The Tobias's approach[1] add new devfreq_suspend() and then

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

On Thu, Nov 24, 2016 at 11:18 AM, hl wrote: > Hi MyungJoo Ham, [] >> >> We still need to sync the all status even i call target() in >> devfreq_suspend/resume_device >> directly, so still need update_devfreq() other setp except >> devfreq->governor->get_target_freq(devfreq,

Re: [PATCH v1 & v6 1/2] PM/devfreq: add suspend frequency support

On Thu, Nov 24, 2016 at 11:18 AM, hl wrote: > Hi MyungJoo Ham, [] >> >> We still need to sync the all status even i call target() in >> devfreq_suspend/resume_device >> directly, so still need update_devfreq() other setp except >> devfreq->governor->get_target_freq(devfreq, ); > > And i think it