Merge pull request #441 from FrameworkComputer/hx30.reduce_call_gettemp_function

Reduced the frequency to execute the PECI over eSPI transaction

Author: kiram9

board/hx30/peci_customization.c

@@ -6,6 +6,7 @@
 #include "chipset.h"
 #include "console.h"
 #include "board.h"
+#include "hooks.h"
 #include "host_command.h"
 #include "peci.h"
 #include "peci_customization.h"
@@ -16,6 +17,8 @@
 #define CPUTS(outstr) cputs(CC_THERMAL, outstr)
 #define CPRINTS(format, args...) cprints(CC_THERMAL, format, ## args)
 
+static int peci_temp;
+
 /*****************************************************************************/
 /* Internal functions */
 
@@ -250,22 +253,36 @@ __override int stop_read_peci_temp(void)
 
 int peci_over_espi_temp_sensor_get_val(int idx, int *temp_ptr)
 {
-	int i, rv;
+	if (peci_temp == 0xfffe)
+		return EC_ERROR_NOT_POWERED;
+
+	if (peci_temp == 0xffff)
+		return EC_ERROR_INVAL;
+
+	*temp_ptr = peci_temp;
+
+	return EC_SUCCESS;
+}
+
+void read_peci_over_espi_gettemp(void)
+{
+	int rv;
+	int i;
 
 	rv = stop_read_peci_temp();
 
-	if (rv != EC_SUCCESS)
-		return rv;
+	if (rv != EC_SUCCESS) {
+		peci_temp = 0xfffe;
+	} else {
+		for (i = 0; i < 2; i++) {
+			rv = peci_over_espi_get_cpu_temp(&peci_temp);
+			if (!rv)
+				break;
+			msleep(10);
+		}
 
-	/*
-	 * Retry reading PECI CPU temperature if the first sample is
-	 * invalid or failed to obtain.
-	 */
-	for (i = 0; i < 2; i++) {
-		rv = peci_over_espi_get_cpu_temp(temp_ptr);
-		if (!rv)
-			break;
+		if (rv != EC_SUCCESS)
+			peci_temp = 0xffff;
 	}
-
-	return rv;
 }
+DECLARE_HOOK(HOOK_SECOND, read_peci_over_espi_gettemp, HOOK_PRIO_DEFAULT);

board/hx30/peci_customization.h

@@ -60,4 +60,13 @@ int peci_update_PL1(int watt);
 int peci_update_PL2(int watt);
 int peci_update_PL4(int watt);
 int peci_update_PsysPL2(int watt);
+
+/**
+ * This function return the peci gettemp value from global variant, the
+ * actually value is read from read_peci_over_espi_gettemp();
+ *
+ * @param idx		no used
+ * @param temp_ptr	return temp value pointer
+ * @return int		return get value status
+ */
 int peci_over_espi_temp_sensor_get_val(int idx, int *temp_ptr);