linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: mikko.perttunen@kapsi.fi (Mikko Perttunen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 3/4] ARM: tegra: Add thermal trip points for Jetson TK1
Date: Wed, 24 Sep 2014 21:43:55 +0300	[thread overview]
Message-ID: <542310EB.7050802@kapsi.fi> (raw)
In-Reply-To: <20140924184157.GB7593@developer>

On 09/24/2014 09:41 PM, Eduardo Valentin wrote:
> On Wed, Sep 24, 2014 at 09:34:16PM +0300, Mikko Perttunen wrote:
>> On 09/24/2014 09:32 PM, Eduardo Valentin wrote:
>>> Hello Mikko,
>>>
>>> On Thu, Aug 21, 2014 at 01:17:21PM +0300, Mikko Perttunen wrote:
>>>> 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.

>
>>
>> Mikko
>>
>>>
>>>>    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
>>>
>>

  reply	other threads:[~2014-09-24 18:43 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-21 10:17 [PATCH v5 0/4] Tegra124 soctherm driver Mikko Perttunen
2014-08-21 10:17 ` [PATCH v5 1/4] of: Add bindings for nvidia,tegra124-soctherm Mikko Perttunen
2014-09-24 18:40   ` Eduardo Valentin
2014-09-24 19:43     ` Stephen Warren
2014-08-21 10:17 ` [PATCH v5 2/4] ARM: tegra: Add soctherm and thermal zones to Tegra124 device tree Mikko Perttunen
2014-08-21 10:17 ` [PATCH v5 3/4] ARM: tegra: Add thermal trip points for Jetson TK1 Mikko Perttunen
2014-09-24 18:32   ` Eduardo Valentin
2014-09-24 18:34     ` Mikko Perttunen
2014-09-24 18:41       ` Eduardo Valentin
2014-09-24 18:43         ` Mikko Perttunen [this message]
2014-09-24 18:48           ` Eduardo Valentin
2014-09-24 19:01             ` Mikko Perttunen
2014-10-03 19:57     ` Pavel Machek
2014-10-06  7:24       ` Peter De Schrijver
2014-08-21 10:17 ` [PATCH v5 4/4] thermal: Add Tegra SOCTHERM thermal management driver Mikko Perttunen
2014-09-24 19:18   ` Eduardo Valentin
2014-09-24 19:32     ` Mikko Perttunen
2014-09-24 23:24       ` Eduardo Valentin
2014-09-25  5:59       ` Thierry Reding
2014-09-25  7:55         ` Mikko Perttunen
2014-08-21 12:20 ` [PATCH v5 0/4] Tegra124 soctherm driver Juha-Matti Tilli
2014-08-21 16:01   ` Eduardo Valentin
2014-08-21 18:03     ` Juha-Matti Tilli
2014-08-27 15:36     ` Mikko Perttunen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=542310EB.7050802@kapsi.fi \
    --to=mikko.perttunen@kapsi.fi \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).