Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-08-29 Thread Amit Kucheria
On Wed, Aug 29, 2018 at 3:09 PM Daniel Lezcano wrote: > > On 18/07/2018 09:49, Amit Kucheria wrote: > > One thermal zone per cpu is defined > > The thermal zones are very close, especially when the CPUs belong to the > same 'cluster'. Very likely the temperature will propagate from one core > to

Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-08-29 Thread Amit Kucheria
On Wed, Aug 29, 2018 at 3:09 PM Daniel Lezcano wrote: > > On 18/07/2018 09:49, Amit Kucheria wrote: > > One thermal zone per cpu is defined > > The thermal zones are very close, especially when the CPUs belong to the > same 'cluster'. Very likely the temperature will propagate from one core > to

Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-08-29 Thread Matthias Kaehlcke
On Wed, Aug 29, 2018 at 11:39:12AM +0200, Daniel Lezcano wrote: > On 18/07/2018 09:49, Amit Kucheria wrote: > > One thermal zone per cpu is defined > > The thermal zones are very close, especially when the CPUs belong to the > same 'cluster'. Very likely the temperature will propagate from one

Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-08-29 Thread Matthias Kaehlcke
On Wed, Aug 29, 2018 at 11:39:12AM +0200, Daniel Lezcano wrote: > On 18/07/2018 09:49, Amit Kucheria wrote: > > One thermal zone per cpu is defined > > The thermal zones are very close, especially when the CPUs belong to the > same 'cluster'. Very likely the temperature will propagate from one

Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-08-29 Thread Daniel Lezcano
On 18/07/2018 09:49, Amit Kucheria wrote: > One thermal zone per cpu is defined The thermal zones are very close, especially when the CPUs belong to the same 'cluster'. Very likely the temperature will propagate from one core to another core, so when one core reaches the trip0, there is good

Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-08-29 Thread Daniel Lezcano
On 18/07/2018 09:49, Amit Kucheria wrote: > One thermal zone per cpu is defined The thermal zones are very close, especially when the CPUs belong to the same 'cluster'. Very likely the temperature will propagate from one core to another core, so when one core reaches the trip0, there is good

Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-08-28 Thread Amit Kucheria
Andy, Do you have any feedback for this patch? Regards, Amit On Wed, Jul 18, 2018 at 1:19 PM, Amit Kucheria wrote: > One thermal zone per cpu is defined > > Signed-off-by: Amit Kucheria > --- > arch/arm64/boot/dts/qcom/sdm845.dtsi | 170 > +++ > 1 file

Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-08-28 Thread Amit Kucheria
Andy, Do you have any feedback for this patch? Regards, Amit On Wed, Jul 18, 2018 at 1:19 PM, Amit Kucheria wrote: > One thermal zone per cpu is defined > > Signed-off-by: Amit Kucheria > --- > arch/arm64/boot/dts/qcom/sdm845.dtsi | 170 > +++ > 1 file

Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-07-26 Thread Amit Kucheria
On Thu, Jul 26, 2018 at 6:17 AM, Matthias Kaehlcke wrote: > Hi Amit, > > On Wed, Jul 18, 2018 at 01:19:17PM +0530, Amit Kucheria wrote: >> One thermal zone per cpu is defined >> >> Signed-off-by: Amit Kucheria >> --- >> arch/arm64/boot/dts/qcom/sdm845.dtsi | 170 >>

Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-07-26 Thread Amit Kucheria
On Thu, Jul 26, 2018 at 6:17 AM, Matthias Kaehlcke wrote: > Hi Amit, > > On Wed, Jul 18, 2018 at 01:19:17PM +0530, Amit Kucheria wrote: >> One thermal zone per cpu is defined >> >> Signed-off-by: Amit Kucheria >> --- >> arch/arm64/boot/dts/qcom/sdm845.dtsi | 170 >>

Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-07-25 Thread Matthias Kaehlcke
Hi Amit, On Wed, Jul 18, 2018 at 01:19:17PM +0530, Amit Kucheria wrote: > One thermal zone per cpu is defined > > Signed-off-by: Amit Kucheria > --- > arch/arm64/boot/dts/qcom/sdm845.dtsi | 170 > +++ > 1 file changed, 170 insertions(+) > > diff --git

Re: [PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-07-25 Thread Matthias Kaehlcke
Hi Amit, On Wed, Jul 18, 2018 at 01:19:17PM +0530, Amit Kucheria wrote: > One thermal zone per cpu is defined > > Signed-off-by: Amit Kucheria > --- > arch/arm64/boot/dts/qcom/sdm845.dtsi | 170 > +++ > 1 file changed, 170 insertions(+) > > diff --git

[PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-07-18 Thread Amit Kucheria
One thermal zone per cpu is defined Signed-off-by: Amit Kucheria --- arch/arm64/boot/dts/qcom/sdm845.dtsi | 170 +++ 1 file changed, 170 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi index

[PATCH v1] arm64: dts: sdm845: enable tsens thermal zones

2018-07-18 Thread Amit Kucheria
One thermal zone per cpu is defined Signed-off-by: Amit Kucheria --- arch/arm64/boot/dts/qcom/sdm845.dtsi | 170 +++ 1 file changed, 170 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi index