[PATCH v4 0/3] thermal: mediatek: Add cpu power cooling model

2015-11-27 Thread Dawei Chien
Use Intelligent Power Allocation (IPA) technical to add dynamic power model for binding CPU thermal zone. The power allocator governor allocates power budget to control CPU temperature. Power Allocator governor is able to keep SOC temperature within a defined temperature range to avoid SOC overhea

Re: [PATCH v3 1/2] thermal: mediatek: Add cpu power cooling model.

2015-11-12 Thread Viresh Kumar
On 10-11-15, 11:20, Javi Merino wrote: > The way they are described here is useful only for this platform, but > it's not generic. It only takes into account voltage as (I assume) > it's the only variable that affects it in this implementation. A > generalized version of the static power should t

Re: [PATCH v3 0/2] thermal: mediatek: Add cpu power cooling model

2015-11-11 Thread dawei chien
On Fri, 2015-11-06 at 08:50 +0530, Viresh Kumar wrote: > On 05-11-15, 19:09, dawei chien wrote: > > Thank you for your kindly explaining, now I could understand what I > > miss, I will send device tree binding on next version such like > > following description. > > > > --- a/Documentation/devicet

Re: [PATCH v3 1/2] thermal: mediatek: Add cpu power cooling model.

2015-11-11 Thread Javi Merino
On Tue, Nov 10, 2015 at 10:41:22AM -0800, Eduardo Valentin wrote: > On Tue, Nov 10, 2015 at 11:20:18AM +, Javi Merino wrote: > > On Fri, Nov 06, 2015 at 08:54:33AM +0530, Viresh Kumar wrote: > > > Cc'ing Javi (which you should have as he wrote the power-thing for > > > cpu-cooling). > > > > >

Re: [PATCH v3 1/2] thermal: mediatek: Add cpu power cooling model.

2015-11-10 Thread Eduardo Valentin
On Tue, Nov 10, 2015 at 11:20:18AM +, Javi Merino wrote: > On Fri, Nov 06, 2015 at 08:54:33AM +0530, Viresh Kumar wrote: > > Cc'ing Javi (which you should have as he wrote the power-thing for > > cpu-cooling). > > > > On 05-11-15, 19:10, dawei chien wrote: > > > This is because our platform cu

Re: [PATCH v3 1/2] thermal: mediatek: Add cpu power cooling model.

2015-11-10 Thread Javi Merino
On Fri, Nov 06, 2015 at 08:54:33AM +0530, Viresh Kumar wrote: > Cc'ing Javi (which you should have as he wrote the power-thing for > cpu-cooling). > > On 05-11-15, 19:10, dawei chien wrote: > > This is because our platform currently only support mt8173_cpufreq.c, so > > that I only add static powe

Re: [PATCH v3 1/2] thermal: mediatek: Add cpu power cooling model.

2015-11-05 Thread Viresh Kumar
Cc'ing Javi (which you should have as he wrote the power-thing for cpu-cooling). On 05-11-15, 19:10, dawei chien wrote: > This is because our platform currently only support mt8173_cpufreq.c, so > that I only add static power model for our owner IC. Bindings are (normally) supposed to be general

Re: [PATCH v3 0/2] thermal: mediatek: Add cpu power cooling model

2015-11-05 Thread Viresh Kumar
On 05-11-15, 19:09, dawei chien wrote: > Thank you for your kindly explaining, now I could understand what I > miss, I will send device tree binding on next version such like > following description. > > --- a/Documentation/devicetree/bindings/clock/mt8173-cpu-dvfs.txt > +++ b/Documentation/device

Re: [PATCH v3 1/2] thermal: mediatek: Add cpu power cooling model.

2015-11-05 Thread dawei chien
On Wed, 2015-11-04 at 11:40 -0800, Eduardo Valentin wrote: > On Thu, Oct 22, 2015 at 08:02:38PM +0800, Dawei Chien wrote: > > This power model is base on Intelligent Power Allocation (IPA) > > technical, requires that the operating-points of the CPUs are > > registered using the kernel's opp librar

Re: [PATCH v3 0/2] thermal: mediatek: Add cpu power cooling model

