zork/baseboard: move temperature sensors definition to board variant

There are some differences with temperature sensors for each zork variant.
This patch move temperature sensors definition to board variant.

BUG=b:157711232
BRANCH=none
TEST=Verified console "temps" can read temperature.
Build all zork board variant.

Signed-off-by: Devin Lu <Devin.Lu@quantatw.com>
Change-Id: Ib109ea5adc98e6b6825f8c91db19911dc682a52e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2227787
Reviewed-by: Edward Hill <ecgh@chromium.org>
Commit-Queue: Edward Hill <ecgh@chromium.org>
Tested-by: Edward Hill <ecgh@chromium.org>

13 files changed, 310 insertions, 70 deletions

diff --git a/baseboard/zork/baseboard.c b/baseboard/zork/baseboard.c
index 703e188024..a848341bb2 100644
--- a/baseboard/zork/baseboard.c
+++ b/baseboard/zork/baseboard.c
@@ -18,8 +18,6 @@
 #include "cros_board_info.h"
 #include "driver/accelgyro_bmi_common.h"
 #include "driver/retimer/pi3hdx1204.h"
-#include "driver/temp_sensor/sb_tsi.h"
-#include "driver/temp_sensor/tmp432.h"
 #include "driver/usb_mux/amd_fp5.h"
 #include "ec_commands.h"
 #include "extpower.h"
@@ -39,7 +37,6 @@
 #include "system.h"
 #include "task.h"
 #include "tcpci.h"
-#include "temp_sensor.h"
 #include "thermistor.h"
 #include "usb_mux.h"
 #include "usb_pd.h"
@@ -55,24 +52,6 @@ const enum gpio_signal hibernate_wake_pins[] = {
 };
 const int hibernate_wake_pins_used =  ARRAY_SIZE(hibernate_wake_pins);
 
