[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 3:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/42078/2/src/ec/google/chromeec/ec.c... 
PS2, Line 1531: 	uint8_t num_ports;
              : 	int ret;
              : 	struct chromeec_command cmd;
              : 	int i;
              : 
              : 	ret = google_chromeec_get_num_pd_ports(&num_ports);
              : 	if (ret < 0)
              : 		return -1;
              : 
              : 	for (i = 0; i < num_ports; i++)
> Any specific reason why? the function was added here so only logical to me that it is propagated acr […]

google_chromeec_pd_get_amode() alreacdy existed, so changing it from sending a "raw" command to calling google_chromeec_get_num_pd_ports() is worth its own commit IMO