2015-11-05 Thread dawei chien
On Mon, 2015-11-02 at 17:40 +0530, Viresh Kumar wrote: > On 02-11-15, 18:46, dawei chien wrote: > > On Wed, 2015-10-28 at 21:14 +0530, Viresh Kumar wrote: > > > Sorry for being extremely late in reviewing this stuff. You are > > > already on v3 and I haven't reviewed it once. Mostly due to bad timi

Re: [PATCH v3 1/2] thermal: mediatek: Add cpu power cooling model.

2015-11-04 Thread Eduardo Valentin
On Thu, Oct 22, 2015 at 08:02:38PM +0800, Dawei Chien wrote: > This power model is base on Intelligent Power Allocation (IPA) > technical, requires that the operating-points of the CPUs are > registered using the kernel's opp library and the > `cpufreq_frequency_table` is assigned to the `struct de

Re: [PATCH v3 0/2] thermal: mediatek: Add cpu power cooling model

2015-11-04 Thread Eduardo Valentin
9891/ > >> > >> Change since V1: > >> include mt8171.h and sort header file for mt8173.dtsi > >> > >> Change since V2: > >> Move dynamic/static power model in device tree > >> > >> Dawei.Chien (2): > >> thermal: mediatek:

Re: [PATCH v3 0/2] thermal: mediatek: Add cpu power cooling model

2015-11-02 Thread Viresh Kumar
On 02-11-15, 15:53, Punit Agrawal wrote: > For dynamic power, I had posted some patches[0][1][2] introducing the > binding as well as updating cooling device registration via cpufreq > driver. Now that the SCPI hwmon driver is merged, I should re-send the > remaining patches. > > [0] http://lkml.i

Re: [PATCH v3 0/2] thermal: mediatek: Add cpu power cooling model

2015-11-02 Thread Punit Agrawal
k.kernel.org/patch/7249821/ >> https://patchwork.kernel.org/patch/7249861/ >> https://patchwork.kernel.org/patch/7249891/ >> >> Change since V1: >> include mt8171.h and sort header file for mt8173.dtsi >> >> Change since V2: >> Move dynamic/static power mo

Re: [PATCH v3 0/2] thermal: mediatek: Add cpu power cooling model

2015-11-02 Thread Viresh Kumar
On 02-11-15, 18:46, dawei chien wrote: > On Wed, 2015-10-28 at 21:14 +0530, Viresh Kumar wrote: > > Sorry for being extremely late in reviewing this stuff. You are > > already on v3 and I haven't reviewed it once. Mostly due to bad timing > > of my holidays and other work pressure. > > You're welc

Re: [PATCH v3 0/2] thermal: mediatek: Add cpu power cooling model

2015-11-02 Thread dawei chien
7;s thermal driver V9 > > https://patchwork.kernel.org/patch/7249821/ > > https://patchwork.kernel.org/patch/7249861/ > > https://patchwork.kernel.org/patch/7249891/ > > > > Change since V1: > > include mt8171.h and sort header file for mt8173.dtsi > >

Re: [PATCH v3 0/2] thermal: mediatek: Add cpu power cooling model

2015-10-28 Thread Viresh Kumar
.org/patch/7249891/ > > Change since V1: > include mt8171.h and sort header file for mt8173.dtsi > > Change since V2: > Move dynamic/static power model in device tree > > Dawei.Chien (2): > thermal: mediatek: Add cpu power cooling model. > arm64: dts: mt8173:

[PATCH v3 1/2] thermal: mediatek: Add cpu power cooling model.

2015-10-22 Thread Dawei Chien
This power model is base on Intelligent Power Allocation (IPA) technical, requires that the operating-points of the CPUs are registered using the kernel's opp library and the `cpufreq_frequency_table` is assigned to the `struct device` of the cpu MT8173. Signed-off-by: Dawei.Chien --- This patch

[PATCH v3 0/2] thermal: mediatek: Add cpu power cooling model

2015-10-22 Thread Dawei Chien
power model in device tree Dawei.Chien (2): thermal: mediatek: Add cpu power cooling model. arm64: dts: mt8173: Add thermal zone node for mt8173. arch/arm64/boot/dts/mediatek/mt8173.dtsi | 44 ++ drivers/cpufreq/mt8173-cpufreq.c | 97 ++

Re: [PATCH v2 1/2] thermal: mediatek: Add cpu power cooling model.

