Re: [PATCH] thermal: qcom: tsens-v0_1: Add support for MDM9607

2021-01-27 Thread Konrad Dybcio
> Afaict, there's no need to initialize base0 and base1, they are both > assigned to before used. > > Also, they are temporary variables within each case (even with the > fallthrough), so you should only need a single "base". > Right. Fixed, will send in a v3 (I sent a v2 shortly after this one b

Re: [PATCH] thermal: qcom: tsens-v0_1: Add support for MDM9607

2021-01-27 Thread Bjorn Andersson
On Wed 27 Jan 12:14 CST 2021, Konrad Dybcio wrote: > MDM9607 TSENS IP is very similar to the one of MSM8916, with > minor adjustments to various tuning values. > > Signed-off-by: Konrad Dybcio > --- > .../bindings/thermal/qcom-tsens.yaml | 2 + > drivers/thermal/qcom/tsens-v0_1.c

[PATCH] thermal: qcom: tsens-v0_1: Add support for MDM9607

2021-01-27 Thread Konrad Dybcio
MDM9607 TSENS IP is very similar to the one of MSM8916, with minor adjustments to various tuning values. Signed-off-by: Konrad Dybcio --- .../bindings/thermal/qcom-tsens.yaml | 2 + drivers/thermal/qcom/tsens-v0_1.c | 100 +- drivers/thermal/qcom/tsens.c