View Change

1 comment:

• File src/ec/google/chromeec/ec.h:

  • Patch Set #1, 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.

To view, visit change 31626. To unsubscribe, or for help writing mail filters, visit settings.