Thread (24 messages) 24 messages, 8 authors, 2014-10-06

[PATCH v5 3/4] ARM: tegra: Add thermal trip points for Jetson TK1

From: edubezval@gmail.com (Eduardo Valentin)
Date: 2014-09-24 18:48:46
Also in: linux-pm, linux-tegra, lkml

Hello Mikko,

On Wed, Sep 24, 2014 at 09:43:55PM +0300, Mikko Perttunen wrote:
On 09/24/2014 09:41 PM, Eduardo Valentin wrote:
quoted
On Wed, Sep 24, 2014 at 09:34:16PM +0300, Mikko Perttunen wrote:
quoted
On 09/24/2014 09:32 PM, Eduardo Valentin wrote:
quoted
Hello Mikko,

On Thu, Aug 21, 2014 at 01:17:21PM +0300, Mikko Perttunen wrote:
quoted
This adds critical trip points to the Jetson TK1 device tree.
The device will do a controlled shutdown when either the CPU, GPU
or MEM thermal zone reaches 101 degrees Celsius.

Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com>
---
v5: added cooling-maps nodes
OK. But no cooling map entry in it. What was the reason again you don't
need/want to add the maps?
There is currently no cooling device to map to.
Not even cpufreq cooling? Is CPUfreq operational in Tegra?
Indeed, it is not currently operational (except on Tegra20). There is a 
series for Tegra124 but it needs more work. Not going in during this cycle.
OK. Now I understand. In this case, can you please explain the situation
with a TODO comment inside the cooling-maps?

The way it is now, looks awkwards.
quoted
quoted
Mikko
quoted
quoted
   arch/arm/boot/dts/tegra124-jetson-tk1.dts | 41 +++++++++++++++++++++++++++++++
   1 file changed, 41 insertions(+)
diff --git a/arch/arm/boot/dts/tegra124-jetson-tk1.dts b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
index 0cdb5cf..bcdab7f 100644
--- a/arch/arm/boot/dts/tegra124-jetson-tk1.dts
+++ b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
@@ -1876,4 +1876,45 @@
   			 <&tegra_car TEGRA124_CLK_EXTERN1>;
   		clock-names = "pll_a", "pll_a_out0", "mclk";
   	};
+
+	thermal-zones {
+		cpu {
+			trips {
+				trip at 0 {
+					temperature = <101000>;
+					hysteresis = <0>;
+					type = "critical";
+				};
+			};
+
+			cooling-maps {
+			};
+		};
+
+		mem {
+			trips {
+				trip at 0 {
+					temperature = <101000>;
+					hysteresis = <0>;
+					type = "critical";
+				};
+			};
+
+			cooling-maps {
+			};
+		};
+
+		gpu {
+			trips {
+				trip at 0 {
+					temperature = <101000>;
+					hysteresis = <0>;
+					type = "critical";
+				};
+			};
+
+			cooling-maps {
+			};
+		};
+	};
   };
--
1.8.1.5
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help