From: Eduardo Valentin <edubezval@gmail.com> To: Mikko Perttunen <mikko.perttunen@kapsi.fi> Cc: Mikko Perttunen <mperttunen@nvidia.com>, rui.zhang@intel.com, swarren@wwwdotorg.org, thierry.reding@gmail.com, linux-pm@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, juha-matti.tilli@iki.fi Subject: Re: [PATCH v5 3/4] ARM: tegra: Add thermal trip points for Jetson TK1 Date: Wed, 24 Sep 2014 14:41:58 -0400 [thread overview] Message-ID: <20140924184157.GB7593@developer> (raw) In-Reply-To: <54230EA8.7080604@kapsi.fi> 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? > > 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@0 { > >> + temperature = <101000>; > >> + hysteresis = <0>; > >> + type = "critical"; > >> + }; > >> + }; > >> + > >> + cooling-maps { > >> + }; > >> + }; > >> + > >> + mem { > >> + trips { > >> + trip@0 { > >> + temperature = <101000>; > >> + hysteresis = <0>; > >> + type = "critical"; > >> + }; > >> + }; > >> + > >> + cooling-maps { > >> + }; > >> + }; > >> + > >> + gpu { > >> + trips { > >> + trip@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@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > >
WARNING: multiple messages have this Message-ID (diff)
From: edubezval@gmail.com (Eduardo Valentin) 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 14:41:58 -0400 [thread overview] Message-ID: <20140924184157.GB7593@developer> (raw) In-Reply-To: <54230EA8.7080604@kapsi.fi> 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? > > 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 > > >
next prev parent reply other threads:[~2014-09-24 18:41 UTC|newest] Thread overview: 60+ 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 ` Mikko Perttunen 2014-08-21 10:17 ` Mikko Perttunen 2014-08-21 10:17 ` [PATCH v5 1/4] of: Add bindings for nvidia,tegra124-soctherm Mikko Perttunen 2014-08-21 10:17 ` Mikko Perttunen 2014-08-21 10:17 ` Mikko Perttunen [not found] ` <1408616242-21009-2-git-send-email-mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2014-09-24 18:40 ` Eduardo Valentin 2014-09-24 18:40 ` Eduardo Valentin 2014-09-24 18:40 ` Eduardo Valentin 2014-09-24 19:43 ` Stephen Warren 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 ` Mikko Perttunen 2014-08-21 10:17 ` Mikko Perttunen 2014-08-21 10:17 ` [PATCH v5 3/4] ARM: tegra: Add thermal trip points for Jetson TK1 Mikko Perttunen 2014-08-21 10:17 ` Mikko Perttunen 2014-08-21 10:17 ` Mikko Perttunen [not found] ` <1408616242-21009-4-git-send-email-mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2014-09-24 18:32 ` Eduardo Valentin 2014-09-24 18:32 ` Eduardo Valentin 2014-09-24 18:32 ` Eduardo Valentin 2014-09-24 18:34 ` Mikko Perttunen 2014-09-24 18:34 ` Mikko Perttunen 2014-09-24 18:34 ` Mikko Perttunen 2014-09-24 18:41 ` Eduardo Valentin [this message] 2014-09-24 18:41 ` Eduardo Valentin 2014-09-24 18:43 ` Mikko Perttunen 2014-09-24 18:43 ` Mikko Perttunen 2014-09-24 18:48 ` Eduardo Valentin 2014-09-24 18:48 ` Eduardo Valentin 2014-09-24 19:01 ` Mikko Perttunen 2014-09-24 19:01 ` Mikko Perttunen 2014-10-03 19:57 ` Pavel Machek 2014-10-03 19:57 ` Pavel Machek 2014-10-06 7:24 ` Peter De Schrijver 2014-10-06 7:24 ` Peter De Schrijver 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-08-21 10:17 ` Mikko Perttunen 2014-08-21 10:17 ` Mikko Perttunen [not found] ` <1408616242-21009-5-git-send-email-mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2014-09-24 19:18 ` Eduardo Valentin 2014-09-24 19:18 ` Eduardo Valentin 2014-09-24 19:18 ` Eduardo Valentin 2014-09-24 19:32 ` Mikko Perttunen 2014-09-24 19:32 ` Mikko Perttunen 2014-09-24 23:24 ` Eduardo Valentin 2014-09-24 23:24 ` Eduardo Valentin [not found] ` <54231C3D.6040900-/1wQRMveznE@public.gmane.org> 2014-09-25 5:59 ` Thierry Reding 2014-09-25 5:59 ` Thierry Reding 2014-09-25 5:59 ` Thierry Reding 2014-09-25 7:55 ` Mikko Perttunen 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 12:20 ` Juha-Matti Tilli 2014-08-21 16:01 ` Eduardo Valentin 2014-08-21 16:01 ` Eduardo Valentin 2014-08-21 18:03 ` Juha-Matti Tilli 2014-08-21 18:03 ` Juha-Matti Tilli 2014-08-27 15:36 ` Mikko Perttunen 2014-08-27 15:36 ` Mikko Perttunen 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=20140924184157.GB7593@developer \ --to=edubezval@gmail.com \ --cc=juha-matti.tilli@iki.fi \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=linux-tegra@vger.kernel.org \ --cc=mikko.perttunen@kapsi.fi \ --cc=mperttunen@nvidia.com \ --cc=rui.zhang@intel.com \ --cc=swarren@wwwdotorg.org \ --cc=thierry.reding@gmail.com \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.