diff mbox

[V3,1/2] powerpc/mpc85xx: Update TMU device tree node for T1040/T1042

Message ID 1477361742-589-1-git-send-email-hongtao.jia@nxp.com (mailing list archive)
State Not Applicable
Delegated to: Scott Wood
Headers show

Commit Message

Hongtao Jia Oct. 25, 2016, 2:15 a.m. UTC 
From: Hongtao Jia <hongtao.jia@nxp.com>

Update #thermal-sensor-cells from 0 to 1 according to the new binding. The
sensor specifier added is the monitoring site ID, and represents the "n" in
TRITSRn and TRATSRn.

Signed-off-by: Jia Hongtao <hongtao.jia@nxp.com>
---
Changes for V3:
* Update the commit log to a better description.

Changes for V2:
* Rebase on latest linux-next tree (next-20161006).

 arch/powerpc/boot/dts/fsl/t1040si-post.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Crystal Wood Nov. 23, 2016, 7:07 a.m. UTC | #1 
On Tue, 2016-10-25 at 10:15 +0800, Jia Hongtao wrote:
> From: Hongtao Jia <hongtao.jia@nxp.com>
> 
> Update #thermal-sensor-cells from 0 to 1 according to the new binding. The
> sensor specifier added is the monitoring site ID, and represents the "n" in
> TRITSRn and TRATSRn.
> 
> Signed-off-by: Jia Hongtao <hongtao.jia@nxp.com>

Where can I find this new binding?  As of the current linux-next I don't see
anything in qoriq-thermal.txt about this.

-Scott
Hongtao Jia Nov. 23, 2016, 7:14 a.m. UTC | #2 
> -----Original Message-----

> From: Scott Wood [mailto:oss@buserror.net]

> Sent: Wednesday, November 23, 2016 3:07 PM

> To: Troy Jia <hongtao.jia@nxp.com>; rui.zhang@intel.com; edubezval@gmail.com;

> robh+dt@kernel.org; Scott Wood <scott.wood@nxp.com>; shawnguo@kernel.org

> Cc: devicetree@vger.kernel.org; linuxppc-dev@lists.ozlabs.org; linux-

> kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org

> Subject: Re: [PATCH V3 1/2] powerpc/mpc85xx: Update TMU device tree node for

> T1040/T1042

> 

> On Tue, 2016-10-25 at 10:15 +0800, Jia Hongtao wrote:

> > From: Hongtao Jia <hongtao.jia@nxp.com>

> >

> > Update #thermal-sensor-cells from 0 to 1 according to the new binding.

> > The sensor specifier added is the monitoring site ID, and represents

> > the "n" in TRITSRn and TRATSRn.

> >

> > Signed-off-by: Jia Hongtao <hongtao.jia@nxp.com>

> 

> Where can I find this new binding?  As of the current linux-next I don't see anything

> in qoriq-thermal.txt about this.


Hi Rui Zhang,

As we discussed before. The time was inappropriate as merge window was about to close.
So do you have any plan for applying the binding file recently?

-Hongtao.
diff mbox

Patch

diff --git a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi
index 44e399b..145c7f4 100644
--- a/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi
+++ b/arch/powerpc/boot/dts/fsl/t1040si-post.dtsi
@@ -526,7 +526,7 @@ 
 
 				       0x00030000 0x00000012
 				       0x00030001 0x0000001d>;
-		#thermal-sensor-cells = <0>;
+		#thermal-sensor-cells = <1>;
 	};
 
 	thermal-zones {
@@ -534,7 +534,7 @@ 
 			polling-delay-passive = <1000>;
 			polling-delay = <5000>;
 
-			thermal-sensors = <&tmu>;
+			thermal-sensors = <&tmu 2>;
 
 			trips {
 				cpu_alert: cpu-alert {