Reduced the frequency to execute the PECI over eSPI transaction

We observed there are three functions that execute the PECI over the eSPI transaction.
If one transaction is fail, EC will retry the transaction for a short time
and this phenomenon will cause the below issue:

1. System warm boot hang
2. Type-C port loss function
3. System power on the first time fail (after removing coin battery, batt, and ac)
4. stress test fail (power on too slow)

We read the peci gettemp via hook_second and store the value in the global variant,
and bypass the variant value to other functions

Signed-off-by: Josh Tsai <josh_tsai@compal.corp-partner.google.com>

Author: Josh Tsai

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);