From 6046739b9d349d43f6d064660aaf9df49a99cc5f Mon Sep 17 00:00:00 2001 From: Tim Wawrzynczak Date: Mon, 18 May 2020 13:45:43 -0600 Subject: ec/google/chromeec: Add new *-switch properties to USBC.CONx devices The Linux ChromeOS EC driver now looks for 3 new properties under each USBC.CONx device contained within the ChromeOS EC device. These properties are just a reference to the device that controls the switches for USB 2/3 muxing, SBU lines, and CC lines. It uses the new function, soc_get_pmc_mux_device() to retrieve the device. Change-Id: I03cd83f9b2901b5583053fac8ab6eab64717a07d Signed-off-by: Tim Wawrzynczak Reviewed-on: https://review.coreboot.org/c/coreboot/+/40618 Reviewed-by: Duncan Laurie Reviewed-by: Aaron Durbin Tested-by: build bot (Jenkins) --- src/ec/google/chromeec/ec_acpi.c | 33 +++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+) (limited to 'src/ec/google') diff --git a/src/ec/google/chromeec/ec_acpi.c b/src/ec/google/chromeec/ec_acpi.c index 7b859e95af..d87a49c88d 100644 --- a/src/ec/google/chromeec/ec_acpi.c +++ b/src/ec/google/chromeec/ec_acpi.c @@ -19,6 +19,14 @@ #define GOOGLE_CHROMEEC_USBC_DEVICE_HID "GOOG0014" #define GOOGLE_CHROMEEC_USBC_DEVICE_NAME "USBC" +/* Avoid adding a false dependency on an SoC or intel/common */ +extern const struct device *soc_get_pmc_mux_device(int port_number); + +__weak const struct device *soc_get_pmc_mux_device(int port_number) +{ + return NULL; +} + const char *google_chromeec_acpi_name(const struct device *dev) { /* @@ -186,6 +194,30 @@ static void add_usb_port_references(struct acpi_dp *dsd, int port_number) } } +/* + * Another helper for fill_ssdt_typec_device(). For each port, this one adds references to the + * ACPI device which control the orientation, USB data role and data muxing. + */ +static void add_switch_references(struct acpi_dp *dsd, int port_number) +{ + const struct device *dev; + const char *path; + + dev = soc_get_pmc_mux_device(port_number); + if (!dev) { + printk(BIOS_ERR, "ERROR: %s: No SOC PMC MUX device found", __func__); + return; + } + + path = acpi_device_path(dev); + if (!path) + return; + + acpi_dp_add_reference(dsd, "orientation-switch", path); + acpi_dp_add_reference(dsd, "usb-role-switch", path); + acpi_dp_add_reference(dsd, "mode-switch", path); +} + static void fill_ssdt_typec_device(const struct device *dev) { struct usb_pd_port_caps port_caps; @@ -218,6 +250,7 @@ static void fill_ssdt_typec_device(const struct device *dev) acpi_dp_add_integer(dsd, "port-number", i); add_port_caps(dsd, &port_caps); add_usb_port_references(dsd, i); + add_switch_references(dsd, i); acpi_dp_write(dsd); acpigen_pop_len(); /* Device CONx */ -- cgit v1.2.3