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

[Daisuke Nojiri (Code Review)]

Daisuke Nojiri 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 2:

(1 comment)

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.
> Yes, updating the comment seems fine.

Sorry it was ambiguous. I was referring to this entire patch. Do we have a plan to use positive integers for something else?

As long as we know we won't break any board, it's ok to do this change now (sparing positive integers for future use) but currently the risk isn't worth.