Re: [PATCH 2/3] arm64: dts: ti: k3-am62a: Set Critical Temp. to 105C

[Nishanth Menon <nm@ti.com>]

On 21:11-20250502, Daniel Schultz wrote:
> 
> On 5/2/25 16:49, Nishanth Menon wrote:
> > On 07:26-20250502, Daniel Schultz wrote:
> > > The AM62Ax SoC supports two temperature ranges:
> > > * A: -40 to 105C - Extended Industrial
> > > * I: -40 to 125C - Automotive
> > > 
> > > By default, use the lower limit (105 °C) so that any AM62A running
> > > in Extended Industrial mode will shut down safely before overheating.
> > > 
> > > If the bootloader detects an Automotive-grade device, it should
> > > override this and raise the critical trip point to 125 °C.
> > > 
> > > Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
> > > ---
> > >   arch/arm64/boot/dts/ti/k3-am62a-thermal.dtsi | 6 +++---
> > >   1 file changed, 3 insertions(+), 3 deletions(-)
> > > 
> > > diff --git a/arch/arm64/boot/dts/ti/k3-am62a-thermal.dtsi b/arch/arm64/boot/dts/ti/k3-am62a-thermal.dtsi
> > > index 39ff9118b6c4..40dcb9bab965 100644
> > > --- a/arch/arm64/boot/dts/ti/k3-am62a-thermal.dtsi
> > > +++ b/arch/arm64/boot/dts/ti/k3-am62a-thermal.dtsi
> > Why not create a separate header for industrial grade and rename this as
> > automotive grade and let the board file pick the right grade used on the
> > board?
> 
> Mostly because I copied this from 10e7bfd8114c207ac and with the most recent
> temperature fixups in U-Boot, we also have the missing bootloader logic.
> 
> I would also prefer an out-of-the-box solution for that.

Personally, I am not in favor of depending on U-boot (there are other
bootloaders as well in the ecosystem) monkeying with dt nodes.

I suggest renaming the dtsi and introducing a industrial dtsi. And
depending on the samples used by the board, use the correct dtsi.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D