[PATCH v7 3/7] arm64: dts: msm8996: thermal: Initialise via DT and add second controller
From: mka@chromium.org (Matthias Kaehlcke)
Date: 2018-07-17 23:42:30
Also in:
linux-arm-msm, linux-devicetree, lkml
On Thu, Jul 12, 2018 at 02:09:04PM +0530, Amit Kucheria wrote:
quoted hunk ↗ jump to hunk
We also split up the regmap address space into two, for the TM and SROT registers. This was required to deal with different address offsets for the TM and SROT registers across different SoC families. 8996 has two TSENS IP blocks, initialise the second one too. Since tsens-common.c/init_common() currently only registers one address space, the order is important (TM before SROT). This is OK since the code doesn't really use the SROT functionality yet. Signed-off-by: Amit Kucheria <redacted> Reviewed-by: Bjorn Andersson <redacted> Tested-by: Matthias Kaehlcke <mka@chromium.org> --- arch/arm64/boot/dts/qcom/msm8996.dtsi | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-)diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi index 8c7f9ca..688e752 100644 --- a/arch/arm64/boot/dts/qcom/msm8996.dtsi +++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi@@ -459,9 +459,19 @@ status = "disabled"; }; - tsens0: thermal-sensor at 4a8000 { + tsens0: thermal-sensor at 4a9000 {
~~~~~~ I suppose the address of the TM block is used here instead of the SROT address (which is lower) since SROT functionality is currently not used. Would/should this change if/when the driver uses SROT?
compatible = "qcom,msm8996-tsens";
- reg = <0x4a8000 0x2000>;
+ reg = <0x4a9000 0x1000>, /* TM */
+ <0x4a8000 0x1000>; /* SROT */
+ #qcom,sensors = <13>;
+ #thermal-sensor-cells = <1>;
+ };
+
+ tsens1: thermal-sensor at 4ad000 {
+ compatible = "qcom,msm8996-tsens";
+ reg = <0x4ad000 0x1000>, /* TM */
+ <0x4ac000 0x1000>; /* SROT */
+ #qcom,sensors = <8>;
#thermal-sensor-cells = <1>;
};