summaryrefslogtreecommitdiff
path: root/common/usbc/usb_mode.c
diff options
context:
space:
mode:
authorAbe Levkoy <alevkoy@chromium.org>2021-08-27 15:21:39 -0600
committerCommit Bot <commit-bot@chromium.org>2021-08-31 18:18:20 +0000
commit003ec08f8c67845ae8481d3fb4124555213b04c4 (patch)
tree3502c59532731d0003532657b1a2993e8ca9ad81 /common/usbc/usb_mode.c
parent5fa669005c3779b7310bf6bfe43f741a3d8ae871 (diff)
downloadchrome-ec-003ec08f8c67845ae8481d3fb4124555213b04c4.tar.gz
TCPM: Rename enum tcpm_sop_type and its constants
As a followup to CL:3104290, give the TCPCI TRANSMIT and RX_BUF_FRAME_TYPE types more consistent names. Most of them can be used for receiving, not just transmitting. Fix lint errors thus revealed. BUG=b:155476419 TEST=make buildall BRANCH=none Signed-off-by: Abe Levkoy <alevkoy@chromium.org> Change-Id: I399ec479eacc18622fc4d3f55f8bdabf4560fcff Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3125995 Reviewed-by: Keith Short <keithshort@chromium.org>
Diffstat (limited to 'common/usbc/usb_mode.c')
-rw-r--r--common/usbc/usb_mode.c36
1 files changed, 18 insertions, 18 deletions
diff --git a/common/usbc/usb_mode.c b/common/usbc/usb_mode.c
index 3829d5463f..1620ed4e1f 100644
--- a/common/usbc/usb_mode.c
+++ b/common/usbc/usb_mode.c
@@ -143,13 +143,13 @@ void enter_usb_failed(int port)
usb4_state[port] = USB4_INACTIVE;
}
-static bool enter_usb_response_valid(int port, enum tcpm_sop_type type)
+static bool enter_usb_response_valid(int port, enum tcpci_msg_type type)
{
/*
* Check for an unexpected response.
*/
if (get_usb_pd_cable_type(port) == IDH_PTYPE_PCABLE &&
- type != TCPC_TX_SOP) {
+ type != TCPCI_MSG_SOP) {
enter_usb_failed(port);
return false;
}
@@ -159,7 +159,7 @@ static bool enter_usb_response_valid(int port, enum tcpm_sop_type type)
bool enter_usb_port_partner_is_capable(int port)
{
const struct pd_discovery *disc =
- pd_get_am_discovery(port, TCPC_TX_SOP);
+ pd_get_am_discovery(port, TCPCI_MSG_SOP);
if (usb4_state[port] == USB4_INACTIVE)
return false;
@@ -177,9 +177,9 @@ bool enter_usb_cable_is_capable(int port)
return false;
} else if (get_usb_pd_cable_type(port) == IDH_PTYPE_ACABLE) {
struct pd_discovery *disc_sop_prime =
- pd_get_am_discovery(port, TCPC_TX_SOP_PRIME);
+ pd_get_am_discovery(port, TCPCI_MSG_SOP_PRIME);
- if (pd_get_vdo_ver(port, TCPC_TX_SOP_PRIME) >= VDM_VER20 &&
+ if (pd_get_vdo_ver(port, TCPCI_MSG_SOP_PRIME) >= VDM_VER20 &&
disc_sop_prime->identity.product_t1.a_rev30.vdo_ver >=
VDO_VERSION_1_3) {
union active_cable_vdo2_rev30 a2_rev30 =
@@ -200,14 +200,14 @@ bool enter_usb_cable_is_capable(int port)
*/
} else {
const struct pd_discovery *disc =
- pd_get_am_discovery(port, TCPC_TX_SOP);
+ pd_get_am_discovery(port, TCPCI_MSG_SOP);
union tbt_mode_resp_cable cable_mode_resp = {
.raw_value = pd_get_tbt_mode_vdo(port,
- TCPC_TX_SOP_PRIME) };
+ TCPCI_MSG_SOP_PRIME) };
if (!disc->identity.idh.modal_support ||
!pd_is_mode_discovered_for_svid(port,
- TCPC_TX_SOP_PRIME, USB_VID_INTEL) ||
+ TCPCI_MSG_SOP_PRIME, USB_VID_INTEL) ||
cable_mode_resp.tbt_rounded !=
TBT_GEN3_GEN4_ROUNDED_NON_ROUNDED)
return false;
@@ -216,7 +216,7 @@ bool enter_usb_cable_is_capable(int port)
return true;
}
-void enter_usb_accepted(int port, enum tcpm_sop_type type)
+void enter_usb_accepted(int port, enum tcpci_msg_type type)
{
struct pd_discovery *disc;
@@ -225,7 +225,7 @@ void enter_usb_accepted(int port, enum tcpm_sop_type type)
switch (usb4_state[port]) {
case USB4_ENTER_SOP_PRIME:
- disc = pd_get_am_discovery(port, TCPC_TX_SOP_PRIME);
+ disc = pd_get_am_discovery(port, TCPCI_MSG_SOP_PRIME);
if (disc->identity.product_t1.a_rev20.sop_p_p)
usb4_state[port] = USB4_ENTER_SOP_PRIME_PRIME;
else
@@ -254,7 +254,7 @@ void enter_usb_accepted(int port, enum tcpm_sop_type type)
}
}
-void enter_usb_rejected(int port, enum tcpm_sop_type type)
+void enter_usb_rejected(int port, enum tcpci_msg_type type)
{
if (!enter_usb_response_valid(port, type) ||
usb4_state[port] == USB4_ACTIVE)
@@ -263,13 +263,13 @@ void enter_usb_rejected(int port, enum tcpm_sop_type type)
enter_usb_failed(port);
}
-uint32_t enter_usb_setup_next_msg(int port, enum tcpm_sop_type *type)
+uint32_t enter_usb_setup_next_msg(int port, enum tcpci_msg_type *type)
{
struct pd_discovery *disc_sop_prime;
switch (usb4_state[port]) {
case USB4_START:
- disc_sop_prime = pd_get_am_discovery(port, TCPC_TX_SOP_PRIME);
+ disc_sop_prime = pd_get_am_discovery(port, TCPCI_MSG_SOP_PRIME);
/*
* Ref: Tiger Lake Platform PD Controller Interface Requirements
* for Integrated USBC, section A.2.2: USB4 as DFP.
@@ -279,24 +279,24 @@ uint32_t enter_usb_setup_next_msg(int port, enum tcpm_sop_type *type)
*/
usb_mux_set_safe_mode(port);
- if (pd_get_vdo_ver(port, TCPC_TX_SOP_PRIME) < VDM_VER20 ||
+ if (pd_get_vdo_ver(port, TCPCI_MSG_SOP_PRIME) < VDM_VER20 ||
disc_sop_prime->identity.product_t1.a_rev30.vdo_ver <
VDO_VERSION_1_3 ||
get_usb_pd_cable_type(port) == IDH_PTYPE_PCABLE) {
usb4_state[port] = USB4_ENTER_SOP;
} else {
usb4_state[port] = USB4_ENTER_SOP_PRIME;
- *type = TCPC_TX_SOP_PRIME;
+ *type = TCPCI_MSG_SOP_PRIME;
}
break;
case USB4_ENTER_SOP_PRIME:
- *type = TCPC_TX_SOP_PRIME;
+ *type = TCPCI_MSG_SOP_PRIME;
break;
case USB4_ENTER_SOP_PRIME_PRIME:
- *type = TCPC_TX_SOP_PRIME_PRIME;
+ *type = TCPCI_MSG_SOP_PRIME_PRIME;
break;
case USB4_ENTER_SOP:
- *type = TCPC_TX_SOP;
+ *type = TCPCI_MSG_SOP;
break;
case USB4_ACTIVE:
return -1;