Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38526 )
Change subject: ec/google/wilco: add ec command set cpu id ......................................................................
Patch Set 3:
(4 comments)
https://review.coreboot.org/c/coreboot/+/38526/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38526/3//COMMIT_MSG@7 PS3, Line 7: add ec One space.
https://review.coreboot.org/c/coreboot/+/38526/3//COMMIT_MSG@10 PS3, Line 10: EC. Fits on one line.
https://review.coreboot.org/c/coreboot/+/38526/3//COMMIT_MSG@11 PS3, Line 11: What does the EC do with that?
https://review.coreboot.org/c/coreboot/+/38526/3/src/ec/google/wilco/command... File src/ec/google/wilco/commands.c:
https://review.coreboot.org/c/coreboot/+/38526/3/src/ec/google/wilco/command... PS3, Line 240: {0} Spaces around 0?