| Message ID | 20190401032905.32711-1-andy.tang@nxp.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v4] arm64: dts: ls1088a: add one more thermal zone node | expand |
On 01/04/2019 05:29, Yuantian Tang wrote: > Ls1088a has 2 thermal sensors, core cluster and SoC platform. Core cluster > sensor is used to monitor the temperature of core and SoC platform is for > platform. The current dts only support the first sensor. > This patch adds the second sensor node to dts to enable it. > > Signed-off-by: Yuantian Tang <andy.tang@nxp.com> > --- > v4: > - use hyphen instead of underscore in node name > v3: > - use more descriptive name for each zone > v2: > - Add more information about sensors to description > PS: In order to keep consistency to the first thermal-zone node, there will > be "WARNING: line over 80 characters" warnings. > > arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi | 43 +++++++++++++++++++++-- > 1 files changed, 39 insertions(+), 4 deletions(-) > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi > index 661137f..d6d4ff2 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi > @@ -129,19 +129,19 @@ > }; > > thermal-zones { > - cpu_thermal: cpu-thermal { > + core-cluster-thermal { > polling-delay-passive = <1000>; > polling-delay = <5000>; > thermal-sensors = <&tmu 0>; > > trips { > - cpu_alert: cpu-alert { > + core_cluster_alert: core-cluster-alert { > temperature = <85000>; > hysteresis = <2000>; > type = "passive"; > }; > > - cpu_crit: cpu-crit { > + core_cluster_crit: core-cluster-crit { > temperature = <95000>; > hysteresis = <2000>; > type = "critical"; > @@ -150,7 +150,42 @@ > > cooling-maps { > map0 { > - trip = <&cpu_alert>; > + trip = <&core_cluster_alert>; > + cooling-device = > + <&cpu0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, > + <&cpu1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, > + <&cpu2 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, > + <&cpu3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, > + <&cpu4 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, > + <&cpu5 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, > + <&cpu6 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, > + <&cpu7 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>; > + }; Does it make sense to assign the same cooling devices to two different thermal zones running different instances of governor ? > + }; > + }; > + > + platform-thermal { > + polling-delay-passive = <1000>; > + polling-delay = <5000>; > + thermal-sensors = <&tmu 1>; > + > + trips { > + platform_alert: platform-alert { > + temperature = <85000>; > + hysteresis = <2000>; > + type = "passive"; > + }; > + > + platform_crit: platform-crit { > + temperature = <95000>; > + hysteresis = <2000>; > + type = "critical"; > + }; > + }; > + > + cooling-maps { > + map0 { > + trip = <&platform_alert>; > cooling-device = > <&cpu0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, > <&cpu1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, >
Hi Daniel, > -----Original Message----- > From: Daniel Lezcano <daniel.lezcano@linaro.org> > Sent: 2019年4月4日 10:35 > To: Andy Tang <andy.tang@nxp.com>; shawnguo@kernel.org > Cc: Leo Li <leoyang.li@nxp.com>; robh+dt@kernel.org; mark.rutland@arm.com; > linux-arm-kernel@lists.infradead.org; devicetree@vger.kernel.org; > linux-kernel@vger.kernel.org; linux-pm@vger.kernel.org; rui.zhang@intel.com; > edubezval@gmail.com > Subject: Re: [PATCH v4] arm64: dts: ls1088a: add one more thermal zone node > > + <&cpu1 THERMAL_NO_LIMIT > THERMAL_NO_LIMIT>, > > + <&cpu2 THERMAL_NO_LIMIT > THERMAL_NO_LIMIT>, > > + <&cpu3 THERMAL_NO_LIMIT > THERMAL_NO_LIMIT>, > > + <&cpu4 THERMAL_NO_LIMIT > THERMAL_NO_LIMIT>, > > + <&cpu5 THERMAL_NO_LIMIT > THERMAL_NO_LIMIT>, > > + <&cpu6 THERMAL_NO_LIMIT > THERMAL_NO_LIMIT>, > > + <&cpu7 THERMAL_NO_LIMIT > THERMAL_NO_LIMIT>; > > + }; > > Does it make sense to assign the same cooling devices to two different thermal > zones running different instances of governor ? Even though I didn't find any error when test, I do think it is inappropriate. BR, Andy > > > + }; > > + }; > > + > > + platform-thermal { > > + polling-delay-passive = <1000>; > > + polling-delay = <5000>; > > + thermal-sensors = <&tmu 1>; > > + > > + trips { > > + platform_alert: platform-alert { > > + temperature = <85000>; > > + hysteresis = <2000>; > > + type = "passive"; > > + }; > > + > > + platform_crit: platform-crit { > > + temperature = <95000>; > > + hysteresis = <2000>; > > + type = "critical"; > > + }; > > + }; > > + > > + cooling-maps { > > + map0 { > > + trip = <&platform_alert>; > > cooling-device = > > <&cpu0 THERMAL_NO_LIMIT > THERMAL_NO_LIMIT>, > > <&cpu1 THERMAL_NO_LIMIT > THERMAL_NO_LIMIT>, > > > > > -- > > <https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.lin > aro.org%2F&data=02%7C01%7Candy.tang%40nxp.com%7Ce1b704832ea34 > 6ae738408d6b8a6283f%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C > 636899421110441574&sdata=C7mrUztObkHMZyqEJg53h99SdIweJSeKhi8l > %2BQpUQhA%3D&reserved=0> Linaro.org │ Open source software for > ARM SoCs > > Follow Linaro: > <https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.fac > ebook.com%2Fpages%2FLinaro&data=02%7C01%7Candy.tang%40nxp.com > %7Ce1b704832ea346ae738408d6b8a6283f%7C686ea1d3bc2b4c6fa92cd99c5c30 > 1635%7C0%7C0%7C636899421110441574&sdata=Pxj%2Blfc%2FC1KiQBD0 > kzTjQGT2AgjCbYlcXhnscWTb6pw%3D&reserved=0> Facebook | > <https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Ftwitter.c > om%2F%23!%2Flinaroorg&data=02%7C01%7Candy.tang%40nxp.com%7Ce > 1b704832ea346ae738408d6b8a6283f%7C686ea1d3bc2b4c6fa92cd99c5c301635 > %7C0%7C0%7C636899421110441574&sdata=YJQRgMTwcG%2B3H1g%2FX > h3zaw%2BXy8OdlstL5AH4R0eXmN8%3D&reserved=0> Twitter | > <https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.lin > aro.org%2Flinaro-blog%2F&data=02%7C01%7Candy.tang%40nxp.com%7C > e1b704832ea346ae738408d6b8a6283f%7C686ea1d3bc2b4c6fa92cd99c5c30163 > 5%7C0%7C0%7C636899421110441574&sdata=rlkO6mkHgdam8K2RTwsOf > 8ExPF2KRy%2FrMwwyshRBKAg%3D&reserved=0> Blog
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi index 661137f..d6d4ff2 100644 --- a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi +++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi @@ -129,19 +129,19 @@ }; thermal-zones { - cpu_thermal: cpu-thermal { + core-cluster-thermal { polling-delay-passive = <1000>; polling-delay = <5000>; thermal-sensors = <&tmu 0>; trips { - cpu_alert: cpu-alert { + core_cluster_alert: core-cluster-alert { temperature = <85000>; hysteresis = <2000>; type = "passive"; }; - cpu_crit: cpu-crit { + core_cluster_crit: core-cluster-crit { temperature = <95000>; hysteresis = <2000>; type = "critical"; @@ -150,7 +150,42 @@ cooling-maps { map0 { - trip = <&cpu_alert>; + trip = <&core_cluster_alert>; + cooling-device = + <&cpu0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, + <&cpu1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, + <&cpu2 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, + <&cpu3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, + <&cpu4 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, + <&cpu5 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, + <&cpu6 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, + <&cpu7 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>; + }; + }; + }; + + platform-thermal { + polling-delay-passive = <1000>; + polling-delay = <5000>; + thermal-sensors = <&tmu 1>; + + trips { + platform_alert: platform-alert { + temperature = <85000>; + hysteresis = <2000>; + type = "passive"; + }; + + platform_crit: platform-crit { + temperature = <95000>; + hysteresis = <2000>; + type = "critical"; + }; + }; + + cooling-maps { + map0 { + trip = <&platform_alert>; cooling-device = <&cpu0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>, <&cpu1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
Ls1088a has 2 thermal sensors, core cluster and SoC platform. Core cluster sensor is used to monitor the temperature of core and SoC platform is for platform. The current dts only support the first sensor. This patch adds the second sensor node to dts to enable it. Signed-off-by: Yuantian Tang <andy.tang@nxp.com> --- v4: - use hyphen instead of underscore in node name v3: - use more descriptive name for each zone v2: - Add more information about sensors to description PS: In order to keep consistency to the first thermal-zone node, there will be "WARNING: line over 80 characters" warnings. arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi | 43 +++++++++++++++++++++-- 1 files changed, 39 insertions(+), 4 deletions(-)