Messages in this thread | | | Date | Tue, 9 Jan 2024 11:09:42 +0100 | Subject | Re: [PATCH 2/2] arm64: dts: qcom: sm7225-fairphone-fp4: Add PM6150L thermals | From | Konrad Dybcio <> |
| |
On 1/5/24 15:54, Luca Weiss wrote: > Configure the thermals for the PA_THERM1, MSM_THERM, PA_THERM0, > RFC_CAM_THERM, CAM_FLASH_THERM and QUIET_THERM thermistors connected to > PM6150L. > > Due to hardware constraints we can only register 4 zones with > pm6150l_adc_tm, the other 2 we can register via generic-adc-thermal.
Ugh.. so the ADC can support more inputs than the ADC_TM that was designed to ship alongside it can?
And that's why the "generic-adc-thermal"-provided zones need to be polled?
> > The trip points can really only be considered as placeholders, more > configuration with cooling etc. can be added later. > > Signed-off-by: Luca Weiss <luca.weiss@fairphone.com> > --- [...]
I've read the sentence above, but.. > + sdm-skin-thermal { > + polling-delay-passive = <1000>; > + polling-delay = <5000>; > + thermal-sensors = <&msm_therm_sensor>; > + > + trips { > + active-config0 { > + temperature = <125000>; > + hysteresis = <1000>; > + type = "passive";
I don't fancy burnt fingers for dinner!
Konrad
| |