[coreboot-gerrit] Change in coreboot[master]: src/ec/google/chromeec: Get Type-C Mux info from EC (TCPM)

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42078 )

Change subject: src/ec/google/chromeec: Get Type-C Mux info from EC (TCPM)
......................................................................

Patch Set 11:

(2 comments)

https://review.coreboot.org/c/coreboot/+/42078/11/src/ec/google/chromeec/ec.... 
File src/ec/google/chromeec/ec.c:

https://review.coreboot.org/c/coreboot/+/42078/11/src/ec/google/chromeec/ec.... 
PS11, Line 1505: int
how about making this unsigned too? then the check on line 1521 can go away

https://review.coreboot.org/c/coreboot/+/42078/11/src/ec/google/chromeec/ec_... 
File src/ec/google/chromeec/ec_acpi.c:

PS11: 
The change in this unrelated file is why I think a separate change, underneath this, that has your changes to google_chromeec_get_num_pd_ports(), would be preferable.