diff options
| author | Abe Levkoy <alevkoy@chromium.org> | 2021-02-22 16:54:23 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-02-23 01:36:53 +0000 |
| commit | b3c8abc9a71a516f6aad5759b83bfadd131c2b07 (patch) | |
| tree | 8ad38261d802cb790778330132718472d8cd64dc /common | |
| parent | e193be9c9b02c97cb303605eb473d7a3a4ba7a65 (diff) | |
| download | chrome-ec-b3c8abc9a71a516f6aad5759b83bfadd131c2b07.tar.gz | |
TCPMv2: Don't check VBUS when entering BIST mode
Temporarily remove check for VBUS = vSafe5V when entering BIST mode. It
does this by measuring VBUS, but this facility is inadequate to the
purpose on Volteer. Since this even more of an edge case than actually
running BIST tests, remove the check for now, pending a more general
solution.
BUG=b:180957710,b:173023378
TEST=make buildall; pass TDA.2.1.2.2 BMC PHY RX INT REJ on voxel
BRANCH=firmware-volteer-13672.B-main
Signed-off-by: Abe Levkoy <alevkoy@chromium.org>
Change-Id: I172c1bcc10731c2e405344cdc6ad2b5a1ed80ced
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2714203
Reviewed-by: Diana Z <dzigterman@chromium.org>
Diffstat (limited to 'common')
| -rw-r--r-- | common/usbc/usb_pe_drp_sm.c | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/common/usbc/usb_pe_drp_sm.c b/common/usbc/usb_pe_drp_sm.c index 2c3d442dfb..3c17a17d4d 100644 --- a/common/usbc/usb_pe_drp_sm.c +++ b/common/usbc/usb_pe_drp_sm.c @@ -4898,21 +4898,6 @@ static void pe_bist_tx_entry(int port) print_current_state(port); - /* - * Ignore BIST messages when not operating at vSafe5V. If charge_manager - * isn't enabled, but PD has gotten this far, then VBUS should be at - * vSafe5V. - */ - if (IS_ENABLED(CONFIG_CHARGE_MANAGER)) { - int vbus_voltage_mv = charge_manager_get_vbus_voltage(port); - - if (vbus_voltage_mv < PD_V_SAFE5V_MIN || - vbus_voltage_mv > PD_V_SAFE5V_MAX) { - pe_set_ready_state(port); - return; - } - } - if (mode == BIST_CARRIER_MODE_2) { /* * PE_BIST_Carrier_Mode embedded here. |