fwk: sb_tsi: poll to update the cpu die temperature

There are three functions call the temp_sensor_read().
This change creates the function to update the cpu die temperature once
per second.

BRANCH=fwk-dogwood-27111
BUG=None
TEST=use ec console command "i2ctrace enable 3 0x4c" to check that
     ec reads the temperature once per second.

Signed-off-by: Josh Tsai <Josh_Tsai@compal.com>

Author: Josh-Tsai

driver/temp_sensor/sb_tsi.c

@@ -17,25 +17,58 @@
 #include "temp_sensor/sb_tsi.h"
 #include "util.h"
 
+static int temp;
+
 static int raw_read8(const int offset, int *data_ptr)
 {
 	return i2c_read8(I2C_PORT_THERMAL_AP, SB_TSI_I2C_ADDR_FLAGS, offset,
 			 data_ptr);
 }
 
-int sb_tsi_get_val(int idx, int *temp_ptr)
+#ifndef CONFIG_ZEPHYR
+static void sb_tsi_poll(void)
+{
+	int data, ret;
+
+	/* FT4 SB-TSI sensor only powered in S0 */
+	if (!chipset_in_state(CHIPSET_STATE_ON) || chipset_in_low_power_mode())
+		return;
+
+	/* Read the value over I2C */
+	ret = raw_read8(SB_TSI_TEMP_H, &data);
+
+	if (ret)
+		return;
+
+	temp = C_TO_K(data);
+}
+DECLARE_HOOK(HOOK_SECOND, sb_tsi_poll, HOOK_PRIO_TEMP_SENSOR);
+#else
+void sb_tsi_update_temperature(int idx)
 {
-	int ret;
+	int data, ret;
+
 	/* There is only one temp sensor on the FT4 */
 	if (idx != 0)
-		return EC_ERROR_PARAM1;
+		return;
 	/* FT4 SB-TSI sensor only powered in S0 */
 	if (!chipset_in_state(CHIPSET_STATE_ON) || chipset_in_low_power_mode())
-		return EC_ERROR_NOT_POWERED;
+		return;
 	/* Read the value over I2C */
-	ret = raw_read8(SB_TSI_TEMP_H, temp_ptr);
+	ret = raw_read8(SB_TSI_TEMP_H, &data);
 	if (ret)
-		return ret;
-	*temp_ptr = C_TO_K(*temp_ptr);
+		return;
+	temp = C_TO_K(data);
+}
+#endif /* CONFIG_ZEPHYR */
+
+
+int sb_tsi_get_val(int idx, int *temp_ptr)
+{
+	/* There is only one temp sensor on the FT4 */
+	if (idx != 0)
+		return EC_ERROR_PARAM1;
+
+	*temp_ptr = temp;
 	return EC_SUCCESS;
 }

include/driver/temp_sensor/sb_tsi.h

@@ -33,6 +33,15 @@
 #define SB_TSI_MANUFACTURE_ID 0xFE
 #define SB_TSI_REVISION 0xFF
 
+#ifdef CONFIG_ZEPHYR
+/**
+ * Polling to read the sensor value
+ *
+ * @param idx		Index to read. Only 0 is valid for sb_tsi.
+ */
+void sb_tsi_update_temperature(int idx);
+#endif /* CONFIG_ZEPHYR */
+
 /**
  * Get the value of a sensor in K.
  *

zephyr/shim/src/temp_sensors.c

@@ -177,7 +177,7 @@ __maybe_unused static int sb_tsi_get_temp(const struct temp_sensor_t *sensor,
 #define GET_ZEPHYR_TEMP_SENSOR_SB_TSI(named_id)                          \
 	(&(const struct zephyr_temp_sensor){ .read = &sb_tsi_get_temp,   \
 					     .thermistor = NULL,         \
-					     .update_temperature = NULL, \
+					     .update_temperature = sb_tsi_update_temperature, \
 					     FILL_POWER_GOOD(named_id) })
 
 #define TEMP_SB_TSI(named_id, sensor_id)                                \