Re: [PATCH] thermal/cpu_cooling: remove local cooling state variable

On Mon, 3 Aug 2015 08:43:25 +0530 Viresh Kumar wrote: > On 01-08-15, 17:04, Viresh Kumar wrote: > > On 31-07-15, 08:30, Radivoje Jovanovic wrote: > > > > I agree with you that this patch is trivial for the current > > > implementation since the notifier, as it is

Re: [PATCH 3/4] thermal/powerclamp: add cpu id for skylake h/s

+ Rui On Fri, 31 Jul 2015 08:07:45 -0700 Radivoje Jovanovic wrote: > From: Radivoje Jovanovic > > Add support for Intel Skylake H/S > > Signed-off-by: Radivoje Jovanovic > --- > drivers/thermal/intel_powerclamp.c | 1 + > 1 file changed, 1 insertion(+) > >

Re: [PATCH 4/4] thermal/powerclamp: add cpu id for Skylake u/y

+Rui On Fri, 31 Jul 2015 08:07:54 -0700 Radivoje Jovanovic wrote: > From: Radivoje Jovanovic > > Add support for Intel Skylake u/y > > Signed-off-by: Radivoje Jovanovic > --- > drivers/thermal/intel_powerclamp.c | 1 + > 1 file changed, 1 insertion(+) > >

Re: [PATCH 1/4] powercap / RAPL: Add support for Skylake H/S

On Sat, 01 Aug 2015 02:12:18 +0200 Hi Rafael, "Rafael J. Wysocki" wrote: > On Friday, July 31, 2015 08:07:10 AM Radivoje Jovanovic wrote: > > From: Radivoje Jovanovic > > > > This patche enabled RAPL to support Intel Skylake H/S > > > > Signed-off-

Re: [PATCH] thermal/cpu_cooling: remove local cooling state variable

to the > lowest frequency. Right ? > > On 30-07-15, 13:21, Radivoje Jovanovic wrote: > > In this case both userspace thermal solution and cpu_cooling are > > changing policy->max and the userspace solution will let governor > > or HW (depends on architecture) decide the clip

[PATCH 2/4] powercap / RAPL: Add support for Broadwell-H

From: Radivoje Jovanovic This patch enabled RAPL to support Broadwell-H Signed-off-by: Radivoje Jovanovic --- drivers/powercap/intel_rapl.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/powercap/intel_rapl.c b/drivers/powercap/intel_rapl.c index bca9620..a5eddfb 100644 --- a

[PATCH 3/4] thermal/powerclamp: add cpu id for skylake h/s

From: Radivoje Jovanovic Add support for Intel Skylake H/S Signed-off-by: Radivoje Jovanovic --- drivers/thermal/intel_powerclamp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/thermal/intel_powerclamp.c b/drivers/thermal/intel_powerclamp.c index 5820e85..63879a1 100644 --- a

[PATCH 4/4] thermal/powerclamp: add cpu id for Skylake u/y

From: Radivoje Jovanovic Add support for Intel Skylake u/y Signed-off-by: Radivoje Jovanovic --- drivers/thermal/intel_powerclamp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/thermal/intel_powerclamp.c b/drivers/thermal/intel_powerclamp.c index 63879a1..6e01723 100644 --- a

[PATCH 1/4] powercap / RAPL: Add support for Skylake H/S

From: Radivoje Jovanovic This patche enabled RAPL to support Intel Skylake H/S Signed-off-by: Radivoje Jovanovic --- drivers/powercap/intel_rapl.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/powercap/intel_rapl.c b/drivers/powercap/intel_rapl.c index 482b22d..bca9620 100644

Re: [PATCH] thermal/cpu_cooling: remove local cooling state variable

On Thu, 30 Jul 2015 13:35:41 +0530 Viresh Kumar wrote: Hi Viresh, > Cc'ing Rafael as well.. > > On 29-07-15, 17:46, Punit Agrawal wrote: > > [ adding Viresh ] > > Thanks. That earned me few more patches ;) > > > Radivoje Jovanovic writes: > > >

Re: [PATCH] thermal/cpu_cooling: remove local cooling state variable

On Wed, 29 Jul 2015 17:46:37 +0100 Punit Agrawal wrote: Hi Agarwal, > [ adding Viresh ] > > Radivoje Jovanovic writes: > > > Hi Agarwal, > > > > On Fri, 24 Jul 2015 16:26:12 +0100 > > Punit Agrawal wrote: > > > >> Radivoje Jovan

Re: [PATCH] thermal/cpu_cooling: remove local cooling state variable

Hi Agarwal, On Fri, 24 Jul 2015 16:26:12 +0100 Punit Agrawal wrote: > Radivoje Jovanovic writes: > > > From: Radivoje Jovanovic > > > > there is no need to keep local state variable. if another driver > > changes the policy under our feet the cpu_cooling driver

[PATCH] thermal/cpu_cooling: remove local cooling state variable

From: Radivoje Jovanovic there is no need to keep local state variable. if another driver changes the policy under our feet the cpu_cooling driver will have the wrong state. Get current state from the policy directly instead Signed-off-by: Radivoje Jovanovic --- drivers/thermal/cpu_cooling.c

thermal/powerclamp: add cpu id for denlow platform

From: Jacob Pan Add support for Intel Denlow UP server platform. Signed-off-by: Jacob Pan --- drivers/thermal/intel_powerclamp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/thermal/intel_powerclamp.c b/drivers/thermal/intel_powerclamp.c index 725718e..d534357 100644 --- a/driv