Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Sudeep Holla
On 22/11/17 15:34, Eduardo Valentin wrote: > On Wed, Nov 22, 2017 at 10:59:21AM +, Sudeep Holla wrote: >> (sorry for chiming in quite late) >> >> On 21/11/17 18:12, Eduardo Valentin wrote: >>> On Tue, Nov 21, 2017 at 06:00:07PM +, Javi Merino wrote: Hi, On Tue, Nov 21,

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Sudeep Holla
On 22/11/17 15:34, Eduardo Valentin wrote: > On Wed, Nov 22, 2017 at 10:59:21AM +, Sudeep Holla wrote: >> (sorry for chiming in quite late) >> >> On 21/11/17 18:12, Eduardo Valentin wrote: >>> On Tue, Nov 21, 2017 at 06:00:07PM +, Javi Merino wrote: Hi, On Tue, Nov 21,

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Eduardo Valentin
On Wed, Nov 22, 2017 at 04:40:13PM +0530, Viresh Kumar wrote: > On 22-11-17, 10:59, Sudeep Holla wrote: > > IMO, if the $subject code is expected to be used on Juno, then my answer > > is no if one can't test it reliably and also prove that static power > > really matters on Juno. So far, I have

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Eduardo Valentin
On Wed, Nov 22, 2017 at 04:40:13PM +0530, Viresh Kumar wrote: > On 22-11-17, 10:59, Sudeep Holla wrote: > > IMO, if the $subject code is expected to be used on Juno, then my answer > > is no if one can't test it reliably and also prove that static power > > really matters on Juno. So far, I have

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Eduardo Valentin
On Wed, Nov 22, 2017 at 10:59:21AM +, Sudeep Holla wrote: > (sorry for chiming in quite late) > > On 21/11/17 18:12, Eduardo Valentin wrote: > > On Tue, Nov 21, 2017 at 06:00:07PM +, Javi Merino wrote: > >> Hi, > >> > >> On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: >

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Eduardo Valentin
On Wed, Nov 22, 2017 at 10:59:21AM +, Sudeep Holla wrote: > (sorry for chiming in quite late) > > On 21/11/17 18:12, Eduardo Valentin wrote: > > On Tue, Nov 21, 2017 at 06:00:07PM +, Javi Merino wrote: > >> Hi, > >> > >> On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: >

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Sudeep Holla
On 22/11/17 11:10, Viresh Kumar wrote: > On 22-11-17, 10:59, Sudeep Holla wrote: >> IMO, if the $subject code is expected to be used on Juno, then my answer >> is no if one can't test it reliably and also prove that static power >> really matters on Juno. So far, I have heard both the above is

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Sudeep Holla
On 22/11/17 11:10, Viresh Kumar wrote: > On 22-11-17, 10:59, Sudeep Holla wrote: >> IMO, if the $subject code is expected to be used on Juno, then my answer >> is no if one can't test it reliably and also prove that static power >> really matters on Juno. So far, I have heard both the above is

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Viresh Kumar
On 22-11-17, 10:59, Sudeep Holla wrote: > IMO, if the $subject code is expected to be used on Juno, then my answer > is no if one can't test it reliably and also prove that static power > really matters on Juno. So far, I have heard both the above is not > possible. So please delete the code if

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Viresh Kumar
On 22-11-17, 10:59, Sudeep Holla wrote: > IMO, if the $subject code is expected to be used on Juno, then my answer > is no if one can't test it reliably and also prove that static power > really matters on Juno. So far, I have heard both the above is not > possible. So please delete the code if

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Viresh Kumar
On 22-11-17, 06:55, Viresh Kumar wrote: > On 21-11-17, 18:00, Javi Merino wrote: > > As Ionela pointed out earlier in the thread, the cpufreq driver for Juno > > was not acceptable for mainline because it used platform specific code. > > Can we get a link to that thread? I don't remember what I

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Viresh Kumar
On 22-11-17, 06:55, Viresh Kumar wrote: > On 21-11-17, 18:00, Javi Merino wrote: > > As Ionela pointed out earlier in the thread, the cpufreq driver for Juno > > was not acceptable for mainline because it used platform specific code. > > Can we get a link to that thread? I don't remember what I

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Sudeep Holla
(sorry for chiming in quite late) On 21/11/17 18:12, Eduardo Valentin wrote: > On Tue, Nov 21, 2017 at 06:00:07PM +, Javi Merino wrote: >> Hi, >> >> On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: >> [...] >> >> In a nutshell, mainline does not want platform specific code,

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-22 Thread Sudeep Holla
(sorry for chiming in quite late) On 21/11/17 18:12, Eduardo Valentin wrote: > On Tue, Nov 21, 2017 at 06:00:07PM +, Javi Merino wrote: >> Hi, >> >> On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: >> [...] >> >> In a nutshell, mainline does not want platform specific code,

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Viresh Kumar
On 21-11-17, 18:00, Javi Merino wrote: > As Ionela pointed out earlier in the thread, the cpufreq driver for Juno > was not acceptable for mainline because it used platform specific code. Can we get a link to that thread? I don't remember what I have commented earlier but the above doesn't seem

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Viresh Kumar
On 21-11-17, 18:00, Javi Merino wrote: > As Ionela pointed out earlier in the thread, the cpufreq driver for Juno > was not acceptable for mainline because it used platform specific code. Can we get a link to that thread? I don't remember what I have commented earlier but the above doesn't seem

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Lukasz Luba
On 21/11/17 19:13, Eduardo Valentin wrote: On Tue, Nov 21, 2017 at 07:05:46PM +0100, Daniel Lezcano wrote: On 21/11/2017 19:00, Javi Merino wrote: Hi, On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: [snip] As I said before, the minimal you guys (ARM and Linaro) can do is

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Lukasz Luba
On 21/11/17 19:13, Eduardo Valentin wrote: On Tue, Nov 21, 2017 at 07:05:46PM +0100, Daniel Lezcano wrote: On 21/11/2017 19:00, Javi Merino wrote: Hi, On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: [snip] As I said before, the minimal you guys (ARM and Linaro) can do is

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Eduardo Valentin
On Tue, Nov 21, 2017 at 07:05:46PM +0100, Daniel Lezcano wrote: > On 21/11/2017 19:00, Javi Merino wrote: > > Hi, > > > > On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: > > > > [snip] > > > >> As I said before, the minimal you guys (ARM and Linaro) can do is to at > >> least

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Eduardo Valentin
On Tue, Nov 21, 2017 at 07:05:46PM +0100, Daniel Lezcano wrote: > On 21/11/2017 19:00, Javi Merino wrote: > > Hi, > > > > On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: > > > > [snip] > > > >> As I said before, the minimal you guys (ARM and Linaro) can do is to at > >> least

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Eduardo Valentin
On Tue, Nov 21, 2017 at 06:00:07PM +, Javi Merino wrote: > Hi, > > On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: > > [snip] > > > As I said before, the minimal you guys (ARM and Linaro) can do is to at > > least upstream the Juno code! as a reference. Come on guys? what

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Eduardo Valentin
On Tue, Nov 21, 2017 at 06:00:07PM +, Javi Merino wrote: > Hi, > > On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: > > [snip] > > > As I said before, the minimal you guys (ARM and Linaro) can do is to at > > least upstream the Juno code! as a reference. Come on guys? what

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Daniel Lezcano
On 21/11/2017 19:00, Javi Merino wrote: > Hi, > > On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: > > [snip] > >> As I said before, the minimal you guys (ARM and Linaro) can do is to at >> least upstream the Juno code! as a reference. Come on guys? what is >> preventing you

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Daniel Lezcano
On 21/11/2017 19:00, Javi Merino wrote: > Hi, > > On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: > > [snip] > >> As I said before, the minimal you guys (ARM and Linaro) can do is to at >> least upstream the Juno code! as a reference. Come on guys? what is >> preventing you

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Javi Merino
Hi, On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: [snip] > As I said before, the minimal you guys (ARM and Linaro) can do is to at > least upstream the Juno code! as a reference. Come on guys? what is > preventing you to upstream Juno model? As Ionela pointed out earlier

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Javi Merino
Hi, On Tue, Nov 21, 2017 at 08:57:06AM -0800, Eduardo Valentin wrote: [snip] > As I said before, the minimal you guys (ARM and Linaro) can do is to at > least upstream the Juno code! as a reference. Come on guys? what is > preventing you to upstream Juno model? As Ionela pointed out earlier

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Daniel Lezcano
On 21/11/2017 18:09, Eduardo Valentin wrote: [ ... ] > Well, I really do not see the point of a ask to extend the current API > if have no single user of it. What are the current users problems with > the API? What needs to be improved? What are the problems? We cannot > tell, guess why? We have

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Daniel Lezcano
On 21/11/2017 18:09, Eduardo Valentin wrote: [ ... ] > Well, I really do not see the point of a ask to extend the current API > if have no single user of it. What are the current users problems with > the API? What needs to be improved? What are the problems? We cannot > tell, guess why? We have

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Eduardo Valentin
Hi, On Tue, Nov 21, 2017 at 11:30:44AM +, Ionela Voinescu wrote: > Hi guys, > > On 17/11/17 11:02, Punit Agrawal wrote: > > Hi Eduardo, > > > > Eduardo Valentin writes: > > > >> Hello, > >> > >> On Fri, Nov 17, 2017 at 12:31:41AM +0100, Rafael J. Wysocki wrote: > >>>

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Eduardo Valentin
Hi, On Tue, Nov 21, 2017 at 11:30:44AM +, Ionela Voinescu wrote: > Hi guys, > > On 17/11/17 11:02, Punit Agrawal wrote: > > Hi Eduardo, > > > > Eduardo Valentin writes: > > > >> Hello, > >> > >> On Fri, Nov 17, 2017 at 12:31:41AM +0100, Rafael J. Wysocki wrote: > >>> On Thursday, November

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Lukasz Luba
Hi Vincent, On 21/11/17 17:08, Vincent Guittot wrote: Hi Lukasz On 21 November 2017 at 16:56, Lukasz Luba wrote: On 21/11/17 14:06, Daniel Lezcano wrote: On 21/11/2017 12:30, Ionela Voinescu wrote: [ ... ] A DT model would be easy to support with the current code

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Lukasz Luba
Hi Vincent, On 21/11/17 17:08, Vincent Guittot wrote: Hi Lukasz On 21 November 2017 at 16:56, Lukasz Luba wrote: On 21/11/17 14:06, Daniel Lezcano wrote: On 21/11/2017 12:30, Ionela Voinescu wrote: [ ... ] A DT model would be easy to support with the current code but it would be very

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Eduardo Valentin
Hello folks, On Tue, Nov 21, 2017 at 05:08:34PM +0100, Vincent Guittot wrote: > Hi Lukasz > > On 21 November 2017 at 16:56, Lukasz Luba wrote: > > On 21/11/17 14:06, Daniel Lezcano wrote: > >> > >> On 21/11/2017 12:30, Ionela Voinescu wrote: > >> > >> [ ... ] > >> > >>> A

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Eduardo Valentin
Hello folks, On Tue, Nov 21, 2017 at 05:08:34PM +0100, Vincent Guittot wrote: > Hi Lukasz > > On 21 November 2017 at 16:56, Lukasz Luba wrote: > > On 21/11/17 14:06, Daniel Lezcano wrote: > >> > >> On 21/11/2017 12:30, Ionela Voinescu wrote: > >> > >> [ ... ] > >> > >>> A DT model would be easy

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Vincent Guittot
Hi Lukasz On 21 November 2017 at 16:56, Lukasz Luba wrote: > On 21/11/17 14:06, Daniel Lezcano wrote: >> >> On 21/11/2017 12:30, Ionela Voinescu wrote: >> >> [ ... ] >> >>> A DT model would be easy to support with the current code but it would >>> be very inaccurate. >> >>

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Vincent Guittot
Hi Lukasz On 21 November 2017 at 16:56, Lukasz Luba wrote: > On 21/11/17 14:06, Daniel Lezcano wrote: >> >> On 21/11/2017 12:30, Ionela Voinescu wrote: >> >> [ ... ] >> >>> A DT model would be easy to support with the current code but it would >>> be very inaccurate. >> >> >> Why ? >> >> [ ... ]

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Lukasz Luba
On 21/11/17 14:06, Daniel Lezcano wrote: On 21/11/2017 12:30, Ionela Voinescu wrote: [ ... ] A DT model would be easy to support with the current code but it would be very inaccurate. Why ? [ ... ] Hi all, The DT solution won't fly, the reason can be found below. I agree with Ionela

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Lukasz Luba
On 21/11/17 14:06, Daniel Lezcano wrote: On 21/11/2017 12:30, Ionela Voinescu wrote: [ ... ] A DT model would be easy to support with the current code but it would be very inaccurate. Why ? [ ... ] Hi all, The DT solution won't fly, the reason can be found below. I agree with Ionela

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Daniel Lezcano
On 21/11/2017 12:30, Ionela Voinescu wrote: [ ... ] > A DT model would be easy to support with the current code but it would > be very inaccurate. Why ? [ ... ] -- Linaro.org │ Open source software for ARM SoCs Follow Linaro:

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Daniel Lezcano
On 21/11/2017 12:30, Ionela Voinescu wrote: [ ... ] > A DT model would be easy to support with the current code but it would > be very inaccurate. Why ? [ ... ] -- Linaro.org │ Open source software for ARM SoCs Follow Linaro:

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Ionela Voinescu
Hi guys, On 17/11/17 11:02, Punit Agrawal wrote: > Hi Eduardo, > > Eduardo Valentin writes: > >> Hello, >> >> On Fri, Nov 17, 2017 at 12:31:41AM +0100, Rafael J. Wysocki wrote: >>> On Thursday, November 16, 2017 4:20:58 PM CET Viresh Kumar wrote: On 16-11-17, 15:02,

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-21 Thread Ionela Voinescu
Hi guys, On 17/11/17 11:02, Punit Agrawal wrote: > Hi Eduardo, > > Eduardo Valentin writes: > >> Hello, >> >> On Fri, Nov 17, 2017 at 12:31:41AM +0100, Rafael J. Wysocki wrote: >>> On Thursday, November 16, 2017 4:20:58 PM CET Viresh Kumar wrote: On 16-11-17, 15:02, Ionela Voinescu wrote:

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-17 Thread Viresh Kumar
On 17-11-17, 11:02, Punit Agrawal wrote: > I'd like to think this patchset has come out of a plan to develop > functionality on top but I don't know if that is the case. If you are talking about my patch series (?), then no, I don't have any plans on top of it. I just cleaned things up a bit.

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-17 Thread Viresh Kumar
On 17-11-17, 11:02, Punit Agrawal wrote: > I'd like to think this patchset has come out of a plan to develop > functionality on top but I don't know if that is the case. If you are talking about my patch series (?), then no, I don't have any plans on top of it. I just cleaned things up a bit.

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-17 Thread Punit Agrawal
Hi Eduardo, Eduardo Valentin writes: > Hello, > > On Fri, Nov 17, 2017 at 12:31:41AM +0100, Rafael J. Wysocki wrote: >> On Thursday, November 16, 2017 4:20:58 PM CET Viresh Kumar wrote: >> > On 16-11-17, 15:02, Ionela Voinescu wrote: >> > > When it was added in lsk 3.18 in

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-17 Thread Punit Agrawal
Hi Eduardo, Eduardo Valentin writes: > Hello, > > On Fri, Nov 17, 2017 at 12:31:41AM +0100, Rafael J. Wysocki wrote: >> On Thursday, November 16, 2017 4:20:58 PM CET Viresh Kumar wrote: >> > On 16-11-17, 15:02, Ionela Voinescu wrote: >> > > When it was added in lsk 3.18 in what was then a

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-16 Thread Daniel Lezcano
On 16/11/2017 03:47, Viresh Kumar wrote: > On 15-11-17, 19:17, Rafael J. Wysocki wrote: >> However, I would like to see a clear declaration from whoever is >> maintaining that code today that there is a plan in place to upstream >> it and that this plan will actually be acted on. And, better yet,

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-16 Thread Daniel Lezcano
On 16/11/2017 03:47, Viresh Kumar wrote: > On 15-11-17, 19:17, Rafael J. Wysocki wrote: >> However, I would like to see a clear declaration from whoever is >> maintaining that code today that there is a plan in place to upstream >> it and that this plan will actually be acted on. And, better yet,

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-16 Thread Eduardo Valentin
Hello, On Fri, Nov 17, 2017 at 12:31:41AM +0100, Rafael J. Wysocki wrote: > On Thursday, November 16, 2017 4:20:58 PM CET Viresh Kumar wrote: > > On 16-11-17, 15:02, Ionela Voinescu wrote: > > > When it was added in lsk 3.18 in what was then a thermal driver for Juno > > > it was believed to have

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-16 Thread Eduardo Valentin
Hello, On Fri, Nov 17, 2017 at 12:31:41AM +0100, Rafael J. Wysocki wrote: > On Thursday, November 16, 2017 4:20:58 PM CET Viresh Kumar wrote: > > On 16-11-17, 15:02, Ionela Voinescu wrote: > > > When it was added in lsk 3.18 in what was then a thermal driver for Juno > > > it was believed to have

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-16 Thread Rafael J. Wysocki
On Thursday, November 16, 2017 4:20:58 PM CET Viresh Kumar wrote: > On 16-11-17, 15:02, Ionela Voinescu wrote: > > When it was added in lsk 3.18 in what was then a thermal driver for Juno > > it was believed to have an effect in thermal mitigation, but that was > > not proven later as to justify

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-16 Thread Rafael J. Wysocki
On Thursday, November 16, 2017 4:20:58 PM CET Viresh Kumar wrote: > On 16-11-17, 15:02, Ionela Voinescu wrote: > > When it was added in lsk 3.18 in what was then a thermal driver for Juno > > it was believed to have an effect in thermal mitigation, but that was > > not proven later as to justify

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-16 Thread Viresh Kumar
On 16-11-17, 15:02, Ionela Voinescu wrote: > When it was added in lsk 3.18 in what was then a thermal driver for Juno > it was believed to have an effect in thermal mitigation, but that was > not proven later as to justify posting it upstream, and that is why the > code never made it in mainline.

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-16 Thread Viresh Kumar
On 16-11-17, 15:02, Ionela Voinescu wrote: > When it was added in lsk 3.18 in what was then a thermal driver for Juno > it was believed to have an effect in thermal mitigation, but that was > not proven later as to justify posting it upstream, and that is why the > code never made it in mainline.

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-16 Thread Ionela Voinescu
Hi guys, On 15/11/17 18:20, Eduardo Valentin wrote: > On Wed, Nov 15, 2017 at 07:17:49PM +0100, Rafael J. Wysocki wrote: >> On Wed, Nov 15, 2017 at 4:43 PM, Eduardo Valentin >> wrote: >>> On Wed, Nov 15, 2017 at 11:18:03AM +0100, Daniel Lezcano wrote: On 15/11/2017

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-16 Thread Ionela Voinescu
Hi guys, On 15/11/17 18:20, Eduardo Valentin wrote: > On Wed, Nov 15, 2017 at 07:17:49PM +0100, Rafael J. Wysocki wrote: >> On Wed, Nov 15, 2017 at 4:43 PM, Eduardo Valentin >> wrote: >>> On Wed, Nov 15, 2017 at 11:18:03AM +0100, Daniel Lezcano wrote: On 15/11/2017 10:19, Viresh Kumar

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Viresh Kumar
On 15-11-17, 19:17, Rafael J. Wysocki wrote: > However, I would like to see a clear declaration from whoever is > maintaining that code today that there is a plan in place to upstream > it and that this plan will actually be acted on. And, better yet, > *when* that can be expected to happen.

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Viresh Kumar
On 15-11-17, 19:17, Rafael J. Wysocki wrote: > However, I would like to see a clear declaration from whoever is > maintaining that code today that there is a plan in place to upstream > it and that this plan will actually be acted on. And, better yet, > *when* that can be expected to happen.

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Eduardo Valentin
On Wed, Nov 15, 2017 at 07:17:49PM +0100, Rafael J. Wysocki wrote: > On Wed, Nov 15, 2017 at 4:43 PM, Eduardo Valentin wrote: > > On Wed, Nov 15, 2017 at 11:18:03AM +0100, Daniel Lezcano wrote: > >> On 15/11/2017 10:19, Viresh Kumar wrote: > >> > No one has used it for the

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Eduardo Valentin
On Wed, Nov 15, 2017 at 07:17:49PM +0100, Rafael J. Wysocki wrote: > On Wed, Nov 15, 2017 at 4:43 PM, Eduardo Valentin wrote: > > On Wed, Nov 15, 2017 at 11:18:03AM +0100, Daniel Lezcano wrote: > >> On 15/11/2017 10:19, Viresh Kumar wrote: > >> > No one has used it for the last two and half years

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Rafael J. Wysocki
On Wed, Nov 15, 2017 at 4:43 PM, Eduardo Valentin wrote: > On Wed, Nov 15, 2017 at 11:18:03AM +0100, Daniel Lezcano wrote: >> On 15/11/2017 10:19, Viresh Kumar wrote: >> > No one has used it for the last two and half years (since it was >> > introduced by commit c36cf0717631

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Rafael J. Wysocki
On Wed, Nov 15, 2017 at 4:43 PM, Eduardo Valentin wrote: > On Wed, Nov 15, 2017 at 11:18:03AM +0100, Daniel Lezcano wrote: >> On 15/11/2017 10:19, Viresh Kumar wrote: >> > No one has used it for the last two and half years (since it was >> > introduced by commit c36cf0717631 ("thermal:

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Eduardo Valentin
On Wed, Nov 15, 2017 at 11:18:03AM +0100, Daniel Lezcano wrote: > On 15/11/2017 10:19, Viresh Kumar wrote: > > No one has used it for the last two and half years (since it was > > introduced by commit c36cf0717631 ("thermal: cpu_cooling: implement the > > power cooling device API")), get rid of

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Eduardo Valentin
On Wed, Nov 15, 2017 at 11:18:03AM +0100, Daniel Lezcano wrote: > On 15/11/2017 10:19, Viresh Kumar wrote: > > No one has used it for the last two and half years (since it was > > introduced by commit c36cf0717631 ("thermal: cpu_cooling: implement the > > power cooling device API")), get rid of

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Eduardo Valentin
On Wed, Nov 15, 2017 at 12:39:36PM +0100, Daniel Lezcano wrote: > On 15/11/2017 12:31, Javi Merino wrote: > > On Wed, Nov 15, 2017 at 02:49:48PM +0530, Viresh Kumar wrote: > >> No one has used it for the last two and half years (since it was > >> introduced by commit c36cf0717631 ("thermal:

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Eduardo Valentin
On Wed, Nov 15, 2017 at 12:39:36PM +0100, Daniel Lezcano wrote: > On 15/11/2017 12:31, Javi Merino wrote: > > On Wed, Nov 15, 2017 at 02:49:48PM +0530, Viresh Kumar wrote: > >> No one has used it for the last two and half years (since it was > >> introduced by commit c36cf0717631 ("thermal:

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Daniel Lezcano
On 15/11/2017 12:31, Javi Merino wrote: > On Wed, Nov 15, 2017 at 02:49:48PM +0530, Viresh Kumar wrote: >> No one has used it for the last two and half years (since it was >> introduced by commit c36cf0717631 ("thermal: cpu_cooling: implement the >> power cooling device API")), get rid of it. > >

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Daniel Lezcano
On 15/11/2017 12:31, Javi Merino wrote: > On Wed, Nov 15, 2017 at 02:49:48PM +0530, Viresh Kumar wrote: >> No one has used it for the last two and half years (since it was >> introduced by commit c36cf0717631 ("thermal: cpu_cooling: implement the >> power cooling device API")), get rid of it. > >

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Javi Merino
On Wed, Nov 15, 2017 at 02:49:48PM +0530, Viresh Kumar wrote: > No one has used it for the last two and half years (since it was > introduced by commit c36cf0717631 ("thermal: cpu_cooling: implement the > power cooling device API")), get rid of it. Linaro used it in lsk 3.18 for the cpufreq

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Javi Merino
On Wed, Nov 15, 2017 at 02:49:48PM +0530, Viresh Kumar wrote: > No one has used it for the last two and half years (since it was > introduced by commit c36cf0717631 ("thermal: cpu_cooling: implement the > power cooling device API")), get rid of it. Linaro used it in lsk 3.18 for the cpufreq

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Viresh Kumar
On 15-11-17, 11:18, Daniel Lezcano wrote: > Even if I agree that is not used to in the mainstream kernel, it is part > of the EAS which is currently merged in Android. Okay, I had chat with Daniel offline after looking at Android sources and here are the key-points: - No one is using this feature

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Viresh Kumar
On 15-11-17, 11:18, Daniel Lezcano wrote: > Even if I agree that is not used to in the mainstream kernel, it is part > of the EAS which is currently merged in Android. Okay, I had chat with Daniel offline after looking at Android sources and here are the key-points: - No one is using this feature

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Daniel Lezcano
On 15/11/2017 10:19, Viresh Kumar wrote: > No one has used it for the last two and half years (since it was > introduced by commit c36cf0717631 ("thermal: cpu_cooling: implement the > power cooling device API")), get rid of it. > > Cc: Javi Merino > Cc: Punit Agrawal

Re: [PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Daniel Lezcano
On 15/11/2017 10:19, Viresh Kumar wrote: > No one has used it for the last two and half years (since it was > introduced by commit c36cf0717631 ("thermal: cpu_cooling: implement the > power cooling device API")), get rid of it. > > Cc: Javi Merino > Cc: Punit Agrawal > Signed-off-by: Viresh

[PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Viresh Kumar
No one has used it for the last two and half years (since it was introduced by commit c36cf0717631 ("thermal: cpu_cooling: implement the power cooling device API")), get rid of it. Cc: Javi Merino Cc: Punit Agrawal Signed-off-by: Viresh Kumar

[PATCH 4/4] cpu_cooling: Drop static-power related stuff

2017-11-15 Thread Viresh Kumar
No one has used it for the last two and half years (since it was introduced by commit c36cf0717631 ("thermal: cpu_cooling: implement the power cooling device API")), get rid of it. Cc: Javi Merino Cc: Punit Agrawal Signed-off-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 106