[coreboot-gerrit] Change in coreboot[master]: ec/google/chromeec: ignore LIMIT_POWER based on command code in response

[Rizwan Qureshi (Code Review)]

Rizwan Qureshi has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37947 )

Change subject: ec/google/chromeec: ignore LIMIT_POWER based on command code in response
......................................................................

ec/google/chromeec: ignore LIMIT_POWER based on command code in response

Assume that LIMIT_POWER is not requested if the ec does not support it.
Do this by checking the command code in the response message instead
of return value.

BUG=b:146165519
BRANCH=None
TEST=Boot puff with EC which does not support LIMIT_POWER param.

Change-Id: Ib2f5f69a53f204acebfab3e36aab2960eeec1204
Signed-off-by: Rizwan Qureshi 
---
M src/ec/google/chromeec/ec.c
1 file changed, 10 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/37947/1

diff --git a/src/ec/google/chromeec/ec.c b/src/ec/google/chromeec/ec.c
index de43eb5..02b26a0 100644
--- a/src/ec/google/chromeec/ec.c
+++ b/src/ec/google/chromeec/ec.c
@@ -760,14 +760,19 @@
    int rv;

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;
+
+	if (rv != 0 && (cmd.cmd_code == EC_RES_INVALID_COMMAND ||
+				cmd.cmd_code == EC_RES_INVALID_PARAM)) {
+                printk(BIOS_INFO, "PARAM_LIMIT_POWER not supported by EC.\n");
+                *limit_power = 0;
+                return 0;
+        } else if (rv != 0) {
+		return -1;
    }

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

int google_chromeec_get_protocol_info(