[coreboot-gerrit] Change in ...coreboot[master]: google_chromeec_command: makes the error status consistent

[YH Lin (Code Review)]

YH Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31626 )

Change subject: google_chromeec_command: makes the error status consistent
......................................................................

Patch Set 1:

(1 comment)

Thanks for the comments. Keeping the check as they are while returning "-1" seems to be acceptable by all it seems?

https://review.coreboot.org/#/c/31626/1/src/ec/google/chromeec/ec.h 
File src/ec/google/chromeec/ec.h:

https://review.coreboot.org/#/c/31626/1/src/ec/google/chromeec/ec.h@164 
PS1, Line 164: @return 0 on success or negative integer for errors.
> Currently, != 0 works and it's consistent. […]

Yes, the caller can definitely use "!= 0" which still conforms to this comment. But the comment itself needs to be accurate or else it would be misleading?