-const struct adc_t adc_channels[] = {
-	[ADC_TEMP_SENSOR_CHARGER] = {
-		.name = "CHARGER",
-		.input_ch = NPCX_ADC_CH2,
-		.factor_mul = ADC_MAX_VOLT,
-		.factor_div = ADC_READ_MAX + 1,
-		.shift = 0,
-	},
-	[ADC_TEMP_SENSOR_SOC] = {
-		.name = "SOC",
-		.input_ch = NPCX_ADC_CH3,
-		.factor_mul = ADC_MAX_VOLT,
-		.factor_div = ADC_READ_MAX + 1,
-		.shift = 0,
-	},
-};
-BUILD_ASSERT(ARRAY_SIZE(adc_channels) == ADC_CH_COUNT);
-
 const struct power_signal_info power_signal_list[] = {
 	[X86_SLP_S3_N] = {
 		.gpio = GPIO_PCH_SLP_S3_L,
@@ -169,7 +148,7 @@ struct keyboard_scan_config keyscan_config = {
  * Values are calculated from the "Resistance VS. Temperature" table on the
  * Murata page for part NCP15WB473F03RC. Vdd=3.3V, R=30.9Kohm.
  */
-static const struct thermistor_data_pair thermistor_data[] = {
+const struct thermistor_data_pair thermistor_data[] = {
 	{ 2761 / THERMISTOR_SCALING_FACTOR, 0},
 	{ 2492 / THERMISTOR_SCALING_FACTOR, 10},
 	{ 2167 / THERMISTOR_SCALING_FACTOR, 20},
@@ -185,13 +164,13 @@ static const struct thermistor_data_pair thermistor_data[] = {
 	{ 283 / THERMISTOR_SCALING_FACTOR, 100}
 };
 
-static const struct thermistor_info thermistor_info = {
+const struct thermistor_info thermistor_info = {
 	.scaling_factor = THERMISTOR_SCALING_FACTOR,
 	.num_pairs = ARRAY_SIZE(thermistor_data),
 	.data = thermistor_data,
 };
 
-static int board_get_temp(int idx, int *temp_k)
+int board_get_temp(int idx, int *temp_k)
 {
 	int mv;
 	int temp_c;
@@ -222,36 +201,6 @@ static int board_get_temp(int idx, int *temp_k)
 	return EC_SUCCESS;
 }
 
-const struct temp_sensor_t temp_sensors[] = {
-	[TEMP_SENSOR_CHARGER] = {
-		.name = "Charger",
-		.type = TEMP_SENSOR_TYPE_BOARD,
-		.read = board_get_temp,
-		.idx = TEMP_SENSOR_CHARGER,
-	},
-	[TEMP_SENSOR_SOC] = {
-		.name = "SOC",
-		.type = TEMP_SENSOR_TYPE_BOARD,
-		.read = board_get_temp,
-		.idx = TEMP_SENSOR_SOC,
-	},
-	[TEMP_SENSOR_CPU] = {
-		.name = "CPU",
-		.type = TEMP_SENSOR_TYPE_CPU,
-		.read = sb_tsi_get_val,
-		.idx = 0,
-	},
-#ifdef BOARD_MORPHIUS
-	[TEMP_SENSOR_5V_REGULATOR] = {
-		.name = "5V_REGULATOR",
-		.type = TEMP_SENSOR_TYPE_BOARD,
-		.read = tmp432_get_val,
-		.idx = TMP432_IDX_LOCAL,
-	},
-#endif
-};
-BUILD_ASSERT(ARRAY_SIZE(temp_sensors) == TEMP_SENSOR_COUNT);
-
 #ifndef TEST_BUILD
 void lid_angle_peripheral_enable(int enable)
 {
diff --git a/baseboard/zork/baseboard.h b/baseboard/zork/baseboard.h
index 239c1d583d..04fd6d34c6 100644
--- a/baseboard/zork/baseboard.h
+++ b/baseboard/zork/baseboard.h
@@ -271,12 +271,6 @@
 #include "math_util.h"
 #include "registers.h"
 
-enum adc_channel {
-	ADC_TEMP_SENSOR_CHARGER,
-	ADC_TEMP_SENSOR_SOC,
-	ADC_CH_COUNT
-};
-
 enum power_signal {
 	X86_SLP_S3_N,
 	X86_SLP_S5_N,
@@ -285,16 +279,6 @@ enum power_signal {
 	POWER_SIGNAL_COUNT
 };
 
-enum temp_sensor_id {
-	TEMP_SENSOR_CHARGER = 0,
-	TEMP_SENSOR_SOC,
-	TEMP_SENSOR_CPU,
-#ifdef BOARD_MORPHIUS
-	TEMP_SENSOR_5V_REGULATOR,
-#endif
-	TEMP_SENSOR_COUNT
-};
-
 enum fan_channel {
 	FAN_CH_0 = 0,
 	/* Number of FAN channels */
@@ -359,6 +343,7 @@ int board_tcpc_fast_role_swap_enable(int port, int enable);
 
 void pi3hdx1204_retimer_power(void);
 __override_proto int check_hdmi_hpd_status(void);
+int board_get_temp(int idx, int *temp_k);
 
 #endif /* !__ASSEMBLER__ */
 
diff --git a/baseboard/zork/variant_dalboz.c b/baseboard/zork/variant_dalboz.c
index a3eb193e3e..bea1a15d9c 100644
--- a/baseboard/zork/variant_dalboz.c
+++ b/baseboard/zork/variant_dalboz.c
@@ -3,12 +3,56 @@
  * found in the LICENSE file.
  */
 
+#include "adc.h"
+#include "adc_chip.h"
 #include "charger.h"
 #include "common.h"
 #include "driver/charger/isl9241.h"
+#include "driver/temp_sensor/sb_tsi.h"
 #include "gpio.h"
 #include "hooks.h"
 #include "i2c.h"
+#include "temp_sensor.h"
+
+const struct adc_t adc_channels[] = {
+	[ADC_TEMP_SENSOR_CHARGER] = {
+		.name = "CHARGER",
+		.input_ch = NPCX_ADC_CH2,
+		.factor_mul = ADC_MAX_VOLT,
+		.factor_div = ADC_READ_MAX + 1,
+		.shift = 0,
+	},
+	[ADC_TEMP_SENSOR_SOC] = {
+		.name = "SOC",
+		.input_ch = NPCX_ADC_CH3,
+		.factor_mul = ADC_MAX_VOLT,
+		.factor_div = ADC_READ_MAX + 1,
+		.shift = 0,
+	},
+};
+BUILD_ASSERT(ARRAY_SIZE(adc_channels) == ADC_CH_COUNT);
+
+const struct temp_sensor_t temp_sensors[] = {
+	[TEMP_SENSOR_CHARGER] = {
+		.name = "Charger",
+		.type = TEMP_SENSOR_TYPE_BOARD,
+		.read = board_get_temp,
+		.idx = TEMP_SENSOR_CHARGER,
+	},
+	[TEMP_SENSOR_SOC] = {
+		.name = "SOC",
+		.type = TEMP_SENSOR_TYPE_BOARD,
+		.read = board_get_temp,
+		.idx = TEMP_SENSOR_SOC,
+	},
+	[TEMP_SENSOR_CPU] = {
+		.name = "CPU",
+		.type = TEMP_SENSOR_TYPE_CPU,
+		.read = sb_tsi_get_val,
+		.idx = 0,
+	},
+};
+BUILD_ASSERT(ARRAY_SIZE(temp_sensors) == TEMP_SENSOR_COUNT);
 
 const struct i2c_port_t i2c_ports[] = {
 	{
diff --git a/board/berknip/board.c b/board/berknip/board.c
index 1562c91eeb..634168dc89 100644
--- a/board/berknip/board.c
+++ b/board/berknip/board.c
@@ -5,12 +5,15 @@
 
 /* Berknip board configuration */
 
+#include "adc.h"
+#include "adc_chip.h"
 #include "button.h"
 #include "cbi_ec_fw_config.h"
 #include "driver/accelgyro_bmi_common.h"
 #include "driver/accel_kionix.h"
 #include "driver/accel_kx022.h"
 #include "driver/retimer/tusb544.h"
+#include "driver/temp_sensor/sb_tsi.h"
 #include "driver/usb_mux/amd_fp5.h"
 #include "driver/usb_mux/ps8743.h"
 #include "extpower.h"
@@ -26,6 +29,7 @@
 #include "switch.h"
 #include "system.h"
 #include "task.h"
+#include "temp_sensor.h"
 #include "usb_charge.h"
 #include "usb_mux.h"
 
@@ -313,6 +317,46 @@ const struct fan_t fans[] = {
 };
 BUILD_ASSERT(ARRAY_SIZE(fans) == FAN_CH_COUNT);
 
+const struct adc_t adc_channels[] = {
+	[ADC_TEMP_SENSOR_CHARGER] = {
+		.name = "CHARGER",
+		.input_ch = NPCX_ADC_CH2,
+		.factor_mul = ADC_MAX_VOLT,
+		.factor_div = ADC_READ_MAX + 1,
+		.shift = 0,
+	},
+	[ADC_TEMP_SENSOR_SOC] = {
+		.name = "SOC",
+		.input_ch = NPCX_ADC_CH3,
+		.factor_mul = ADC_MAX_VOLT,
+		.factor_div = ADC_READ_MAX + 1,
+		.shift = 0,
+	},
+};
+BUILD_ASSERT(ARRAY_SIZE(adc_channels) == ADC_CH_COUNT);
+
+const struct temp_sensor_t temp_sensors[] = {
+	[TEMP_SENSOR_CHARGER] = {
+		.name = "Charger",
+		.type = TEMP_SENSOR_TYPE_BOARD,
+		.read = board_get_temp,
+		.idx = TEMP_SENSOR_CHARGER,
+	},
+	[TEMP_SENSOR_SOC] = {
+		.name = "SOC",
+		.type = TEMP_SENSOR_TYPE_BOARD,
+		.read = board_get_temp,
+		.idx = TEMP_SENSOR_SOC,
+	},
+	[TEMP_SENSOR_CPU] = {
+		.name = "CPU",
+		.type = TEMP_SENSOR_TYPE_CPU,
+		.read = sb_tsi_get_val,
+		.idx = 0,
+	},
+};
+BUILD_ASSERT(ARRAY_SIZE(temp_sensors) == TEMP_SENSOR_COUNT);
+
 const static struct ec_thermal_config thermal_thermistor = {
 	.temp_host = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(75),
diff --git a/board/berknip/board.h b/board/berknip/board.h
index cf0fd6df07..83e5614ab6 100644
--- a/board/berknip/board.h
+++ b/board/berknip/board.h
@@ -69,6 +69,12 @@
 
 #ifndef __ASSEMBLER__
 
+enum adc_channel {
+	ADC_TEMP_SENSOR_CHARGER,
+	ADC_TEMP_SENSOR_SOC,
+	ADC_CH_COUNT
+};
+
 enum battery_type {
 	BATTERY_SIMPLO_HIGHPOWER,
 	BATTERY_COSMX,
@@ -87,6 +93,13 @@ enum pwm_channel {
 	PWM_CH_COUNT
 };
 
+enum temp_sensor_id {
+	TEMP_SENSOR_CHARGER = 0,
+	TEMP_SENSOR_SOC,
+	TEMP_SENSOR_CPU,
+	TEMP_SENSOR_COUNT
+};
+
 
 /*****************************************************************************
  * CBI EC FW Configuration
diff --git a/board/dalboz/board.h b/board/dalboz/board.h
index 99f92da64e..08c7a5aa42 100644
--- a/board/dalboz/board.h
+++ b/board/dalboz/board.h
@@ -83,6 +83,12 @@
 /* This I2C moved. Temporarily detect and support the V0 HW. */
 extern int I2C_PORT_BATTERY;
 
+enum adc_channel {
+	ADC_TEMP_SENSOR_CHARGER,
+	ADC_TEMP_SENSOR_SOC,
+	ADC_CH_COUNT
+};
+
 enum battery_type {
 	BATTERY_SMP,
 	BATTERY_LGC,
@@ -106,6 +112,13 @@ enum ioex_port {
 	? GPIO_USB3_C0_DP2_HPD \
 	: GPIO_DP1_HPD)
 
+enum temp_sensor_id {
+	TEMP_SENSOR_CHARGER = 0,
+	TEMP_SENSOR_SOC,
+	TEMP_SENSOR_CPU,
+	TEMP_SENSOR_COUNT
+};
+
 
 /*****************************************************************************
  * CBI EC FW Configuration
diff --git a/board/ezkinil/board.c b/board/ezkinil/board.c
index 8a9c979e27..ce0906ef35 100644
--- a/board/ezkinil/board.c
+++ b/board/ezkinil/board.c
@@ -3,12 +3,15 @@
  * found in the LICENSE file.
  */
 
+#include "adc.h"
+#include "adc_chip.h"
 #include "button.h"
 #include "charge_state_v2.h"
 #include "driver/accelgyro_bmi_common.h"
 #include "driver/accel_kionix.h"
 #include "driver/accel_kx022.h"
 #include "driver/retimer/tusb544.h"
+#include "driver/temp_sensor/sb_tsi.h"
 #include "driver/usb_mux/amd_fp5.h"
 #include "driver/usb_mux/ps8743.h"
 #include "extpower.h"
@@ -24,6 +27,7 @@
 #include "switch.h"
 #include "system.h"
 #include "task.h"
+#include "temp_sensor.h"
 #include "usb_charge.h"
 #include "usb_mux.h"
 
@@ -362,6 +366,46 @@ const struct fan_t fans[] = {
 };
 BUILD_ASSERT(ARRAY_SIZE(fans) == FAN_CH_COUNT);
 
+const struct adc_t adc_channels[] = {
+	[ADC_TEMP_SENSOR_CHARGER] = {
+		.name = "CHARGER",
+		.input_ch = NPCX_ADC_CH2,
+		.factor_mul = ADC_MAX_VOLT,
+		.factor_div = ADC_READ_MAX + 1,
+		.shift = 0,
+	},
+	[ADC_TEMP_SENSOR_SOC] = {
+		.name = "SOC",
+		.input_ch = NPCX_ADC_CH3,
+		.factor_mul = ADC_MAX_VOLT,
+		.factor_div = ADC_READ_MAX + 1,
+		.shift = 0,
+	},
+};
+BUILD_ASSERT(ARRAY_SIZE(adc_channels) == ADC_CH_COUNT);
+
+const struct temp_sensor_t temp_sensors[] = {
+	[TEMP_SENSOR_CHARGER] = {
+		.name = "Charger",
+		.type = TEMP_SENSOR_TYPE_BOARD,
+		.read = board_get_temp,
+		.idx = TEMP_SENSOR_CHARGER,
+	},
+	[TEMP_SENSOR_SOC] = {
+		.name = "SOC",
+		.type = TEMP_SENSOR_TYPE_BOARD,
+		.read = board_get_temp,
+		.idx = TEMP_SENSOR_SOC,
+	},
+	[TEMP_SENSOR_CPU] = {
+		.name = "CPU",
+		.type = TEMP_SENSOR_TYPE_CPU,
+		.read = sb_tsi_get_val,
+		.idx = 0,
+	},
+};
+BUILD_ASSERT(ARRAY_SIZE(temp_sensors) == TEMP_SENSOR_COUNT);
+
 const static struct ec_thermal_config thermal_thermistor = {
 	.temp_host = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(85),
diff --git a/board/ezkinil/board.h b/board/ezkinil/board.h
index f16c4badb4..7233a9e3ef 100644
--- a/board/ezkinil/board.h
+++ b/board/ezkinil/board.h
@@ -67,6 +67,12 @@
 
 #ifndef __ASSEMBLER__
 
+enum adc_channel {
+	ADC_TEMP_SENSOR_CHARGER,
+	ADC_TEMP_SENSOR_SOC,
+	ADC_CH_COUNT
+};
+
 enum battery_type {
 	BATTERY_AP19B8M,
 	BATTERY_AP18C7M,
@@ -85,6 +91,13 @@ enum pwm_channel {
 	PWM_CH_COUNT
 };
 
+enum temp_sensor_id {
+	TEMP_SENSOR_CHARGER = 0,
+	TEMP_SENSOR_SOC,
+	TEMP_SENSOR_CPU,
+	TEMP_SENSOR_COUNT
+};
+
 
 /*****************************************************************************
  * CBI EC FW Configuration
diff --git a/board/morphius/board.c b/board/morphius/board.c
index 34b223095b..3b6130d3bf 100644
--- a/board/morphius/board.c
+++ b/board/morphius/board.c
@@ -5,12 +5,16 @@
 
 /* Morphius board configuration */
 
+#include "adc.h"
+#include "adc_chip.h"
 #include "battery_smart.h"
 #include "button.h"
 #include "driver/accelgyro_bmi_common.h"
 #include "driver/accel_kionix.h"
 #include "driver/accel_kx022.h"
 #include "driver/retimer/pi3dpx1207.h"
+#include "driver/temp_sensor/sb_tsi.h"
+#include "driver/temp_sensor/tmp432.h"
 #include "driver/usb_mux/amd_fp5.h"
 #include "extpower.h"
 #include "gpio.h"
@@ -27,6 +31,7 @@
 #include "switch.h"
 #include "system.h"
 #include "task.h"
+#include "temp_sensor.h"
 #include "usb_mux.h"
 #include "usb_charge.h"
 
@@ -326,6 +331,52 @@ const struct fan_t fans[] = {
 };
 BUILD_ASSERT(ARRAY_SIZE(fans) == FAN_CH_COUNT);
 
+const struct adc_t adc_channels[] = {
+	[ADC_TEMP_SENSOR_CHARGER] = {
+		.name = "CHARGER",
+		.input_ch = NPCX_ADC_CH2,
+		.factor_mul = ADC_MAX_VOLT,
+		.factor_div = ADC_READ_MAX + 1,
+		.shift = 0,
+	},
+	[ADC_TEMP_SENSOR_SOC] = {
+		.name = "SOC",
+		.input_ch = NPCX_ADC_CH3,
+		.factor_mul = ADC_MAX_VOLT,
+		.factor_div = ADC_READ_MAX + 1,
+		.shift = 0,
+	},
+};
+BUILD_ASSERT(ARRAY_SIZE(adc_channels) == ADC_CH_COUNT);
+
+const struct temp_sensor_t temp_sensors[] = {
+	[TEMP_SENSOR_CHARGER] = {
+		.name = "Charger",
+		.type = TEMP_SENSOR_TYPE_BOARD,
+		.read = board_get_temp,
+		.idx = TEMP_SENSOR_CHARGER,
+	},
+	[TEMP_SENSOR_SOC] = {
+		.name = "SOC",
+		.type = TEMP_SENSOR_TYPE_BOARD,
+		.read = board_get_temp,
+		.idx = TEMP_SENSOR_SOC,
+	},
+	[TEMP_SENSOR_CPU] = {
+		.name = "CPU",
+		.type = TEMP_SENSOR_TYPE_CPU,
+		.read = sb_tsi_get_val,
+		.idx = 0,
+	},
+	[TEMP_SENSOR_5V_REGULATOR] = {
+		.name = "5V_REGULATOR",
+		.type = TEMP_SENSOR_TYPE_BOARD,
+		.read = tmp432_get_val,
+		.idx = TMP432_IDX_LOCAL,
+	},
+};
+BUILD_ASSERT(ARRAY_SIZE(temp_sensors) == TEMP_SENSOR_COUNT);
+
 const static struct ec_thermal_config thermal_thermistor = {
 	.temp_host = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(90),
diff --git a/board/morphius/board.h b/board/morphius/board.h
index a3058674e2..ba89b057b2 100644
--- a/board/morphius/board.h
+++ b/board/morphius/board.h
@@ -78,6 +78,12 @@
 
 void ps2_pwr_en_interrupt(enum gpio_signal signal);
 
+enum adc_channel {
+	ADC_TEMP_SENSOR_CHARGER,
+	ADC_TEMP_SENSOR_SOC,
+	ADC_CH_COUNT
+};
+
 enum battery_type {
 	BATTERY_SMP,
 	BATTERY_SUNWODA,
@@ -98,6 +104,14 @@ enum pwm_channel {
 	PWM_CH_COUNT
 };
 
+enum temp_sensor_id {
+	TEMP_SENSOR_CHARGER = 0,
+	TEMP_SENSOR_SOC,
+	TEMP_SENSOR_CPU,
+	TEMP_SENSOR_5V_REGULATOR,
+	TEMP_SENSOR_COUNT
+};
+
 
 /*****************************************************************************
  * CBI EC FW Configuration
diff --git a/board/trembyle/board.c b/board/trembyle/board.c
index 73dcf89f72..2a5abb448c 100644
--- a/board/trembyle/board.c
+++ b/board/trembyle/board.c
@@ -5,6 +5,8 @@
 
 /* Trembyle board configuration */
 
+#include "adc.h"
+#include "adc_chip.h"
 #include "button.h"
 #include "cbi_ec_fw_config.h"
 #include "driver/accelgyro_bmi_common.h"
@@ -12,6 +14,7 @@
 #include "driver/accel_kx022.h"
 #include "driver/retimer/pi3dpx1207.h"
 #include "driver/retimer/ps8811.h"
+#include "driver/temp_sensor/sb_tsi.h"
 #include "driver/usb_mux/amd_fp5.h"
 #include "extpower.h"
 #include "fan.h"
@@ -26,6 +29,7 @@
 #include "switch.h"
 #include "system.h"
 #include "task.h"
+#include "temp_sensor.h"
 #include "usb_charge.h"
 #include "usb_mux.h"
 
@@ -341,6 +345,46 @@ const struct fan_t fans[] = {
 };
 BUILD_ASSERT(ARRAY_SIZE(fans) == FAN_CH_COUNT);
 
+const struct adc_t adc_channels[] = {
+	[ADC_TEMP_SENSOR_CHARGER] = {
+		.name = "CHARGER",
+		.input_ch = NPCX_ADC_CH2,
+		.factor_mul = ADC_MAX_VOLT,
+		.factor_div = ADC_READ_MAX + 1,
+		.shift = 0,
+	},
+	[ADC_TEMP_SENSOR_SOC] = {
+		.name = "SOC",
+		.input_ch = NPCX_ADC_CH3,
+		.factor_mul = ADC_MAX_VOLT,
+		.factor_div = ADC_READ_MAX + 1,
+		.shift = 0,
+	},
+};
+BUILD_ASSERT(ARRAY_SIZE(adc_channels) == ADC_CH_COUNT);
+
+const struct temp_sensor_t temp_sensors[] = {
+	[TEMP_SENSOR_CHARGER] = {
+		.name = "Charger",
+		.type = TEMP_SENSOR_TYPE_BOARD,
+		.read = board_get_temp,
+		.idx = TEMP_SENSOR_CHARGER,
+	},
+	[TEMP_SENSOR_SOC] = {
+		.name = "SOC",
+		.type = TEMP_SENSOR_TYPE_BOARD,
+		.read = board_get_temp,
+		.idx = TEMP_SENSOR_SOC,
+	},
+	[TEMP_SENSOR_CPU] = {
+		.name = "CPU",
+		.type = TEMP_SENSOR_TYPE_CPU,
+		.read = sb_tsi_get_val,
+		.idx = 0,
+	},
+};
+BUILD_ASSERT(ARRAY_SIZE(temp_sensors) == TEMP_SENSOR_COUNT);
+
 const static struct ec_thermal_config thermal_thermistor = {
 	.temp_host = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(90),
diff --git a/board/trembyle/board.h b/board/trembyle/board.h
index a520b1279a..5be32fff01 100644
--- a/board/trembyle/board.h
+++ b/board/trembyle/board.h
@@ -63,6 +63,12 @@
 
 #ifndef __ASSEMBLER__
 
+enum adc_channel {
+	ADC_TEMP_SENSOR_CHARGER,
+	ADC_TEMP_SENSOR_SOC,
+	ADC_CH_COUNT
+};
+
 enum battery_type {
 	BATTERY_AP18F4M,
 	BATTERY_TYPE_COUNT,
@@ -80,6 +86,13 @@ enum pwm_channel {
 	PWM_CH_COUNT
 };
 
+enum temp_sensor_id {
+	TEMP_SENSOR_CHARGER = 0,
+	TEMP_SENSOR_SOC,
+	TEMP_SENSOR_CPU,
+	TEMP_SENSOR_COUNT
+};
+
 
 /*****************************************************************************
  * CBI EC FW Configuration
diff --git a/board/vilboz/board.h b/board/vilboz/board.h
index fad708dd5c..3af2e95a4e 100644
--- a/board/vilboz/board.h
+++ b/board/vilboz/board.h
@@ -76,6 +76,12 @@
 /* This I2C moved. Temporarily detect and support the V0 HW. */
 extern int I2C_PORT_BATTERY;
 
+enum adc_channel {
+	ADC_TEMP_SENSOR_CHARGER,
+	ADC_TEMP_SENSOR_SOC,
+	ADC_CH_COUNT
+};
+
 enum battery_type {
 	BATTERY_SMP,
 	BATTERY_LGC,
@@ -97,6 +103,13 @@ enum ioex_port {
 	? GPIO_USB3_C0_DP2_HPD \
 	: GPIO_DP1_HPD)
 
+enum temp_sensor_id {
+	TEMP_SENSOR_CHARGER = 0,
+	TEMP_SENSOR_SOC,
+	TEMP_SENSOR_CPU,
+	TEMP_SENSOR_COUNT
+};
+
 
 /*****************************************************************************
  * CBI EC FW Configuration