[coreboot-gerrit] Change in coreboot[master]: google/chromeec: Add a voltage parameter for getting info from USB PD

[Tim Wawrzynczak (Code Review)]

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

Change subject: google/chromeec: Add a voltage parameter for getting info from USB PD
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/39849/3/src/ec/google/chromeec/ec.c 
File src/ec/google/chromeec/ec.c:

https://review.coreboot.org/c/coreboot/+/39849/3/src/ec/google/chromeec/ec.c... 
PS3, Line 1131: t *max_watts, uint16_t *voltage_max
> How about return all three data (power, voltage and current)? Otherwise, I have to find a fizz to en […]

What's the purpose of returning I and V separately? Do the PL calculations depend on the voltage or do they depend on power? It seems disingenuous to return I, V, and P, if you weren't certain that I*V=P. Otherwise, what do the different values really mean?