[coreboot-gerrit] Change in ...coreboot[master]: ec/google/chromeec: fix the error status handling

[YH Lin (Code Review)]

Hello Daisuke Nojiri, Hung-Te Lin, build bot (Jenkins), Patrick Georgi,

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#3).

Change subject: ec/google/chromeec: fix the error status handling
......................................................................

ec/google/chromeec: fix the error status handling

Various instances of google_chromeec_command() can return non-zero number
(both positive and negative) to indicate error, which causes confusion
for the caller. This patch attempts to address the issue by using only
negative number for errors.

In addition, the patch modifies the callers of google_chromeec_command()
such so they will check negative number as error.

BUG=b:123676982,chromium:935038
BRANCH=kukui
TEST=build with kukui/flapjack configurations
Signed-off-by: YH Lin 
Change-Id: I7f0a8a61d01d942cba57036a17dd527fdbbf940c
---
M src/ec/google/chromeec/crosec_proto.c
M src/ec/google/chromeec/ec.c
M src/ec/google/chromeec/ec_i2c.c
M src/ec/google/chromeec/ec_lpc.c
M src/ec/google/chromeec/vstore.c
5 files changed, 21 insertions(+), 21 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/85/31585/3