On Wed Jun 25, 2025 at 3:20 PM CEST, Konrad Dybcio wrote:
> On 6/25/25 11:18 AM, Luca Weiss wrote:
>> Add a dts for the PMIC used e.g. with SM7635 devices.
>>
>> Signed-off-by: Luca Weiss <luca.we...@fairphone.com>
>> ---
>> arch/arm64/boot/dts/qcom/pmxr2230.dtsi | 63
>> ++++++++++++++++++++++++++++++++++
>> 1 file changed, 63 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/pmxr2230.dtsi
>> b/arch/arm64/boot/dts/qcom/pmxr2230.dtsi
>> new file mode 100644
>> index
>> 0000000000000000000000000000000000000000..a7650f3230c01422b76ef78e897e5e7ae2f20cc6
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/pmxr2230.dtsi
>> @@ -0,0 +1,63 @@
>> +// SPDX-License-Identifier: BSD-3-Clause
>> +/*
>> + * Copyright (c) 2025, Luca Weiss <luca.we...@fairphone.com>
>> + */
>> +
>> +#include <dt-bindings/interrupt-controller/irq.h>
>> +#include <dt-bindings/spmi/spmi.h>
>> +
>> +/ {
>> + thermal-zones {
>> + pmxr2230_thermal: pmxr2230-thermal {
>> + polling-delay-passive = <100>;
>> +
>> + thermal-sensors = <&pmxr2230_temp_alarm>;
>> +
>> + trips {
>> + pmxr2230_trip0: trip0 {
>> + temperature = <95000>;
>> + hysteresis = <0>;
>> + type = "passive";
>> + };
>> +
>> + pmxr2230_crit: pmxr2230-crit {
>
> This name is "meh", please change to tripN
>
>> + temperature = <115000>;
>
> Unless there's some actual electrical/physical SKU differences,
> downstream lists 145C as critical for a PMIC carrying the same
> name
[ 0.085990] spmi-temp-alarm c400000.spmi:pmic@1:temp-alarm@a00: No ADC is
configured and critical temperature 145000 mC is above the maximum stage 2
threshold of 125000 mC! Configuring stage 2 shutdown at 125000 mC.
Same for the PMIV0104:
[ 0.086556] spmi-temp-alarm c400000.spmi:pmic@7:temp-alarm@a00: No ADC is
configured and critical temperature 145000 mC is above the maximum stage 2
threshold of 125000 mC! Configuring stage 2 shutdown at 125000 mC.
What shall I do?
Regards
Luca
>
> with that:
>
> Reviewed-by: Konrad Dybcio <konrad.dyb...@oss.qualcomm.com>
>
> Konrad
