[coreboot-gerrit] Change in coreboot[master]: ec/google/wilco: add ec command set cpu id

[EricR Lai (Code Review)]

EricR Lai 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 5:

(3 comments)

https://review.coreboot.org/c/coreboot/+/38526/4/src/ec/google/wilco/command... 
File src/ec/google/wilco/commands.c:

https://review.coreboot.org/c/coreboot/+/38526/4/src/ec/google/wilco/command... 
PS4, Line 231:  * MBOX[3] = CPUID_Low
             :  * MBOX[4] = CPUID_Mid
             :  * MBOX[5] = CPUID_High
> Ack

Done

https://review.coreboot.org/c/coreboot/+/38526/4/src/ec/google/wilco/command... 
PS4, Line 237: */
> space before * looks like it would align the comment.  but this could be just a gerrit ui issue.

Done

https://review.coreboot.org/c/coreboot/+/38526/4/src/ec/google/wilco/command... 
PS4, Line 240: 	uint8_t cpu_id[7] = {0}, i;
> can you add a newline after the declaration and before the code?

Done