* [PATCH V7 08/12] of: add notes of critical trips for soctherm
@ 2016-03-11 3:11 ` Wei Ni
0 siblings, 0 replies; 9+ messages in thread
From: Wei Ni @ 2016-03-11 3:11 UTC (permalink / raw)
To: rui.zhang, edubezval, thierry.reding
Cc: MLongnecker, swarren, mikko.perttunen, linux-tegra, linux-pm,
linux-kernel, Wei Ni
The "critical" type trip in thermal zone can be
set to SOC_THERM hardware, it can trigger shut down
or reset event from hardware.
Signed-off-by: Wei Ni <wni@nvidia.com>
Acked-by: Rob Herring <robh@kernel.org>
---
Documentation/devicetree/bindings/thermal/tegra-soctherm.txt | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt b/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
index 6b68cd150405..351a7376baa8 100644
--- a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
+++ b/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
@@ -26,6 +26,10 @@ Required properties :
of this property. See <dt-bindings/thermal/tegra124-soctherm.h> for a
list of valid values when referring to thermal sensors.
+Note:
+- the "critical" type trip points will be set to SOC_THERM hardware as the
+shut down temperature. Once the temperature of this thermal zone is higher
+than it, the system will be shutdown or reset by hardware.
Example :
@@ -51,5 +55,13 @@ Example: referring to thermal sensors :
thermal-sensors =
<&soctherm TEGRA124_SOCTHERM_SENSOR_CPU>;
+
+ trips {
+ cpu_shutdown_trip: shutdown-trip {
+ temperature = <102500>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};
};
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH V7 08/12] of: add notes of critical trips for soctherm
@ 2016-03-11 3:11 ` Wei Ni
0 siblings, 0 replies; 9+ messages in thread
From: Wei Ni @ 2016-03-11 3:11 UTC (permalink / raw)
To: rui.zhang, edubezval, thierry.reding
Cc: MLongnecker, swarren, mikko.perttunen, linux-tegra, linux-pm,
linux-kernel, Wei Ni
The "critical" type trip in thermal zone can be
set to SOC_THERM hardware, it can trigger shut down
or reset event from hardware.
Signed-off-by: Wei Ni <wni@nvidia.com>
Acked-by: Rob Herring <robh@kernel.org>
---
Documentation/devicetree/bindings/thermal/tegra-soctherm.txt | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt b/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
index 6b68cd150405..351a7376baa8 100644
--- a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
+++ b/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
@@ -26,6 +26,10 @@ Required properties :
of this property. See <dt-bindings/thermal/tegra124-soctherm.h> for a
list of valid values when referring to thermal sensors.
+Note:
+- the "critical" type trip points will be set to SOC_THERM hardware as the
+shut down temperature. Once the temperature of this thermal zone is higher
+than it, the system will be shutdown or reset by hardware.
Example :
@@ -51,5 +55,13 @@ Example: referring to thermal sensors :
thermal-sensors =
<&soctherm TEGRA124_SOCTHERM_SENSOR_CPU>;
+
+ trips {
+ cpu_shutdown_trip: shutdown-trip {
+ temperature = <102500>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};
};
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH V7 08/12] of: add notes of critical trips for soctherm
2016-03-11 3:11 ` Wei Ni
(?)
@ 2016-03-14 21:00 ` Eduardo Valentin
[not found] ` <20160314210027.GA4740-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
-1 siblings, 1 reply; 9+ messages in thread
From: Eduardo Valentin @ 2016-03-14 21:00 UTC (permalink / raw)
To: Wei Ni
Cc: rui.zhang, thierry.reding, MLongnecker, swarren, mikko.perttunen,
linux-tegra, linux-pm, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 523 bytes --]
On Fri, Mar 11, 2016 at 11:11:00AM +0800, Wei Ni wrote:
> The "critical" type trip in thermal zone can be
> set to SOC_THERM hardware, it can trigger shut down
> or reset event from hardware.
>
> Signed-off-by: Wei Ni <wni@nvidia.com>
> Acked-by: Rob Herring <robh@kernel.org>
> ---
> Documentation/devicetree/bindings/thermal/tegra-soctherm.txt | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
I did not see in your patch set an update on the compatible string for
the new chip. Did I miss something?
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V7 08/12] of: add notes of critical trips for soctherm
2016-03-14 21:00 ` Eduardo Valentin
@ 2016-03-15 7:49 ` Wei Ni
0 siblings, 0 replies; 9+ messages in thread
From: Wei Ni @ 2016-03-15 7:49 UTC (permalink / raw)
To: Eduardo Valentin
Cc: rui.zhang-ral2JQCrhuEAvxtiuMwx3w,
thierry.reding-Re5JQEeQqe8AvxtiuMwx3w,
MLongnecker-DDmLM1+adcrQT0dZR+AlfA,
swarren-3lzwWm7+Weoh9ZMKESR00Q, mikko.perttunen-/1wQRMveznE,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
linux-pm-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA
On 2016年03月15日 05:00, Eduardo Valentin wrote:
> * PGP Signed by an unknown key
>
> On Fri, Mar 11, 2016 at 11:11:00AM +0800, Wei Ni wrote:
>> The "critical" type trip in thermal zone can be
>> set to SOC_THERM hardware, it can trigger shut down
>> or reset event from hardware.
>>
>> Signed-off-by: Wei Ni <wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>> Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
>> ---
>> Documentation/devicetree/bindings/thermal/tegra-soctherm.txt | 12 ++++++++++++
>> 1 file changed, 12 insertions(+)
>
> I did not see in your patch set an update on the compatible string for
> the new chip. Did I miss something?
As I said in the previous [00/12], the "commit 193c9d23a0f0" already added the
compatible string. At that time, it just used current tegra_soctherm.c driver to
support Tegra210, it can work, but can't show temperatures correctly.
>
> * Unknown Key
> * 0x7DA4E256
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V7 08/12] of: add notes of critical trips for soctherm
@ 2016-03-15 7:49 ` Wei Ni
0 siblings, 0 replies; 9+ messages in thread
From: Wei Ni @ 2016-03-15 7:49 UTC (permalink / raw)
To: Eduardo Valentin
Cc: rui.zhang, thierry.reding, MLongnecker, swarren, mikko.perttunen,
linux-tegra, linux-pm, linux-kernel
On 2016年03月15日 05:00, Eduardo Valentin wrote:
> * PGP Signed by an unknown key
>
> On Fri, Mar 11, 2016 at 11:11:00AM +0800, Wei Ni wrote:
>> The "critical" type trip in thermal zone can be
>> set to SOC_THERM hardware, it can trigger shut down
>> or reset event from hardware.
>>
>> Signed-off-by: Wei Ni <wni@nvidia.com>
>> Acked-by: Rob Herring <robh@kernel.org>
>> ---
>> Documentation/devicetree/bindings/thermal/tegra-soctherm.txt | 12 ++++++++++++
>> 1 file changed, 12 insertions(+)
>
> I did not see in your patch set an update on the compatible string for
> the new chip. Did I miss something?
As I said in the previous [00/12], the "commit 193c9d23a0f0" already added the
compatible string. At that time, it just used current tegra_soctherm.c driver to
support Tegra210, it can work, but can't show temperatures correctly.
>
> * Unknown Key
> * 0x7DA4E256
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V7 08/12] of: add notes of critical trips for soctherm
2016-03-15 7:49 ` Wei Ni
@ 2016-03-15 8:14 ` Wei Ni
-1 siblings, 0 replies; 9+ messages in thread
From: Wei Ni @ 2016-03-15 8:14 UTC (permalink / raw)
To: Eduardo Valentin
Cc: rui.zhang, thierry.reding, MLongnecker, swarren, mikko.perttunen,
linux-tegra, linux-pm, linux-kernel
On 2016年03月15日 15:49, Wei Ni wrote:
>
>
> On 2016年03月15日 05:00, Eduardo Valentin wrote:
>> * PGP Signed by an unknown key
>>
>> On Fri, Mar 11, 2016 at 11:11:00AM +0800, Wei Ni wrote:
>>> The "critical" type trip in thermal zone can be
>>> set to SOC_THERM hardware, it can trigger shut down
>>> or reset event from hardware.
>>>
>>> Signed-off-by: Wei Ni <wni@nvidia.com>
>>> Acked-by: Rob Herring <robh@kernel.org>
>>> ---
>>> Documentation/devicetree/bindings/thermal/tegra-soctherm.txt | 12 ++++++++++++
>>> 1 file changed, 12 insertions(+)
>>
>> I did not see in your patch set an update on the compatible string for
>> the new chip. Did I miss something?
>
> As I said in the previous [00/12], the "commit 193c9d23a0f0" already added the
> compatible string. At that time, it just used current tegra_soctherm.c driver to
> support Tegra210, it can work, but can't show temperatures correctly.
Oh, sorry, I made a mistake, this "commit 193c9d23a0f0" just added the
compatible string, did not use current tegra_soctherm driver to support Tegra210.
Wei.
>
>>
>> * Unknown Key
>> * 0x7DA4E256
>>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V7 08/12] of: add notes of critical trips for soctherm
@ 2016-03-15 8:14 ` Wei Ni
0 siblings, 0 replies; 9+ messages in thread
From: Wei Ni @ 2016-03-15 8:14 UTC (permalink / raw)
To: Eduardo Valentin
Cc: rui.zhang, thierry.reding, MLongnecker, swarren, mikko.perttunen,
linux-tegra, linux-pm, linux-kernel
On 2016年03月15日 15:49, Wei Ni wrote:
>
>
> On 2016年03月15日 05:00, Eduardo Valentin wrote:
>> * PGP Signed by an unknown key
>>
>> On Fri, Mar 11, 2016 at 11:11:00AM +0800, Wei Ni wrote:
>>> The "critical" type trip in thermal zone can be
>>> set to SOC_THERM hardware, it can trigger shut down
>>> or reset event from hardware.
>>>
>>> Signed-off-by: Wei Ni <wni@nvidia.com>
>>> Acked-by: Rob Herring <robh@kernel.org>
>>> ---
>>> Documentation/devicetree/bindings/thermal/tegra-soctherm.txt | 12 ++++++++++++
>>> 1 file changed, 12 insertions(+)
>>
>> I did not see in your patch set an update on the compatible string for
>> the new chip. Did I miss something?
>
> As I said in the previous [00/12], the "commit 193c9d23a0f0" already added the
> compatible string. At that time, it just used current tegra_soctherm.c driver to
> support Tegra210, it can work, but can't show temperatures correctly.
Oh, sorry, I made a mistake, this "commit 193c9d23a0f0" just added the
compatible string, did not use current tegra_soctherm driver to support Tegra210.
Wei.
>
>>
>> * Unknown Key
>> * 0x7DA4E256
>>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V7 08/12] of: add notes of critical trips for soctherm
2016-03-15 8:14 ` Wei Ni
@ 2016-03-15 19:52 ` Eduardo Valentin
-1 siblings, 0 replies; 9+ messages in thread
From: Eduardo Valentin @ 2016-03-15 19:52 UTC (permalink / raw)
To: Wei Ni
Cc: rui.zhang-ral2JQCrhuEAvxtiuMwx3w,
thierry.reding-Re5JQEeQqe8AvxtiuMwx3w,
MLongnecker-DDmLM1+adcrQT0dZR+AlfA,
swarren-3lzwWm7+Weoh9ZMKESR00Q, mikko.perttunen-/1wQRMveznE,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
linux-pm-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA
[-- Attachment #1: Type: text/plain, Size: 1470 bytes --]
On Tue, Mar 15, 2016 at 04:14:15PM +0800, Wei Ni wrote:
>
>
> On 2016年03月15日 15:49, Wei Ni wrote:
> >
> >
> > On 2016年03月15日 05:00, Eduardo Valentin wrote:
> >> * PGP Signed by an unknown key
> >>
> >> On Fri, Mar 11, 2016 at 11:11:00AM +0800, Wei Ni wrote:
> >>> The "critical" type trip in thermal zone can be
> >>> set to SOC_THERM hardware, it can trigger shut down
> >>> or reset event from hardware.
> >>>
> >>> Signed-off-by: Wei Ni <wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> >>> Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> >>> ---
> >>> Documentation/devicetree/bindings/thermal/tegra-soctherm.txt | 12 ++++++++++++
> >>> 1 file changed, 12 insertions(+)
> >>
> >> I did not see in your patch set an update on the compatible string for
> >> the new chip. Did I miss something?
> >
> > As I said in the previous [00/12], the "commit 193c9d23a0f0" already added the
> > compatible string. At that time, it just used current tegra_soctherm.c driver to
> > support Tegra210, it can work, but can't show temperatures correctly.
>
> Oh, sorry, I made a mistake, this "commit 193c9d23a0f0" just added the
> compatible string, did not use current tegra_soctherm driver to support Tegra210.
Ok. got it. The binding is done, but the driver is being re factored only
now to add the support for this chip version.
>
> Wei.
>
> >
> >>
> >> * Unknown Key
> >> * 0x7DA4E256
> >>
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V7 08/12] of: add notes of critical trips for soctherm
@ 2016-03-15 19:52 ` Eduardo Valentin
0 siblings, 0 replies; 9+ messages in thread
From: Eduardo Valentin @ 2016-03-15 19:52 UTC (permalink / raw)
To: Wei Ni
Cc: rui.zhang, thierry.reding, MLongnecker, swarren, mikko.perttunen,
linux-tegra, linux-pm, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1412 bytes --]
On Tue, Mar 15, 2016 at 04:14:15PM +0800, Wei Ni wrote:
>
>
> On 2016年03月15日 15:49, Wei Ni wrote:
> >
> >
> > On 2016年03月15日 05:00, Eduardo Valentin wrote:
> >> * PGP Signed by an unknown key
> >>
> >> On Fri, Mar 11, 2016 at 11:11:00AM +0800, Wei Ni wrote:
> >>> The "critical" type trip in thermal zone can be
> >>> set to SOC_THERM hardware, it can trigger shut down
> >>> or reset event from hardware.
> >>>
> >>> Signed-off-by: Wei Ni <wni@nvidia.com>
> >>> Acked-by: Rob Herring <robh@kernel.org>
> >>> ---
> >>> Documentation/devicetree/bindings/thermal/tegra-soctherm.txt | 12 ++++++++++++
> >>> 1 file changed, 12 insertions(+)
> >>
> >> I did not see in your patch set an update on the compatible string for
> >> the new chip. Did I miss something?
> >
> > As I said in the previous [00/12], the "commit 193c9d23a0f0" already added the
> > compatible string. At that time, it just used current tegra_soctherm.c driver to
> > support Tegra210, it can work, but can't show temperatures correctly.
>
> Oh, sorry, I made a mistake, this "commit 193c9d23a0f0" just added the
> compatible string, did not use current tegra_soctherm driver to support Tegra210.
Ok. got it. The binding is done, but the driver is being re factored only
now to add the support for this chip version.
>
> Wei.
>
> >
> >>
> >> * Unknown Key
> >> * 0x7DA4E256
> >>
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2016-03-15 19:52 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-11 3:11 [PATCH V7 08/12] of: add notes of critical trips for soctherm Wei Ni
2016-03-11 3:11 ` Wei Ni
2016-03-14 21:00 ` Eduardo Valentin
[not found] ` <20160314210027.GA4740-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2016-03-15 7:49 ` Wei Ni
2016-03-15 7:49 ` Wei Ni
2016-03-15 8:14 ` Wei Ni
2016-03-15 8:14 ` Wei Ni
[not found] ` <56E7C457.5000301-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-03-15 19:52 ` Eduardo Valentin
2016-03-15 19:52 ` Eduardo Valentin
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.