Attention is currently required from: Caveh Jalali, Forest Mittelberg.
Karthik Ramasubramanian has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/84936?usp=email )
Change subject: ec/google/chromeec: Add API to get PD Chip info ......................................................................
ec/google/chromeec: Add API to get PD Chip info
Add API to get Power Delivery (PD) Chip info which includes vendor ID, product ID and firmware version(if any).
BUG=None TEST=Build Brox BIOS image and boot to OS.
Change-Id: I4cc4493ac64d44076877fee633488c95cd09807e Signed-off-by: Karthikeyan Ramasubramanian kramasub@google.com --- M src/ec/google/chromeec/ec.c M src/ec/google/chromeec/ec.h 2 files changed, 23 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/36/84936/1
diff --git a/src/ec/google/chromeec/ec.c b/src/ec/google/chromeec/ec.c index 088f2fe..b47750e 100644 --- a/src/ec/google/chromeec/ec.c +++ b/src/ec/google/chromeec/ec.c @@ -1170,6 +1170,17 @@ return 0; }
+int google_chromeec_get_pd_chip_infoi(int port, int renew, + struct ec_response_pd_chip_info *r) +{ + const struct ec_params_pd_chip_info p = { + .port = port, + .live = renew, + }; + + return ec_cmd_pd_chip_info(PLAT_EC, &p, r); +} + int google_chromeec_get_pd_port_caps(int port, struct usb_pd_port_caps *port_caps) { diff --git a/src/ec/google/chromeec/ec.h b/src/ec/google/chromeec/ec.h index 0062df6..929e392 100644 --- a/src/ec/google/chromeec/ec.h +++ b/src/ec/google/chromeec/ec.h @@ -337,6 +337,18 @@ */ int google_chromeec_get_num_pd_ports(unsigned int *num_ports);
+/** + * Return a port's PD chip information. + * + * @param port The desired port number + * @param renew Refresh cached value + * @param r Result buffer for chip info + * + * @return 0 if ok, -1 on error + */ +int google_chromeec_get_pd_chip_infoi(int port, int renew, + struct ec_response_pd_chip_info *r); + /* Structure representing the capabilities of a USB-PD port */ struct usb_pd_port_caps { enum ec_pd_power_role_caps power_role_cap;