mediatek: mtk_thermal: replace with upstream patch
authorDaniel Golle <daniel@makrotopia.org>
Wed, 18 Jan 2023 03:14:29 +0000 (03:14 +0000)
committerDaniel Golle <daniel@makrotopia.org>
Wed, 15 Feb 2023 03:28:09 +0000 (03:28 +0000)
Replace local patch adding thermal support for MT7986 with version
accepted upstream.

Signed-off-by: Daniel Golle <daniel@makrotopia.org>
target/linux/mediatek/patches-5.15/805-thermal-drivers-mediatek-add-support-for-MT7986-and-.patch [deleted file]
target/linux/mediatek/patches-5.15/805-v6.2-thermal-drivers-mtk-use-function-pointer-for-raw_to_.patch [new file with mode: 0644]
target/linux/mediatek/patches-5.15/806-v6.2-thermal-mediatek-add-support-for-MT7986-and-MT7981.patch [new file with mode: 0644]

diff --git a/target/linux/mediatek/patches-5.15/805-thermal-drivers-mediatek-add-support-for-MT7986-and-.patch b/target/linux/mediatek/patches-5.15/805-thermal-drivers-mediatek-add-support-for-MT7986-and-.patch
deleted file mode 100644 (file)
index 7ec962a..0000000
+++ /dev/null
@@ -1,223 +0,0 @@
-From cd47d86ab09f1f3ec5c86441d4fe95e0cf597c06 Mon Sep 17 00:00:00 2001
-From: Daniel Golle <daniel@makrotopia.org>
-Date: Tue, 13 Sep 2022 00:56:24 +0100
-Subject: [PATCH] thermal/drivers/mediatek: add support for MT7986 and MT7981
-
-Add support for V3 generation thermal found in MT7986 and MT7981 SoCs.
-
-Signed-off-by: Daniel Golle <daniel@makrotopia.org>
----
- drivers/thermal/mtk_thermal.c | 202 +++++++++++++++++++++++++++++++++-
- 1 file changed, 198 insertions(+), 4 deletions(-)
-
---- a/drivers/thermal/mtk_thermal.c
-+++ b/drivers/thermal/mtk_thermal.c
-@@ -150,6 +150,21 @@
- #define CALIB_BUF1_VALID_V2(x)                (((x) >> 4) & 0x1)
- #define CALIB_BUF1_O_SLOPE_SIGN_V2(x) (((x) >> 3) & 0x1)
-+/*
-+ * Layout of the fuses providing the calibration data
-+ * These macros could be used for MT7981 and MT7986.
-+ */
-+#define CALIB_BUF0_ADC_GE_V3(x)               (((x) >> 0) & 0x3ff)
-+#define CALIB_BUF0_ADC_OE_V3(x)               (((x) >> 10) & 0x3ff)
-+#define CALIB_BUF0_DEGC_CALI_V3(x)    (((x) >> 20) & 0x3f)
-+#define CALIB_BUF0_O_SLOPE_V3(x)      (((x) >> 26) & 0x3f)
-+#define CALIB_BUF1_VTS_TS1_V3(x)      (((x) >> 0) & 0x1ff)
-+#define CALIB_BUF1_VTS_TS2_V3(x)      (((x) >> 21) & 0x1ff)
-+#define CALIB_BUF1_VTS_TSABB_V3(x)    (((x) >> 9) & 0x1ff)
-+#define CALIB_BUF1_VALID_V3(x)                (((x) >> 18) & 0x1)
-+#define CALIB_BUF1_O_SLOPE_SIGN_V3(x) (((x) >> 19) & 0x1)
-+#define CALIB_BUF1_ID_V3(x)           (((x) >> 20) & 0x1)
-+
- enum {
-       VTS1,
-       VTS2,
-@@ -163,6 +178,7 @@ enum {
- enum mtk_thermal_version {
-       MTK_THERMAL_V1 = 1,
-       MTK_THERMAL_V2,
-+      MTK_THERMAL_V3,
- };
- /* MT2701 thermal sensors */
-@@ -245,6 +261,27 @@ enum mtk_thermal_version {
- /* The calibration coefficient of sensor  */
- #define MT8183_CALIBRATION    153
-+/* AUXADC channel 11 is used for the temperature sensors */
-+#define MT7986_TEMP_AUXADC_CHANNEL    11
-+
-+/* The total number of temperature sensors in the MT7986 */
-+#define MT7986_NUM_SENSORS            1
-+
-+/* The number of banks in the MT7986 */
-+#define MT7986_NUM_ZONES              1
-+
-+/* The number of sensing points per bank */
-+#define MT7986_NUM_SENSORS_PER_ZONE   1
-+
-+/* MT7986 thermal sensors */
-+#define MT7986_TS1    0
-+
-+/* The number of controller in the MT7986 */
-+#define MT7986_NUM_CONTROLLER         1
-+
-+/* The calibration coefficient of sensor  */
-+#define MT7986_CALIBRATION            165
-+
- struct mtk_thermal;
- struct thermal_bank_cfg {
-@@ -386,6 +423,14 @@ static const int mt7622_mux_values[MT762
- static const int mt7622_vts_index[MT7622_NUM_SENSORS] = { VTS1 };
- static const int mt7622_tc_offset[MT7622_NUM_CONTROLLER] = { 0x0, };
-+/* MT7986 thermal sensor data */
-+static const int mt7986_bank_data[MT7986_NUM_SENSORS] = { MT7986_TS1, };
-+static const int mt7986_msr[MT7986_NUM_SENSORS_PER_ZONE] = { TEMP_MSR0, };
-+static const int mt7986_adcpnp[MT7986_NUM_SENSORS_PER_ZONE] = {       TEMP_ADCPNP0, };
-+static const int mt7986_mux_values[MT7986_NUM_SENSORS] = { 0, };
-+static const int mt7986_vts_index[MT7986_NUM_SENSORS] = { VTS1 };
-+static const int mt7986_tc_offset[MT7986_NUM_CONTROLLER] = { 0x0, };
-+
- /*
-  * The MT8173 thermal controller has four banks. Each bank can read up to
-  * four temperature sensors simultaneously. The MT8173 has a total of 5
-@@ -549,6 +594,30 @@ static const struct mtk_thermal_data mt8
-       .version = MTK_THERMAL_V1,
- };
-+/*
-+ * MT7986 uses AUXADC Channel 11 for raw data access.
-+ */
-+static const struct mtk_thermal_data mt7986_thermal_data = {
-+      .auxadc_channel = MT7986_TEMP_AUXADC_CHANNEL,
-+      .num_banks = MT7986_NUM_ZONES,
-+      .num_sensors = MT7986_NUM_SENSORS,
-+      .vts_index = mt7986_vts_index,
-+      .cali_val = MT7986_CALIBRATION,
-+      .num_controller = MT7986_NUM_CONTROLLER,
-+      .controller_offset = mt7986_tc_offset,
-+      .need_switch_bank = true,
-+      .bank_data = {
-+              {
-+                      .num_sensors = 1,
-+                      .sensors = mt7986_bank_data,
-+              },
-+      },
-+      .msr = mt7986_msr,
-+      .adcpnp = mt7986_adcpnp,
-+      .sensor_mux_values = mt7986_mux_values,
-+      .version = MTK_THERMAL_V3,
-+};
-+
- /**
-  * raw_to_mcelsius - convert a raw ADC value to mcelsius
-  * @mt:       The thermal controller
-@@ -603,6 +672,22 @@ static int raw_to_mcelsius_v2(struct mtk
-       return (format_2 - tmp) * 100;
- }
-+static int raw_to_mcelsius_v3(struct mtk_thermal *mt, int sensno, s32 raw)
-+{
-+      s32 tmp;
-+
-+      if (raw == 0)
-+              return 0;
-+
-+      raw &= 0xfff;
-+      tmp = 100000 * 15 / 16 * 10000;
-+      tmp /= 4096 - 512 + mt->adc_ge;
-+      tmp /= 1490;
-+      tmp *= raw - mt->vts[sensno] - 2900 - mt->adc_oe + 512;
-+
-+      return mt->degc_cali * 500 - tmp;
-+}
-+
- /**
-  * mtk_thermal_get_bank - get bank
-  * @bank:     The bank
-@@ -659,9 +744,12 @@ static int mtk_thermal_bank_temperature(
-               if (mt->conf->version == MTK_THERMAL_V1) {
-                       temp = raw_to_mcelsius_v1(
-                               mt, conf->bank_data[bank->id].sensors[i], raw);
--              } else {
-+              } else if (mt->conf->version == MTK_THERMAL_V2) {
-                       temp = raw_to_mcelsius_v2(
-                               mt, conf->bank_data[bank->id].sensors[i], raw);
-+              } else {
-+                      temp = raw_to_mcelsius_v3(
-+                              mt, conf->bank_data[bank->id].sensors[i], raw);
-               }
-               /*
-@@ -887,6 +975,26 @@ static int mtk_thermal_extract_efuse_v2(
-       return 0;
- }
-+static int mtk_thermal_extract_efuse_v3(struct mtk_thermal *mt, u32 *buf)
-+{
-+      if (!CALIB_BUF1_VALID_V3(buf[1]))
-+              return -EINVAL;
-+
-+      mt->adc_oe = CALIB_BUF0_ADC_OE_V3(buf[0]);
-+      mt->adc_ge = CALIB_BUF0_ADC_GE_V3(buf[0]);
-+      mt->degc_cali = CALIB_BUF0_DEGC_CALI_V3(buf[0]);
-+      mt->o_slope = CALIB_BUF0_O_SLOPE_V3(buf[0]);
-+      mt->vts[VTS1] = CALIB_BUF1_VTS_TS1_V3(buf[1]);
-+      mt->vts[VTS2] = CALIB_BUF1_VTS_TS2_V3(buf[1]);
-+      mt->vts[VTSABB] = CALIB_BUF1_VTS_TSABB_V3(buf[1]);
-+      mt->o_slope_sign = CALIB_BUF1_O_SLOPE_SIGN_V3(buf[1]);
-+
-+      if (CALIB_BUF1_ID_V3(buf[1]) == 0)
-+              mt->o_slope = 0;
-+
-+      return 0;
-+}
-+
- static int mtk_thermal_get_calibration_data(struct device *dev,
-                                           struct mtk_thermal *mt)
- {
-@@ -897,6 +1005,7 @@ static int mtk_thermal_get_calibration_d
-       /* Start with default values */
-       mt->adc_ge = 512;
-+      mt->adc_oe = 512;
-       for (i = 0; i < mt->conf->num_sensors; i++)
-               mt->vts[i] = 260;
-       mt->degc_cali = 40;
-@@ -924,8 +1033,10 @@ static int mtk_thermal_get_calibration_d
-       if (mt->conf->version == MTK_THERMAL_V1)
-               ret = mtk_thermal_extract_efuse_v1(mt, buf);
--      else
-+      else if (mt->conf->version == MTK_THERMAL_V2)
-               ret = mtk_thermal_extract_efuse_v2(mt, buf);
-+      else
-+              ret = mtk_thermal_extract_efuse_v3(mt, buf);
-       if (ret) {
-               dev_info(dev, "Device not calibrated, using default calibration values\n");
-@@ -956,6 +1067,10 @@ static const struct of_device_id mtk_the
-               .data = (void *)&mt7622_thermal_data,
-       },
-       {
-+              .compatible = "mediatek,mt7986-thermal",
-+              .data = (void *)&mt7986_thermal_data,
-+      },
-+      {
-               .compatible = "mediatek,mt8183-thermal",
-               .data = (void *)&mt8183_thermal_data,
-       }, {
-@@ -1070,7 +1185,8 @@ static int mtk_thermal_probe(struct plat
-               goto err_disable_clk_auxadc;
-       }
--      if (mt->conf->version == MTK_THERMAL_V2) {
-+      if (mt->conf->version == MTK_THERMAL_V2 ||
-+          mt->conf->version == MTK_THERMAL_V3) {
-               mtk_thermal_turn_on_buffer(apmixed_base);
-               mtk_thermal_release_periodic_ts(mt, auxadc_base);
-       }
diff --git a/target/linux/mediatek/patches-5.15/805-v6.2-thermal-drivers-mtk-use-function-pointer-for-raw_to_.patch b/target/linux/mediatek/patches-5.15/805-v6.2-thermal-drivers-mtk-use-function-pointer-for-raw_to_.patch
new file mode 100644 (file)
index 0000000..c58ae96
--- /dev/null
@@ -0,0 +1,56 @@
+From 69c17529e8418da3eec703dde31e1b01e5b0f7e8 Mon Sep 17 00:00:00 2001
+From: Daniel Golle <daniel@makrotopia.org>
+Date: Wed, 18 Jan 2023 02:48:41 +0000
+Subject: [PATCH 1/2] thermal/drivers/mtk: use function pointer for
+ raw_to_mcelsius
+
+Instead of having if-else logic selecting either raw_to_mcelsius_v1 or
+raw_to_mcelsius_v2 in mtk_thermal_bank_temperature introduce a function
+pointer raw_to_mcelsius to struct mtk_thermal which is initialized in the
+probe function.
+
+Signed-off-by: Daniel Golle <daniel@makrotopia.org>
+---
+ drivers/thermal/mtk_thermal.c | 17 ++++++++++-------
+ 1 file changed, 10 insertions(+), 7 deletions(-)
+
+--- a/drivers/thermal/mtk_thermal.c
++++ b/drivers/thermal/mtk_thermal.c
+@@ -292,6 +292,8 @@ struct mtk_thermal {
+       const struct mtk_thermal_data *conf;
+       struct mtk_thermal_bank banks[MAX_NUM_ZONES];
++
++      int (*raw_to_mcelsius)(struct mtk_thermal *mt, int sensno, s32 raw);
+ };
+ /* MT8183 thermal sensor data */
+@@ -656,13 +658,9 @@ static int mtk_thermal_bank_temperature(
+       for (i = 0; i < conf->bank_data[bank->id].num_sensors; i++) {
+               raw = readl(mt->thermal_base + conf->msr[i]);
+-              if (mt->conf->version == MTK_THERMAL_V1) {
+-                      temp = raw_to_mcelsius_v1(
+-                              mt, conf->bank_data[bank->id].sensors[i], raw);
+-              } else {
+-                      temp = raw_to_mcelsius_v2(
+-                              mt, conf->bank_data[bank->id].sensors[i], raw);
+-              }
++              temp = mt->raw_to_mcelsius(
++                      mt, conf->bank_data[bank->id].sensors[i], raw);
++
+               /*
+                * The first read of a sensor often contains very high bogus
+@@ -1075,6 +1073,11 @@ static int mtk_thermal_probe(struct plat
+               mtk_thermal_release_periodic_ts(mt, auxadc_base);
+       }
++      if (mt->conf->version == MTK_THERMAL_V1)
++              mt->raw_to_mcelsius = raw_to_mcelsius_v1;
++      else
++              mt->raw_to_mcelsius = raw_to_mcelsius_v2;
++
+       for (ctrl_id = 0; ctrl_id < mt->conf->num_controller ; ctrl_id++)
+               for (i = 0; i < mt->conf->num_banks; i++)
+                       mtk_thermal_init_bank(mt, i, apmixed_phys_base,
diff --git a/target/linux/mediatek/patches-5.15/806-v6.2-thermal-mediatek-add-support-for-MT7986-and-MT7981.patch b/target/linux/mediatek/patches-5.15/806-v6.2-thermal-mediatek-add-support-for-MT7986-and-MT7981.patch
new file mode 100644 (file)
index 0000000..65311d5
--- /dev/null
@@ -0,0 +1,240 @@
+From aa957c759b1182aee00cc35178667f849f941b42 Mon Sep 17 00:00:00 2001
+From: Daniel Golle <daniel@makrotopia.org>
+Date: Wed, 30 Nov 2022 13:19:39 +0000
+Subject: [PATCH 2/2] thermal: mediatek: add support for MT7986 and MT7981
+
+Add support for V3 generation thermal found in MT7986 and MT7981 SoCs.
+Brings code to assign values from efuse as well as new function to
+convert raw temperature to millidegree celsius, as found in MediaTek's
+SDK sources (but cleaned up and de-duplicated)
+
+[1]: https://git01.mediatek.com/plugins/gitiles/openwrt/feeds/mtk-openwrt-feeds/+/baf36c7eef477aae1f8f2653b6c29e2caf48475b
+Signed-off-by: Daniel Golle <daniel@makrotopia.org>
+---
+ drivers/thermal/mtk_thermal.c | 137 ++++++++++++++++++++++++++++++++--
+ 1 file changed, 132 insertions(+), 5 deletions(-)
+
+--- a/drivers/thermal/mtk_thermal.c
++++ b/drivers/thermal/mtk_thermal.c
+@@ -150,6 +150,20 @@
+ #define CALIB_BUF1_VALID_V2(x)                (((x) >> 4) & 0x1)
+ #define CALIB_BUF1_O_SLOPE_SIGN_V2(x) (((x) >> 3) & 0x1)
++/*
++ * Layout of the fuses providing the calibration data
++ * These macros can be used for MT7981 and MT7986.
++ */
++#define CALIB_BUF0_ADC_GE_V3(x)               (((x) >> 0) & 0x3ff)
++#define CALIB_BUF0_DEGC_CALI_V3(x)    (((x) >> 20) & 0x3f)
++#define CALIB_BUF0_O_SLOPE_V3(x)      (((x) >> 26) & 0x3f)
++#define CALIB_BUF1_VTS_TS1_V3(x)      (((x) >> 0) & 0x1ff)
++#define CALIB_BUF1_VTS_TS2_V3(x)      (((x) >> 21) & 0x1ff)
++#define CALIB_BUF1_VTS_TSABB_V3(x)    (((x) >> 9) & 0x1ff)
++#define CALIB_BUF1_VALID_V3(x)                (((x) >> 18) & 0x1)
++#define CALIB_BUF1_O_SLOPE_SIGN_V3(x) (((x) >> 19) & 0x1)
++#define CALIB_BUF1_ID_V3(x)           (((x) >> 20) & 0x1)
++
+ enum {
+       VTS1,
+       VTS2,
+@@ -163,6 +177,7 @@ enum {
+ enum mtk_thermal_version {
+       MTK_THERMAL_V1 = 1,
+       MTK_THERMAL_V2,
++      MTK_THERMAL_V3,
+ };
+ /* MT2701 thermal sensors */
+@@ -245,6 +260,27 @@ enum mtk_thermal_version {
+ /* The calibration coefficient of sensor  */
+ #define MT8183_CALIBRATION    153
++/* AUXADC channel 11 is used for the temperature sensors */
++#define MT7986_TEMP_AUXADC_CHANNEL    11
++
++/* The total number of temperature sensors in the MT7986 */
++#define MT7986_NUM_SENSORS            1
++
++/* The number of banks in the MT7986 */
++#define MT7986_NUM_ZONES              1
++
++/* The number of sensing points per bank */
++#define MT7986_NUM_SENSORS_PER_ZONE   1
++
++/* MT7986 thermal sensors */
++#define MT7986_TS1                    0
++
++/* The number of controller in the MT7986 */
++#define MT7986_NUM_CONTROLLER         1
++
++/* The calibration coefficient of sensor  */
++#define MT7986_CALIBRATION            165
++
+ struct mtk_thermal;
+ struct thermal_bank_cfg {
+@@ -388,6 +424,14 @@ static const int mt7622_mux_values[MT762
+ static const int mt7622_vts_index[MT7622_NUM_SENSORS] = { VTS1 };
+ static const int mt7622_tc_offset[MT7622_NUM_CONTROLLER] = { 0x0, };
++/* MT7986 thermal sensor data */
++static const int mt7986_bank_data[MT7986_NUM_SENSORS] = { MT7986_TS1, };
++static const int mt7986_msr[MT7986_NUM_SENSORS_PER_ZONE] = { TEMP_MSR0, };
++static const int mt7986_adcpnp[MT7986_NUM_SENSORS_PER_ZONE] = { TEMP_ADCPNP0, };
++static const int mt7986_mux_values[MT7986_NUM_SENSORS] = { 0, };
++static const int mt7986_vts_index[MT7986_NUM_SENSORS] = { VTS1 };
++static const int mt7986_tc_offset[MT7986_NUM_CONTROLLER] = { 0x0, };
++
+ /*
+  * The MT8173 thermal controller has four banks. Each bank can read up to
+  * four temperature sensors simultaneously. The MT8173 has a total of 5
+@@ -551,6 +595,30 @@ static const struct mtk_thermal_data mt8
+       .version = MTK_THERMAL_V1,
+ };
++/*
++ * MT7986 uses AUXADC Channel 11 for raw data access.
++ */
++static const struct mtk_thermal_data mt7986_thermal_data = {
++      .auxadc_channel = MT7986_TEMP_AUXADC_CHANNEL,
++      .num_banks = MT7986_NUM_ZONES,
++      .num_sensors = MT7986_NUM_SENSORS,
++      .vts_index = mt7986_vts_index,
++      .cali_val = MT7986_CALIBRATION,
++      .num_controller = MT7986_NUM_CONTROLLER,
++      .controller_offset = mt7986_tc_offset,
++      .need_switch_bank = true,
++      .bank_data = {
++              {
++                      .num_sensors = 1,
++                      .sensors = mt7986_bank_data,
++              },
++      },
++      .msr = mt7986_msr,
++      .adcpnp = mt7986_adcpnp,
++      .sensor_mux_values = mt7986_mux_values,
++      .version = MTK_THERMAL_V3,
++};
++
+ /**
+  * raw_to_mcelsius - convert a raw ADC value to mcelsius
+  * @mt:       The thermal controller
+@@ -605,6 +673,22 @@ static int raw_to_mcelsius_v2(struct mtk
+       return (format_2 - tmp) * 100;
+ }
++static int raw_to_mcelsius_v3(struct mtk_thermal *mt, int sensno, s32 raw)
++{
++      s32 tmp;
++
++      if (raw == 0)
++              return 0;
++
++      raw &= 0xfff;
++      tmp = 100000 * 15 / 16 * 10000;
++      tmp /= 4096 - 512 + mt->adc_ge;
++      tmp /= 1490;
++      tmp *= raw - mt->vts[sensno] - 2900;
++
++      return mt->degc_cali * 500 - tmp;
++}
++
+ /**
+  * mtk_thermal_get_bank - get bank
+  * @bank:     The bank
+@@ -885,6 +969,25 @@ static int mtk_thermal_extract_efuse_v2(
+       return 0;
+ }
++static int mtk_thermal_extract_efuse_v3(struct mtk_thermal *mt, u32 *buf)
++{
++      if (!CALIB_BUF1_VALID_V3(buf[1]))
++              return -EINVAL;
++
++      mt->adc_ge = CALIB_BUF0_ADC_GE_V3(buf[0]);
++      mt->degc_cali = CALIB_BUF0_DEGC_CALI_V3(buf[0]);
++      mt->o_slope = CALIB_BUF0_O_SLOPE_V3(buf[0]);
++      mt->vts[VTS1] = CALIB_BUF1_VTS_TS1_V3(buf[1]);
++      mt->vts[VTS2] = CALIB_BUF1_VTS_TS2_V3(buf[1]);
++      mt->vts[VTSABB] = CALIB_BUF1_VTS_TSABB_V3(buf[1]);
++      mt->o_slope_sign = CALIB_BUF1_O_SLOPE_SIGN_V3(buf[1]);
++
++      if (CALIB_BUF1_ID_V3(buf[1]) == 0)
++              mt->o_slope = 0;
++
++      return 0;
++}
++
+ static int mtk_thermal_get_calibration_data(struct device *dev,
+                                           struct mtk_thermal *mt)
+ {
+@@ -895,6 +998,7 @@ static int mtk_thermal_get_calibration_d
+       /* Start with default values */
+       mt->adc_ge = 512;
++      mt->adc_oe = 512;
+       for (i = 0; i < mt->conf->num_sensors; i++)
+               mt->vts[i] = 260;
+       mt->degc_cali = 40;
+@@ -920,10 +1024,20 @@ static int mtk_thermal_get_calibration_d
+               goto out;
+       }
+-      if (mt->conf->version == MTK_THERMAL_V1)
++      switch (mt->conf->version) {
++      case MTK_THERMAL_V1:
+               ret = mtk_thermal_extract_efuse_v1(mt, buf);
+-      else
++              break;
++      case MTK_THERMAL_V2:
+               ret = mtk_thermal_extract_efuse_v2(mt, buf);
++              break;
++      case MTK_THERMAL_V3:
++              ret = mtk_thermal_extract_efuse_v3(mt, buf);
++              break;
++      default:
++              ret = -EINVAL;
++              break;
++      }
+       if (ret) {
+               dev_info(dev, "Device not calibrated, using default calibration values\n");
+@@ -954,6 +1068,10 @@ static const struct of_device_id mtk_the
+               .data = (void *)&mt7622_thermal_data,
+       },
+       {
++              .compatible = "mediatek,mt7986-thermal",
++              .data = (void *)&mt7986_thermal_data,
++      },
++      {
+               .compatible = "mediatek,mt8183-thermal",
+               .data = (void *)&mt8183_thermal_data,
+       }, {
+@@ -1068,15 +1186,24 @@ static int mtk_thermal_probe(struct plat
+               goto err_disable_clk_auxadc;
+       }
+-      if (mt->conf->version == MTK_THERMAL_V2) {
++      if (mt->conf->version != MTK_THERMAL_V1) {
+               mtk_thermal_turn_on_buffer(apmixed_base);
+               mtk_thermal_release_periodic_ts(mt, auxadc_base);
+       }
+-      if (mt->conf->version == MTK_THERMAL_V1)
++      switch (mt->conf->version) {
++      case MTK_THERMAL_V1:
+               mt->raw_to_mcelsius = raw_to_mcelsius_v1;
+-      else
++              break;
++      case MTK_THERMAL_V2:
+               mt->raw_to_mcelsius = raw_to_mcelsius_v2;
++              break;
++      case MTK_THERMAL_V3:
++              mt->raw_to_mcelsius = raw_to_mcelsius_v3;
++              break;
++      default:
++              break;
++      }
+       for (ctrl_id = 0; ctrl_id < mt->conf->num_controller ; ctrl_id++)
+               for (i = 0; i < mt->conf->num_banks; i++)