linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: imx8mq: Move thermal-zones out of bus node
@ 2019-03-25 15:19 Fabio Estevam
  2019-03-29  3:10 ` Shawn Guo
  0 siblings, 1 reply; 2+ messages in thread
From: Fabio Estevam @ 2019-03-25 15:19 UTC (permalink / raw)
  To: shawnguo; +Cc: linux-arm-kernel, angus, Fabio Estevam, linux-imx, kernel

thermal-zones node does not have any register properties and thus
shouldn't be placed inside the bus.

Move thermal-zones node from soc node to root node in order to fix
the following build warning with W=1:

arch/arm64/boot/dts/freescale/imx8mq.dtsi:305.18-364.6: Warning (simple_bus_reg): /soc@0/bus@30000000/thermal-zones: missing or empty reg/ranges property

Fixes: e464fd2ba4d4 ("arm64: dts: imx8mq: enable the multi sensor TMU")
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 arch/arm64/boot/dts/freescale/imx8mq.dtsi | 122 +++++++++++-----------
 1 file changed, 61 insertions(+), 61 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index 230f198ad87a..feb195984997 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -167,6 +167,67 @@
 		method = "smc";
 	};
 
+	thermal-zones {
+		cpu-thermal {
+			polling-delay-passive = <250>;
+			polling-delay = <2000>;
+			thermal-sensors = <&tmu 0>;
+
+			trips {
+				cpu_alert: cpu-alert {
+					temperature = <80000>;
+					hysteresis = <2000>;
+					type = "passive";
+				};
+
+				cpu-crit {
+					temperature = <90000>;
+					hysteresis = <2000>;
+					type = "critical";
+				};
+			};
+
+			cooling-maps {
+				map0 {
+					trip = <&cpu_alert>;
+					cooling-device =
+						<&A53_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
+						<&A53_1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
+						<&A53_2 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
+						<&A53_3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
+				};
+			};
+		};
+
+		gpu-thermal {
+			polling-delay-passive = <250>;
+			polling-delay = <2000>;
+			thermal-sensors = <&tmu 1>;
+
+			trips {
+				gpu-crit {
+					temperature = <90000>;
+					hysteresis = <2000>;
+					type = "critical";
+				};
+			};
+		};
+
+		vpu-thermal {
+			polling-delay-passive = <250>;
+			polling-delay = <2000>;
+			thermal-sensors = <&tmu 2>;
+
+			trips {
+				vpu-crit {
+					temperature = <90000>;
+					hysteresis = <2000>;
+					type = "critical";
+				};
+			};
+		};
+	};
+
 	timer {
 		compatible = "arm,armv8-timer";
 		interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_LOW>, /* Physical Secure */
@@ -302,67 +363,6 @@
 				#thermal-sensor-cells =  <1>;
 			};
 
-			thermal-zones {
-				cpu-thermal {
-					polling-delay-passive = <250>;
-					polling-delay = <2000>;
-					thermal-sensors = <&tmu 0>;
-
-					trips {
-						cpu_alert: cpu-alert {
-							temperature = <80000>;
-							hysteresis = <2000>;
-							type = "passive";
-						};
-
-						cpu-crit {
-							temperature = <90000>;
-							hysteresis = <2000>;
-							type = "critical";
-						};
-					};
-
-					cooling-maps {
-						map0 {
-							trip = <&cpu_alert>;
-							cooling-device =
-								<&A53_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
-								<&A53_1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
-								<&A53_2 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
-								<&A53_3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
-						};
-					};
-				};
-
-				gpu-thermal {
-					polling-delay-passive = <250>;
-					polling-delay = <2000>;
-					thermal-sensors = <&tmu 1>;
-
-					trips {
-						gpu-crit {
-							temperature = <90000>;
-							hysteresis = <2000>;
-							type = "critical";
-						};
-					};
-				};
-
-				vpu-thermal {
-					polling-delay-passive = <250>;
-					polling-delay = <2000>;
-					thermal-sensors = <&tmu 2>;
-
-					trips {
-						vpu-crit {
-							temperature = <90000>;
-							hysteresis = <2000>;
-							type = "critical";
-						};
-					};
-				};
-			};
-
 			wdog1: watchdog@30280000 {
 				compatible = "fsl,imx8mq-wdt", "fsl,imx21-wdt";
 				reg = <0x30280000 0x10000>;
-- 
2.17.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] arm64: dts: imx8mq: Move thermal-zones out of bus node
  2019-03-25 15:19 [PATCH] arm64: dts: imx8mq: Move thermal-zones out of bus node Fabio Estevam
@ 2019-03-29  3:10 ` Shawn Guo
  0 siblings, 0 replies; 2+ messages in thread
From: Shawn Guo @ 2019-03-29  3:10 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: linux-arm-kernel, angus, linux-imx, kernel