2015-10-15 Thread dawei chien
On Mon, 2015-10-12 at 18:26 +0100, Punit Agrawal wrote: > Mark Rutland writes: > > > On Wed, Oct 07, 2015 at 08:22:40PM +0800, Dawei Chien wrote: > >> From: "Dawei.Chien" > >> > >> This power model is base on Intelligent Power Allocation (IPA) technical, > >> requires that the operating-points

Re: [PATCH v2 1/2] thermal: mediatek: Add cpu power cooling model.

2015-10-12 Thread Punit Agrawal
Mark Rutland writes: > On Wed, Oct 07, 2015 at 08:22:40PM +0800, Dawei Chien wrote: >> From: "Dawei.Chien" >> >> This power model is base on Intelligent Power Allocation (IPA) technical, >> requires that the operating-points of the CPUs are registered using the >> kernel's opp library and the `

Re: [PATCH v2 1/2] thermal: mediatek: Add cpu power cooling model.

2015-10-07 Thread Mark Rutland
On Wed, Oct 07, 2015 at 08:22:40PM +0800, Dawei Chien wrote: > From: "Dawei.Chien" > > This power model is base on Intelligent Power Allocation (IPA) technical, > requires that the operating-points of the CPUs are registered using the > kernel's opp library and the `cpufreq_frequency_table` is as

[PATCH v2 1/2] thermal: mediatek: Add cpu power cooling model.

2015-10-07 Thread Dawei Chien
From: "Dawei.Chien" This power model is base on Intelligent Power Allocation (IPA) technical, requires that the operating-points of the CPUs are registered using the kernel's opp library and the `cpufreq_frequency_table` is assigned to the `struct device` of the cpu MT8173. Signed-off-by: Dawei.

[PATCH v2 0/2] thermal: mediatek: Add cpu power cooling model

2015-10-07 Thread Dawei Chien
ch/7034601/ PATCH2 is base on Sascha's thermal driver V9 https://patchwork.kernel.org/patch/7249821/ https://patchwork.kernel.org/patch/7249861/ https://patchwork.kernel.org/patch/7249891/ Change since V1: include mt8171.h and sort header file for mt8173.dtsi Dawei.Chien (2): thermal: mediatek

Re: thermal: mediatek: Add cpu power cooling model

2015-09-08 Thread dawei chien
Hi Viresh, On Mon, 2015-09-07 at 11:39 +0530, Viresh Kumar wrote: > On 04-09-15, 17:01, Dawei Chien wrote: > > Use Intelligent Power Allocation (IPA) technical to add > > static/dynamic power model for binding CPU thermal zone. > > The power allocator governor allocates power budget to control > >

Re: thermal: mediatek: Add cpu power cooling model

2015-09-06 Thread Viresh Kumar
On 04-09-15, 17:01, Dawei Chien wrote: > Use Intelligent Power Allocation (IPA) technical to add > static/dynamic power model for binding CPU thermal zone. > The power allocator governor allocates power budget to control > CPU temperature. Sorry but this isn't enough really.. I don't have time to

Re: thermal: mediatek: Add cpu power cooling model

2015-09-04 Thread dawei chien
rol > CPU temperature. > > Dawei.Chien (2): > thermal: mediatek: Add cpu power cooling model. > arm64: dts: mt8173: Add thermal zone node for mt8173. > > arch/arm64/boot/dts/mediatek/mt8173.dtsi | 44 ++ > drivers/cpufreq/mt8173-cpufreq.c | 97 >

[PATCH 1/2] thermal: mediatek: Add cpu power cooling model

2015-09-04 Thread Dawei Chien
Use Intelligent Power Allocation (IPA) technical to add static/dynamic power model for binding CPU thermal zone. The power allocator governor allocates power budget to control CPU temperature. Signed-off-by: Dawei Chien --- This patch is base on https://patchwork.kernel.org/patch/7034601/ --- dr

thermal: mediatek: Add cpu power cooling model

2015-09-04 Thread Dawei Chien
Use Intelligent Power Allocation (IPA) technical to add static/dynamic power model for binding CPU thermal zone. The power allocator governor allocates power budget to control CPU temperature. Dawei.Chien (2): thermal: mediatek: Add cpu power cooling model. arm64: dts: mt8173: Add thermal