diff options
author | Abe Levkoy <alevkoy@chromium.org> | 2021-08-27 15:21:39 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-08-31 18:18:20 +0000 |
commit | 003ec08f8c67845ae8481d3fb4124555213b04c4 (patch) | |
tree | 3502c59532731d0003532657b1a2993e8ca9ad81 /common/usbc/usb_mode.c | |
parent | 5fa669005c3779b7310bf6bfe43f741a3d8ae871 (diff) | |
download | chrome-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.c | 36 |
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; |