diff mbox series

[v2,3/7] arm64: dts: freescale: imx95: update a55 thermal trip points

Message ID 20240903-imx95-dts-new-v2-3-8ed795d61358@nxp.com (mailing list archive)
State New, archived
Headers show
Series arm64: dts: freescale: imx95: add various nodes | expand

Commit Message

Peng Fan (OSS) Sept. 3, 2024, 7:17 a.m. UTC
From: Peng Fan <peng.fan@nxp.com>

Update the thermal trip points for automotive and extended industrial
temperature qualification processors.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx95.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Alexander Stein Sept. 5, 2024, 12:24 p.m. UTC | #1
Am Dienstag, 3. September 2024, 09:17:48 CEST schrieb Peng Fan (OSS):
> From: Peng Fan <peng.fan@nxp.com>
> 
> Update the thermal trip points for automotive and extended industrial
> temperature qualification processors.
> 
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/imx95.dtsi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx95.dtsi b/arch/arm64/boot/dts/freescale/imx95.dtsi
> index 314a45e82c38..2cba7a889030 100644
> --- a/arch/arm64/boot/dts/freescale/imx95.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx95.dtsi
> @@ -347,13 +347,13 @@ a55-thermal {
>  
>  			trips {
>  				cpu_alert0: trip0 {
> -					temperature = <85000>;
> +					temperature = <105000>;
>  					hysteresis = <2000>;
>  					type = "passive";
>  				};
>  
>  				cpu_crit0: trip1 {
> -					temperature = <95000>;
> +					temperature = <125000>;

What about commercial and industrial temperature ranges? Can you detected
this at runtime? Is the bootloader supposed to correct these settings?

Best regards,
Alexander

>  					hysteresis = <2000>;
>  					type = "critical";
>  				};
> 
>
Peng Fan Sept. 6, 2024, 8:19 a.m. UTC | #2
> Subject: Re: [PATCH v2 3/7] arm64: dts: freescale: imx95: update a55
> thermal trip points
> 
> Am Dienstag, 3. September 2024, 09:17:48 CEST schrieb Peng Fan
> (OSS):
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > Update the thermal trip points for automotive and extended
> industrial
> > temperature qualification processors.
> >
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > ---
> >  arch/arm64/boot/dts/freescale/imx95.dtsi | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx95.dtsi
> > b/arch/arm64/boot/dts/freescale/imx95.dtsi
> > index 314a45e82c38..2cba7a889030 100644
> > --- a/arch/arm64/boot/dts/freescale/imx95.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx95.dtsi
> > @@ -347,13 +347,13 @@ a55-thermal {
> >
> >  			trips {
> >  				cpu_alert0: trip0 {
> > -					temperature = <85000>;
> > +					temperature = <105000>;
> >  					hysteresis = <2000>;
> >  					type = "passive";
> >  				};
> >
> >  				cpu_crit0: trip1 {
> > -					temperature = <95000>;
> > +					temperature = <125000>;
> 
> What about commercial and industrial temperature ranges? Can you
> detected this at runtime? Is the bootloader supposed to correct these
> settings?

Bootloader is supposed to update or people could overwrite in their
board device tree per which chip grade is being used.

It would be better if there is runtime update by kernel, but no as
of now. We may use nvmem to do runtime in future, but need check.

Regards,
Peng.

> 
> Best regards,
> Alexander
> 
> >  					hysteresis = <2000>;
> >  					type = "critical";
> >  				};
> >
> >
> 
> 
> --
> TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld,
> Germany Amtsgericht München, HRB 105018
> Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
> https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2F
> www.tq-
> group.com%2F&data=05%7C02%7Cpeng.fan%40nxp.com%7C1787566
> 643e742d9965508dccda5ad2d%7C686ea1d3bc2b4c6fa92cd99c5c301
> 635%7C0%7C0%7C638611358666637887%7CUnknown%7CTWFpbGZ
> sb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJX
> VCI6Mn0%3D%7C0%7C%7C%7C&sdata=vL3HTTjWm6F6dRQD4Oek0nj
> UiY%2FNnkqDv5z3k846EdY%3D&reserved=0
>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/freescale/imx95.dtsi b/arch/arm64/boot/dts/freescale/imx95.dtsi
index 314a45e82c38..2cba7a889030 100644
--- a/arch/arm64/boot/dts/freescale/imx95.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx95.dtsi
@@ -347,13 +347,13 @@  a55-thermal {
 
 			trips {
 				cpu_alert0: trip0 {
-					temperature = <85000>;
+					temperature = <105000>;
 					hysteresis = <2000>;
 					type = "passive";
 				};
 
 				cpu_crit0: trip1 {
-					temperature = <95000>;
+					temperature = <125000>;
 					hysteresis = <2000>;
 					type = "critical";
 				};