On Fri, Apr 26, 2019 at 10:42 PM Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> wrote: > > On Mon 15 Apr 09:11 PDT 2019, Jeffrey Hugo wrote: > > diff --git a/arch/arm64/boot/dts/qcom/msm8998-clamshell.dtsi b/arch/arm64/boot/dts/qcom/msm8998-clamshell.dtsi > [..] > > + thermal-zones { > > + battery-thermal { > > + polling-delay-passive = <250>; > > + polling-delay = <1000>; > > + > > + thermal-sensors = <&tsens0 0>; > > I guess you inherited the battery and skin thermal nodes from my MTP > dts. Unfortunately after talking to Amit I think I got these wrong, and > they should be &pmi8998_adc 0 and 5 instead. > > Can you confirm this? Yeah, I pulled thermal from the MTP. Someone pointed this out on the v4 series. I haven't circled back to it yet. > > > + > > + trips { > > + battery_crit: trip0 { > > + temperature = <60000>; > > + hysteresis = <2000>; > > + type = "critical"; > > + }; > > + }; > > + }; > > Regards, > Bjorn