devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Amit Kucheria <amit.kucheria@linaro.org>
To: Yuantian Tang <andy.tang@nxp.com>
Cc: Shawn Guo <shawnguo@kernel.org>, Li Yang <leoyang.li@nxp.com>,
	Rob Herring <robh+dt@kernel.org>,
	lakml <linux-arm-kernel@lists.infradead.org>,
	"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
	<devicetree@vger.kernel.org>, LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/2 v2] arm64: dts: ls208xa: add more thermal zone support
Date: Fri, 21 Aug 2020 11:39:58 +0530	[thread overview]
Message-ID: <CAHLCerMerO3-NR8N6NHnRuJ51xyJAthnpE8NTYr55-575YVrgw@mail.gmail.com> (raw)
In-Reply-To: <20200715064909.9161-2-andy.tang@nxp.com>

On Wed, Jul 15, 2020 at 12:25 PM <andy.tang@nxp.com> wrote:
>
> From: Yuantian Tang <andy.tang@nxp.com>
>
> There are 7 thermal zones in ls208xa soc. Add the other thermal zone
> nodes to enable them.
>
> Signed-off-by: Yuantian Tang <andy.tang@nxp.com>

Reviewed-by: Amit Kucheria <amitk@kernel.org>

> ---
> v2:
>         - remove useless alert trip
>         - add cooling-map to core cluster zones.
>
>  .../arm64/boot/dts/freescale/fsl-ls208xa.dtsi | 141 ++++++++++++++++--
>  1 file changed, 132 insertions(+), 9 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> index 41102dacc2e1..cc36c969dd9d 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> @@ -79,20 +79,62 @@
>         };
>
>         thermal-zones {
> -               cpu_thermal: cpu-thermal {
> +               ddr-controller1 {
>                         polling-delay-passive = <1000>;
>                         polling-delay = <5000>;
> +                       thermal-sensors = <&tmu 1>;
>
> +                       trips {
> +                               ddr-ctrler1-crit {
> +                                       temperature = <95000>;
> +                                       hysteresis = <2000>;
> +                                       type = "critical";
> +                               };
> +                       };
> +               };
> +
> +               ddr-controller2 {
> +                       polling-delay-passive = <1000>;
> +                       polling-delay = <5000>;
> +                       thermal-sensors = <&tmu 2>;
> +
> +                       trips {
> +                               ddr-ctrler2-crit {
> +                                       temperature = <95000>;
> +                                       hysteresis = <2000>;
> +                                       type = "critical";
> +                               };
> +                       };
> +               };
> +
> +               ddr-controller3 {
> +                       polling-delay-passive = <1000>;
> +                       polling-delay = <5000>;
> +                       thermal-sensors = <&tmu 3>;
> +
> +                       trips {
> +                               ddr-ctrler3-crit {
> +                                       temperature = <95000>;
> +                                       hysteresis = <2000>;
> +                                       type = "critical";
> +                               };
> +                       };
> +               };
> +
> +               core-cluster1 {
> +                       polling-delay-passive = <1000>;
> +                       polling-delay = <5000>;
>                         thermal-sensors = <&tmu 4>;
>
>                         trips {
> -                               cpu_alert: cpu-alert {
> -                                       temperature = <75000>;
> +                               core_cluster1_alert: core-cluster1-alert {
> +                                       temperature = <85000>;
>                                         hysteresis = <2000>;
>                                         type = "passive";
>                                 };
> -                               cpu_crit: cpu-crit {
> -                                       temperature = <85000>;
> +
> +                               core-cluster1-crit {
> +                                       temperature = <95000>;
>                                         hysteresis = <2000>;
>                                         type = "critical";
>                                 };
> @@ -100,14 +142,95 @@
>
>                         cooling-maps {
>                                 map0 {
> -                                       trip = <&cpu_alert>;
> +                                       trip = <&core_cluster1_alert>;
>                                         cooling-device =
>                                                 <&cpu0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> -                                               <&cpu1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> +                                               <&cpu1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> +                               };
> +                       };
> +               };
> +
> +               core-cluster2 {
> +                       polling-delay-passive = <1000>;
> +                       polling-delay = <5000>;
> +                       thermal-sensors = <&tmu 5>;
> +
> +                       trips {
> +                               core_cluster2_alert: core-cluster2-alert {
> +                                       temperature = <85000>;
> +                                       hysteresis = <2000>;
> +                                       type = "passive";
> +                               };
> +
> +                               core-cluster2-crit {
> +                                       temperature = <95000>;
> +                                       hysteresis = <2000>;
> +                                       type = "critical";
> +                               };
> +                       };
> +
> +                       cooling-maps {
> +                               map0 {
> +                                       trip = <&core_cluster2_alert>;
> +                                       cooling-device =
>                                                 <&cpu2 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> -                                               <&cpu3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> +                                               <&cpu3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> +                               };
> +                       };
> +               };
> +
> +               core-cluster3 {
> +                       polling-delay-passive = <1000>;
> +                       polling-delay = <5000>;
> +                       thermal-sensors = <&tmu 6>;
> +
> +                       trips {
> +                               core_cluster3_alert: core-cluster3-alert {
> +                                       temperature = <85000>;
> +                                       hysteresis = <2000>;
> +                                       type = "passive";
> +                               };
> +
> +                               core-cluster3-crit {
> +                                       temperature = <95000>;
> +                                       hysteresis = <2000>;
> +                                       type = "critical";
> +                               };
> +                       };
> +
> +                       cooling-maps {
> +                               map0 {
> +                                       trip = <&core_cluster3_alert>;
> +                                       cooling-device =
>                                                 <&cpu4 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> -                                               <&cpu5 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> +                                               <&cpu5 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> +                               };
> +                       };
> +               };
> +
> +               core-cluster4 {
> +                       polling-delay-passive = <1000>;
> +                       polling-delay = <5000>;
> +                       thermal-sensors = <&tmu 7>;
> +
> +                       trips {
> +                               core_cluster4_alert: core-cluster4-alert {
> +                                       temperature = <85000>;
> +                                       hysteresis = <2000>;
> +                                       type = "passive";
> +                               };
> +
> +                               core-cluster4-crit {
> +                                       temperature = <95000>;
> +                                       hysteresis = <2000>;
> +                                       type = "critical";
> +                               };
> +                       };
> +
> +                       cooling-maps {
> +                               map0 {
> +                                       trip = <&core_cluster4_alert>;
> +                                       cooling-device =
>                                                 <&cpu6 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
>                                                 <&cpu7 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
>                                 };
> --
> 2.17.1
>

  reply	other threads:[~2020-08-21  6:10 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-15  6:49 [PATCH 1/2 v2] arm64: dts: ls1088a: add more thermal zone support andy.tang
2020-07-15  6:49 ` [PATCH 2/2 v2] arm64: dts: ls208xa: " andy.tang
2020-08-21  6:09   ` Amit Kucheria [this message]
2020-08-17  8:02 ` [PATCH 1/2 v2] arm64: dts: ls1088a: " Shawn Guo
2020-08-21  6:09 ` Amit Kucheria
2020-08-22  3:29 ` Shawn Guo

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=CAHLCerMerO3-NR8N6NHnRuJ51xyJAthnpE8NTYr55-575YVrgw@mail.gmail.com \
    --to=amit.kucheria@linaro.org \
    --cc=andy.tang@nxp.com \
    --cc=devicetree@vger.kernel.org \
    --cc=leoyang.li@nxp.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=shawnguo@kernel.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).