[coreboot-gerrit] Change in coreboot[master]: ec/google: Fix wedging AP on early ec sw sync

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/37930 )

Change subject: ec/google: Fix wedging AP on early ec sw sync
......................................................................

ec/google: Fix wedging AP on early ec sw sync

If the EC doesn't support the EARLY_EC_SYNC we don't properly set power
limits to reasonable defaults and can wedge the AP by browning out at
the end of vboot.

BRANCH=none
BUG=b:146165519
TEST=./util/abuild/abuild -p none -t google/hatch -x -a

Change-Id: I4e683e5a1c5b453b3742a12a519cad9069e8b7f7
Signed-off-by: Edward O'Callaghan 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/37930
Reviewed-by: Tim Wawrzynczak 
Tested-by: build bot (Jenkins) 
---
M src/ec/google/chromeec/ec.c
1 file changed, 8 insertions(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Tim Wawrzynczak: Looks good to me, approved

diff --git a/src/ec/google/chromeec/ec.c b/src/ec/google/chromeec/ec.c
index 5dff162..de43eb5 100644
--- a/src/ec/google/chromeec/ec.c
+++ b/src/ec/google/chromeec/ec.c
@@ -757,12 +757,17 @@
    	.cmd_data_out = &resp,
    	.cmd_dev_index = 0,
    };
+	int rv;

-	if (google_chromeec_command(&cmd))
-		return -1;
+	rv = google_chromeec_command(&cmd);
+	if (rv == -EC_RES_INVALID_PARAM || rv == -EC_RES_INVALID_COMMAND) {
+		printk(BIOS_INFO, "PARAM_LIMIT_POWER not supported by EC.\n");
+		*limit_power = 0;
+		return 0;
+	}

*limit_power = resp.get_param.value;
-	return 0;
+	return rv;
 }

int google_chromeec_get_protocol_info(