On Mon, Mar 25, 2019 at 12:19:59PM -0300, Fabio Estevam wrote:
> thermal-zones node does not have any register properties and thus
> shouldn't be placed inside the bus.
> 
> Move thermal-zones node from soc node to root node in order to fix
> the following build warning with W=1:
> 
> arch/arm64/boot/dts/freescale/imx8mq.dtsi:305.18-364.6: Warning (simple_bus_reg): /soc@0/bus@30000000/thermal-zones: missing or empty reg/ranges property
> 
> Fixes: e464fd2ba4d4 ("arm64: dts: imx8mq: enable the multi sensor TMU")

My branches could be rebased, so the commit id is not really stable.  I
dropped the tag and applied the patch.

Shawn

> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> ---
>  arch/arm64/boot/dts/freescale/imx8mq.dtsi | 122 +++++++++++-----------
>  1 file changed, 61 insertions(+), 61 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> index 230f198ad87a..feb195984997 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> @@ -167,6 +167,67 @@
>  		method = "smc";
>  	};
>  
> +	thermal-zones {
> +		cpu-thermal {
> +			polling-delay-passive = <250>;
> +			polling-delay = <2000>;
> +			thermal-sensors = <&tmu 0>;
> +
> +			trips {
> +				cpu_alert: cpu-alert {
> +					temperature = <80000>;
> +					hysteresis = <2000>;
> +					type = "passive";
> +				};
> +
> +				cpu-crit {
> +					temperature = <90000>;
> +					hysteresis = <2000>;
> +					type = "critical";
> +				};
> +			};
> +
> +			cooling-maps {
> +				map0 {
> +					trip = <&cpu_alert>;
> +					cooling-device =
> +						<&A53_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> +						<&A53_1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> +						<&A53_2 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> +						<&A53_3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> +				};
> +			};
> +		};
> +
> +		gpu-thermal {
> +			polling-delay-passive = <250>;
> +			polling-delay = <2000>;
> +			thermal-sensors = <&tmu 1>;
> +
> +			trips {
> +				gpu-crit {
> +					temperature = <90000>;
> +					hysteresis = <2000>;
> +					type = "critical";
> +				};
> +			};
> +		};
> +
> +		vpu-thermal {
> +			polling-delay-passive = <250>;
> +			polling-delay = <2000>;
> +			thermal-sensors = <&tmu 2>;
> +
> +			trips {
> +				vpu-crit {
> +					temperature = <90000>;
> +					hysteresis = <2000>;
> +					type = "critical";
> +				};
> +			};
> +		};
> +	};
> +
>  	timer {
>  		compatible = "arm,armv8-timer";
>  		interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_LOW>, /* Physical Secure */
> @@ -302,67 +363,6 @@
>  				#thermal-sensor-cells =  <1>;
>  			};
>  
> -			thermal-zones {
> -				cpu-thermal {
> -					polling-delay-passive = <250>;
> -					polling-delay = <2000>;
> -					thermal-sensors = <&tmu 0>;
> -
> -					trips {
> -						cpu_alert: cpu-alert {
> -							temperature = <80000>;
> -							hysteresis = <2000>;
> -							type = "passive";
> -						};
> -
> -						cpu-crit {
> -							temperature = <90000>;
> -							hysteresis = <2000>;
> -							type = "critical";
> -						};
> -					};
> -
> -					cooling-maps {
> -						map0 {
> -							trip = <&cpu_alert>;
> -							cooling-device =
> -								<&A53_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> -								<&A53_1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> -								<&A53_2 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
> -								<&A53_3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> -						};
> -					};
> -				};
> -
> -				gpu-thermal {
> -					polling-delay-passive = <250>;
> -					polling-delay = <2000>;
> -					thermal-sensors = <&tmu 1>;
> -
> -					trips {
> -						gpu-crit {
> -							temperature = <90000>;
> -							hysteresis = <2000>;
> -							type = "critical";
> -						};
> -					};
> -				};
> -
> -				vpu-thermal {
> -					polling-delay-passive = <250>;
> -					polling-delay = <2000>;
> -					thermal-sensors = <&tmu 2>;
> -
> -					trips {
> -						vpu-crit {
> -							temperature = <90000>;
> -							hysteresis = <2000>;
> -							type = "critical";
> -						};
> -					};
> -				};
> -			};
> -
>  			wdog1: watchdog@30280000 {
>  				compatible = "fsl,imx8mq-wdt", "fsl,imx21-wdt";
>  				reg = <0x30280000 0x10000>;
> -- 
> 2.17.1
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-03-29  3:11 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-25 15:19 [PATCH] arm64: dts: imx8mq: Move thermal-zones out of bus node Fabio Estevam
2019-03-29  3:10 ` Shawn Guo

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).