fwk: isl9241: add the function to read the charger temperature

This change add the isl9241 temperature yaml file and function to
read the isl9241 charger temperature.

BRANCH=fwk-sunflower-26784
BUG=https://app.clickup.com/t/86et2rz87
TEST=use the console command "charger dump" to check the 0x82 register value
     and calculate the value to Celsius and then check the value is the same
     as the "temps".

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

Author: Josh-Tsai

driver/charger/isl9241.c

@@ -1464,3 +1464,20 @@ const struct charger_drv isl9241_drv = {
 	.dump_prochot = &isl9241_dump_prochot_status,
 #endif
 };
+
+int isl9241_get_temperature_val(int idx, int *temp_ptr)
+{
+	int data;
+	uint32_t temp;
+
+	if (idx != 0)
+		return EC_ERROR_PARAM1;
+
+	RETURN_ERROR(isl9241_read(idx, ISL9241_REG_TJ_ADC_RESULTS, &data));
+
+	/* Section 6.5.4 ADC Operation describes the algorithm */
+	temp = 10000 * (252 - data) / 6154;
+	*temp_ptr = C_TO_K(temp);
+
+	return EC_SUCCESS;
+}

driver/charger/isl9241.h

@@ -205,4 +205,14 @@
  */
 int isl9241_set_csin_discharge_fet(int chgnum, bool enable);
 
+/**
+ * Use this function to get the charger ISL9241 temperature.
+ *
+ * @param idx: sensor index
+ * @param temp_ptr: temperature value from charger
+ *
+ * @result EC_SUCCESS if gets the vlaue
+ */
+int isl9241_get_temperature_val(int idx, int *temp_ptr);
+
 #endif /* __CROS_EC_ISL9241_H */

zephyr/dts/bindings/temp/cros-ec,temp-sensor-isl9241.yaml

@@ -0,0 +1,10 @@
+# Copyright 2023 The ChromiumOS Authors
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+description: >
+  Get the last polled charger isl9241 temperature.
+
+compatible: "cros-ec,temp-sensor-isl9241"
+
+include: i2c-device.yaml

zephyr/shim/include/temp_sensor/temp_sensor.h

@@ -22,6 +22,7 @@ extern "C" {
 #define F75303_COMPAT cros_ec_temp_sensor_f75303
 #define F75397_COMPAT cros_ec_temp_sensor_f75397
 #define BATTERY_COMPAT cros_ec_temp_sensor_battery
+#define ISL9241_COMPAT cros_ec_temp_sensor_isl9241
 #define SB_TSI_COMPAT cros_ec_temp_sensor_sb_tsi
 #define THERMISTOR_COMPAT cros_ec_temp_sensor_thermistor
 #define TEMP_SENSORS_COMPAT cros_ec_temp_sensors
@@ -39,6 +40,7 @@ extern "C" {
 	DT_FOREACH_STATUS_OKAY(F75303_COMPAT, fn)                           \
 	DT_FOREACH_STATUS_OKAY(F75397_COMPAT, fn)                           \
 	DT_FOREACH_STATUS_OKAY(BATTERY_COMPAT, fn)                           \
+	DT_FOREACH_STATUS_OKAY(ISL9241_COMPAT, fn)                           \
 	DT_FOREACH_STATUS_OKAY(AMDR23M_COMPAT, fn)                          \
 	DT_FOREACH_STATUS_OKAY(PECI_COMPAT, fn)                          \
 	DT_FOREACH_STATUS_OKAY_VARGS(RT9490_CHG_COMPAT, TEMP_RT9490_FN, fn) \

zephyr/shim/src/temp_sensors.c

@@ -13,6 +13,7 @@
 #include "charge_state.h"
 #include "charger/chg_rt9490.h"
 #include "driver/charger/rt9490.h"
+#include "driver/charger/isl9241.h"
 #include "hooks.h"
 #include "peci.h"
 #include "temp_sensor.h"
@@ -320,6 +321,31 @@ __maybe_unused static int battery_get_temp(const struct temp_sensor_t *sensor,
 		.zephyr_info = GET_ZEPHYR_TEMP_SENSOR_BATTERY(named_id), \
 	}
 
+
+#if DT_HAS_COMPAT_STATUS_OKAY(ISL9241_COMPAT)
+__maybe_unused static int isl9241_get_temp(const struct temp_sensor_t *sensor,
+						int *temp_ptr)
+{
+	return isl9241_get_temperature_val(sensor->idx, temp_ptr);
+}
+#endif /* BATTERY_COMPAT */
+
+#define GET_ZEPHYR_TEMP_SENSOR_ISL9241(named_id)                  \
+	(&(const struct zephyr_temp_sensor){                     \
+		.read = &isl9241_get_temp,                        \
+		.thermistor = NULL,                              \
+		.update_temperature = NULL, \
+		FILL_POWER_GOOD(named_id) })
+
+#define TEMP_ISL9241(named_id, sensor_id)                                \
+	[TEMP_SENSOR_ID(named_id)] = {                                  \
+		.name = DT_NODE_FULL_NAME(sensor_id),                   \
+		.idx = 0,                     \
+		.type = TEMP_SENSOR_TYPE_BOARD,                         \
+		.zephyr_info = GET_ZEPHYR_TEMP_SENSOR_ISL9241(named_id), \
+	}
+
+
 #ifdef CONFIG_PLATFORM_EC_CUSTOMIZED_DESIGN
 #if DT_HAS_COMPAT_STATUS_OKAY(AMDR23M_COMPAT)
 /* The function maybe unused because a temperature sensor can be added to dts
@@ -422,6 +448,7 @@ __maybe_unused static int peci_get_temp(const struct temp_sensor_t *sensor,
 	CHECK_COMPAT(F75303_COMPAT, named_id, sensor_id, TEMP_F75303)         \
 	CHECK_COMPAT(F75397_COMPAT, named_id, sensor_id, TEMP_F75397)         \
 	CHECK_COMPAT(BATTERY_COMPAT, named_id, sensor_id, TEMP_BATTERY)         \
+	CHECK_COMPAT(ISL9241_COMPAT, named_id, sensor_id, TEMP_ISL9241)         \
 	CHECK_COMPAT(AMDR23M_COMPAT, named_id, sensor_id, TEMP_AMDR23M)			\
 	CHECK_COMPAT(PECI_COMPAT, named_id, sensor_id, TEMP_PECI)