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

[Edward O'Callaghan (Code Review)]

Hello Tim Wawrzynczak,

I'd like you to do a code review. Please visit

https://review.coreboot.org/c/coreboot/+/37930

to review the following change.

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.

Change-Id: I4e683e5a1c5b453b3742a12a519cad9069e8b7f7
Signed-off-by: Edward O'Callaghan 
---
M src/ec/google/chromeec/ec.c
1 file changed, 8 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/30/37930/1

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(