Tim Wawrzynczak has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43090 )
Change subject: drivers/intel/pmc_mux: Rename con driver to conn ......................................................................
drivers/intel/pmc_mux: Rename con driver to conn
For historical reasons, Windows has issues with certain names being used for files and directories, 'con' or 'CON' being one of them. Therefore, rename the pmc_mux/con driver to pmc_mux/conn in order to work around this issue.
TEST=built volteer (only user of this driver as of now)
Signed-off-by: Tim Wawrzynczak twawrzynczak@chromium.org Change-Id: Ia78dc4efe647c96a7169a3b95fc3b8944d052c83 --- M src/drivers/intel/pmc_mux/Makefile.inc D src/drivers/intel/pmc_mux/con/Makefile.inc A src/drivers/intel/pmc_mux/conn/Makefile.inc R src/drivers/intel/pmc_mux/conn/chip.h R src/drivers/intel/pmc_mux/conn/conn.c M src/ec/google/chromeec/ec_acpi.c M src/mainboard/google/volteer/variants/volteer/overridetree.cb M src/mainboard/google/volteer/variants/volteer2/overridetree.cb 8 files changed, 32 insertions(+), 32 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/90/43090/1
diff --git a/src/drivers/intel/pmc_mux/Makefile.inc b/src/drivers/intel/pmc_mux/Makefile.inc index f27f014..61c957e 100644 --- a/src/drivers/intel/pmc_mux/Makefile.inc +++ b/src/drivers/intel/pmc_mux/Makefile.inc @@ -1,2 +1,2 @@ -subdirs-y += con +subdirs-y += conn ramstage-$(CONFIG_DRIVERS_INTEL_PMC) += mux.c diff --git a/src/drivers/intel/pmc_mux/con/Makefile.inc b/src/drivers/intel/pmc_mux/con/Makefile.inc deleted file mode 100644 index 213e8fc..0000000 --- a/src/drivers/intel/pmc_mux/con/Makefile.inc +++ /dev/null @@ -1 +0,0 @@ -ramstage-$(CONFIG_DRIVERS_INTEL_PMC) += con.c diff --git a/src/drivers/intel/pmc_mux/conn/Makefile.inc b/src/drivers/intel/pmc_mux/conn/Makefile.inc new file mode 100644 index 0000000..babca2f --- /dev/null +++ b/src/drivers/intel/pmc_mux/conn/Makefile.inc @@ -0,0 +1 @@ +ramstage-$(CONFIG_DRIVERS_INTEL_PMC) += conn.c diff --git a/src/drivers/intel/pmc_mux/con/chip.h b/src/drivers/intel/pmc_mux/conn/chip.h similarity index 80% rename from src/drivers/intel/pmc_mux/con/chip.h rename to src/drivers/intel/pmc_mux/conn/chip.h index 02d018a..8497350 100644 --- a/src/drivers/intel/pmc_mux/con/chip.h +++ b/src/drivers/intel/pmc_mux/conn/chip.h @@ -1,7 +1,7 @@ /* SPDX-License-Identifier: GPL-2.0-or-later */
-#ifndef __DRIVERS_INTEL_PMC_MUX_CON_H__ -#define __DRIVERS_INTEL_PMC_MUX_CON_H__ +#ifndef __DRIVERS_INTEL_PMC_MUX_CONN_H__ +#define __DRIVERS_INTEL_PMC_MUX_CONN_H__
enum typec_orientation { /* The orientation of the signal follows the orientation of the CC lines. */ @@ -12,7 +12,7 @@ TYPEC_ORIENTATION_REVERSE, };
-struct drivers_intel_pmc_mux_con_config { +struct drivers_intel_pmc_mux_conn_config { /* 1-based port numbers (from SoC point of view) */ int usb2_port_number; /* 1-based port numbers (from SoC point of view) */ @@ -23,4 +23,4 @@ enum typec_orientation hsl_orientation; };
-#endif /* __DRIVERS_INTEL_PMC_MUX_CON_H__ */ +#endif /* __DRIVERS_INTEL_PMC_MUX_CONN_H__ */ diff --git a/src/drivers/intel/pmc_mux/con/con.c b/src/drivers/intel/pmc_mux/conn/conn.c similarity index 78% rename from src/drivers/intel/pmc_mux/con/con.c rename to src/drivers/intel/pmc_mux/conn/conn.c index ad95905..16d113b 100644 --- a/src/drivers/intel/pmc_mux/con/con.c +++ b/src/drivers/intel/pmc_mux/conn/conn.c @@ -5,7 +5,7 @@ #include <intelblocks/acpi.h> #include "chip.h"
-static const char *con_acpi_name(const struct device *dev) +static const char *conn_acpi_name(const struct device *dev) { static char name[5]; snprintf(name, sizeof(name), "CON%1X", dev->path.generic.id); @@ -25,9 +25,9 @@ } }
-static void con_fill_ssdt(const struct device *dev) +static void conn_fill_ssdt(const struct device *dev) { - struct drivers_intel_pmc_mux_con_config *config = dev->chip_info; + struct drivers_intel_pmc_mux_conn_config *config = dev->chip_info; struct acpi_dp *dsd; const char *scope; const char *name; @@ -72,19 +72,19 @@ dev_path(dev)); }
-static struct device_operations con_dev_ops = { +static struct device_operations conn_dev_ops = { .read_resources = noop_read_resources, .set_resources = noop_set_resources, - .acpi_name = con_acpi_name, - .acpi_fill_ssdt = con_fill_ssdt, + .acpi_name = conn_acpi_name, + .acpi_fill_ssdt = conn_fill_ssdt, };
-static void con_enable(struct device *dev) +static void conn_enable(struct device *dev) { - dev->ops = &con_dev_ops; + dev->ops = &conn_dev_ops; }
-struct chip_operations drivers_intel_pmc_mux_con_ops = { - CHIP_NAME("Intel PMC MUX CON Driver") - .enable_dev = con_enable, +struct chip_operations drivers_intel_pmc_mux_conn_ops = { + CHIP_NAME("Intel PMC MUX CONN Driver") + .enable_dev = conn_enable, }; diff --git a/src/ec/google/chromeec/ec_acpi.c b/src/ec/google/chromeec/ec_acpi.c index 8c80382..dca5285 100644 --- a/src/ec/google/chromeec/ec_acpi.c +++ b/src/ec/google/chromeec/ec_acpi.c @@ -124,17 +124,17 @@ port_location_to_str(port_caps.port_location)); }
-static int con_id_to_match; +static int conn_id_to_match;
/* A callback to match a port's connector for dev_find_matching_device_on_bus */ static bool match_connector(DEVTREE_CONST struct device *dev) { if (CONFIG(DRIVERS_INTEL_PMC)) { - extern struct chip_operations drivers_intel_pmc_mux_con_ops; + extern struct chip_operations drivers_intel_pmc_mux_conn_ops;
- return (dev->chip_ops == &drivers_intel_pmc_mux_con_ops && + return (dev->chip_ops == &drivers_intel_pmc_mux_conn_ops && dev->path.type == DEVICE_PATH_GENERIC && - dev->path.generic.id == con_id_to_match); + dev->path.generic.id == conn_id_to_match); }
return false; @@ -148,7 +148,7 @@ struct device *usb3_port; struct device *usb4_port; const struct device *mux; - const struct device *con; + const struct device *conn;
if (google_chromeec_get_num_pd_ports(&num_ports)) return; @@ -165,11 +165,11 @@ continue;
/* Get the MUX device, and find the matching connector on its bus */ - con = NULL; + conn = NULL; mux = soc_get_pmc_mux_device(i); if (mux) { - con_id_to_match = i; - con = dev_find_matching_device_on_bus(mux->link_list, match_connector); + conn_id_to_match = i; + conn = dev_find_matching_device_on_bus(mux->link_list, match_connector); }
usb2_port = NULL; @@ -184,9 +184,9 @@ .usb2_port = usb2_port, .usb3_port = usb3_port, .usb4_port = usb4_port, - .orientation_switch = con, - .usb_role_switch = con, - .mode_switch = con, + .orientation_switch = conn, + .usb_role_switch = conn, + .mode_switch = conn, };
acpigen_write_typec_connector(&config, i, add_port_location); diff --git a/src/mainboard/google/volteer/variants/volteer/overridetree.cb b/src/mainboard/google/volteer/variants/volteer/overridetree.cb index 24b869c..c32b80e 100644 --- a/src/mainboard/google/volteer/variants/volteer/overridetree.cb +++ b/src/mainboard/google/volteer/variants/volteer/overridetree.cb @@ -145,13 +145,13 @@ # PMC.MUX device in the ACPI hierarchy. chip drivers/intel/pmc_mux device generic 0 on - chip drivers/intel/pmc_mux/con + chip drivers/intel/pmc_mux/conn register "usb2_port_number" = "9" register "usb3_port_number" = "1" # SBU & HSL follow CC device generic 0 on end end - chip drivers/intel/pmc_mux/con + chip drivers/intel/pmc_mux/conn register "usb2_port_number" = "4" register "usb3_port_number" = "2" # SBU is fixed, HSL follows CC diff --git a/src/mainboard/google/volteer/variants/volteer2/overridetree.cb b/src/mainboard/google/volteer/variants/volteer2/overridetree.cb index 24b869c..c32b80e 100644 --- a/src/mainboard/google/volteer/variants/volteer2/overridetree.cb +++ b/src/mainboard/google/volteer/variants/volteer2/overridetree.cb @@ -145,13 +145,13 @@ # PMC.MUX device in the ACPI hierarchy. chip drivers/intel/pmc_mux device generic 0 on - chip drivers/intel/pmc_mux/con + chip drivers/intel/pmc_mux/conn register "usb2_port_number" = "9" register "usb3_port_number" = "1" # SBU & HSL follow CC device generic 0 on end end - chip drivers/intel/pmc_mux/con + chip drivers/intel/pmc_mux/conn register "usb2_port_number" = "4" register "usb3_port_number" = "2" # SBU is fixed, HSL follows CC
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43090 )
Change subject: drivers/intel/pmc_mux: Rename con driver to conn ......................................................................
Patch Set 1: Code-Review+2
for those who may not know, it fails because CON is an alias for the CONsole
Caveh Jalali has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43090 )
Change subject: drivers/intel/pmc_mux: Rename con driver to conn ......................................................................
Patch Set 1: Code-Review+2
Tim Wawrzynczak has submitted this change. ( https://review.coreboot.org/c/coreboot/+/43090 )
Change subject: drivers/intel/pmc_mux: Rename con driver to conn ......................................................................
drivers/intel/pmc_mux: Rename con driver to conn
For historical reasons, Windows has issues with certain names being used for files and directories, 'con' or 'CON' being one of them. Therefore, rename the pmc_mux/con driver to pmc_mux/conn in order to work around this issue.
TEST=built volteer (only user of this driver as of now)
Signed-off-by: Tim Wawrzynczak twawrzynczak@chromium.org Change-Id: Ia78dc4efe647c96a7169a3b95fc3b8944d052c83 Reviewed-on: https://review.coreboot.org/c/coreboot/+/43090 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Angel Pons th3fanbus@gmail.com Reviewed-by: Caveh Jalali caveh@chromium.org --- M src/drivers/intel/pmc_mux/Makefile.inc D src/drivers/intel/pmc_mux/con/Makefile.inc A src/drivers/intel/pmc_mux/conn/Makefile.inc R src/drivers/intel/pmc_mux/conn/chip.h R src/drivers/intel/pmc_mux/conn/conn.c M src/ec/google/chromeec/ec_acpi.c M src/mainboard/google/volteer/variants/volteer/overridetree.cb M src/mainboard/google/volteer/variants/volteer2/overridetree.cb 8 files changed, 32 insertions(+), 32 deletions(-)
Approvals: build bot (Jenkins): Verified Caveh Jalali: Looks good to me, approved Angel Pons: Looks good to me, approved
diff --git a/src/drivers/intel/pmc_mux/Makefile.inc b/src/drivers/intel/pmc_mux/Makefile.inc index f27f014..61c957e 100644 --- a/src/drivers/intel/pmc_mux/Makefile.inc +++ b/src/drivers/intel/pmc_mux/Makefile.inc @@ -1,2 +1,2 @@ -subdirs-y += con +subdirs-y += conn ramstage-$(CONFIG_DRIVERS_INTEL_PMC) += mux.c diff --git a/src/drivers/intel/pmc_mux/con/Makefile.inc b/src/drivers/intel/pmc_mux/con/Makefile.inc deleted file mode 100644 index 213e8fc..0000000 --- a/src/drivers/intel/pmc_mux/con/Makefile.inc +++ /dev/null @@ -1 +0,0 @@ -ramstage-$(CONFIG_DRIVERS_INTEL_PMC) += con.c diff --git a/src/drivers/intel/pmc_mux/conn/Makefile.inc b/src/drivers/intel/pmc_mux/conn/Makefile.inc new file mode 100644 index 0000000..babca2f --- /dev/null +++ b/src/drivers/intel/pmc_mux/conn/Makefile.inc @@ -0,0 +1 @@ +ramstage-$(CONFIG_DRIVERS_INTEL_PMC) += conn.c diff --git a/src/drivers/intel/pmc_mux/con/chip.h b/src/drivers/intel/pmc_mux/conn/chip.h similarity index 80% rename from src/drivers/intel/pmc_mux/con/chip.h rename to src/drivers/intel/pmc_mux/conn/chip.h index 02d018a..8497350 100644 --- a/src/drivers/intel/pmc_mux/con/chip.h +++ b/src/drivers/intel/pmc_mux/conn/chip.h @@ -1,7 +1,7 @@ /* SPDX-License-Identifier: GPL-2.0-or-later */
-#ifndef __DRIVERS_INTEL_PMC_MUX_CON_H__ -#define __DRIVERS_INTEL_PMC_MUX_CON_H__ +#ifndef __DRIVERS_INTEL_PMC_MUX_CONN_H__ +#define __DRIVERS_INTEL_PMC_MUX_CONN_H__
enum typec_orientation { /* The orientation of the signal follows the orientation of the CC lines. */ @@ -12,7 +12,7 @@ TYPEC_ORIENTATION_REVERSE, };
-struct drivers_intel_pmc_mux_con_config { +struct drivers_intel_pmc_mux_conn_config { /* 1-based port numbers (from SoC point of view) */ int usb2_port_number; /* 1-based port numbers (from SoC point of view) */ @@ -23,4 +23,4 @@ enum typec_orientation hsl_orientation; };
-#endif /* __DRIVERS_INTEL_PMC_MUX_CON_H__ */ +#endif /* __DRIVERS_INTEL_PMC_MUX_CONN_H__ */ diff --git a/src/drivers/intel/pmc_mux/con/con.c b/src/drivers/intel/pmc_mux/conn/conn.c similarity index 78% rename from src/drivers/intel/pmc_mux/con/con.c rename to src/drivers/intel/pmc_mux/conn/conn.c index ad95905..16d113b 100644 --- a/src/drivers/intel/pmc_mux/con/con.c +++ b/src/drivers/intel/pmc_mux/conn/conn.c @@ -5,7 +5,7 @@ #include <intelblocks/acpi.h> #include "chip.h"
-static const char *con_acpi_name(const struct device *dev) +static const char *conn_acpi_name(const struct device *dev) { static char name[5]; snprintf(name, sizeof(name), "CON%1X", dev->path.generic.id); @@ -25,9 +25,9 @@ } }
-static void con_fill_ssdt(const struct device *dev) +static void conn_fill_ssdt(const struct device *dev) { - struct drivers_intel_pmc_mux_con_config *config = dev->chip_info; + struct drivers_intel_pmc_mux_conn_config *config = dev->chip_info; struct acpi_dp *dsd; const char *scope; const char *name; @@ -72,19 +72,19 @@ dev_path(dev)); }
-static struct device_operations con_dev_ops = { +static struct device_operations conn_dev_ops = { .read_resources = noop_read_resources, .set_resources = noop_set_resources, - .acpi_name = con_acpi_name, - .acpi_fill_ssdt = con_fill_ssdt, + .acpi_name = conn_acpi_name, + .acpi_fill_ssdt = conn_fill_ssdt, };
-static void con_enable(struct device *dev) +static void conn_enable(struct device *dev) { - dev->ops = &con_dev_ops; + dev->ops = &conn_dev_ops; }
-struct chip_operations drivers_intel_pmc_mux_con_ops = { - CHIP_NAME("Intel PMC MUX CON Driver") - .enable_dev = con_enable, +struct chip_operations drivers_intel_pmc_mux_conn_ops = { + CHIP_NAME("Intel PMC MUX CONN Driver") + .enable_dev = conn_enable, }; diff --git a/src/ec/google/chromeec/ec_acpi.c b/src/ec/google/chromeec/ec_acpi.c index 8c80382..dca5285 100644 --- a/src/ec/google/chromeec/ec_acpi.c +++ b/src/ec/google/chromeec/ec_acpi.c @@ -124,17 +124,17 @@ port_location_to_str(port_caps.port_location)); }
-static int con_id_to_match; +static int conn_id_to_match;
/* A callback to match a port's connector for dev_find_matching_device_on_bus */ static bool match_connector(DEVTREE_CONST struct device *dev) { if (CONFIG(DRIVERS_INTEL_PMC)) { - extern struct chip_operations drivers_intel_pmc_mux_con_ops; + extern struct chip_operations drivers_intel_pmc_mux_conn_ops;
- return (dev->chip_ops == &drivers_intel_pmc_mux_con_ops && + return (dev->chip_ops == &drivers_intel_pmc_mux_conn_ops && dev->path.type == DEVICE_PATH_GENERIC && - dev->path.generic.id == con_id_to_match); + dev->path.generic.id == conn_id_to_match); }
return false; @@ -148,7 +148,7 @@ struct device *usb3_port; struct device *usb4_port; const struct device *mux; - const struct device *con; + const struct device *conn;
if (google_chromeec_get_num_pd_ports(&num_ports)) return; @@ -165,11 +165,11 @@ continue;
/* Get the MUX device, and find the matching connector on its bus */ - con = NULL; + conn = NULL; mux = soc_get_pmc_mux_device(i); if (mux) { - con_id_to_match = i; - con = dev_find_matching_device_on_bus(mux->link_list, match_connector); + conn_id_to_match = i; + conn = dev_find_matching_device_on_bus(mux->link_list, match_connector); }
usb2_port = NULL; @@ -184,9 +184,9 @@ .usb2_port = usb2_port, .usb3_port = usb3_port, .usb4_port = usb4_port, - .orientation_switch = con, - .usb_role_switch = con, - .mode_switch = con, + .orientation_switch = conn, + .usb_role_switch = conn, + .mode_switch = conn, };
acpigen_write_typec_connector(&config, i, add_port_location); diff --git a/src/mainboard/google/volteer/variants/volteer/overridetree.cb b/src/mainboard/google/volteer/variants/volteer/overridetree.cb index 24b869c..c32b80e 100644 --- a/src/mainboard/google/volteer/variants/volteer/overridetree.cb +++ b/src/mainboard/google/volteer/variants/volteer/overridetree.cb @@ -145,13 +145,13 @@ # PMC.MUX device in the ACPI hierarchy. chip drivers/intel/pmc_mux device generic 0 on - chip drivers/intel/pmc_mux/con + chip drivers/intel/pmc_mux/conn register "usb2_port_number" = "9" register "usb3_port_number" = "1" # SBU & HSL follow CC device generic 0 on end end - chip drivers/intel/pmc_mux/con + chip drivers/intel/pmc_mux/conn register "usb2_port_number" = "4" register "usb3_port_number" = "2" # SBU is fixed, HSL follows CC diff --git a/src/mainboard/google/volteer/variants/volteer2/overridetree.cb b/src/mainboard/google/volteer/variants/volteer2/overridetree.cb index 24b869c..c32b80e 100644 --- a/src/mainboard/google/volteer/variants/volteer2/overridetree.cb +++ b/src/mainboard/google/volteer/variants/volteer2/overridetree.cb @@ -145,13 +145,13 @@ # PMC.MUX device in the ACPI hierarchy. chip drivers/intel/pmc_mux device generic 0 on - chip drivers/intel/pmc_mux/con + chip drivers/intel/pmc_mux/conn register "usb2_port_number" = "9" register "usb3_port_number" = "1" # SBU & HSL follow CC device generic 0 on end end - chip drivers/intel/pmc_mux/con + chip drivers/intel/pmc_mux/conn register "usb2_port_number" = "4" register "usb3_port_number" = "2" # SBU is fixed, HSL follows CC
Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43090 )
Change subject: drivers/intel/pmc_mux: Rename con driver to conn ......................................................................
Patch Set 2: Code-Review+2
-
Angel Pons (Code Review) -
Caveh Jalali (Code Review)
-
Frans Hendriks (Code Review)
-
Tim Wawrzynczak (Code Review)