From f86f7c8a1785ad661d9aa23c9e3727768728ae60 Mon Sep 17 00:00:00 2001 From: Diana Z Date: Mon, 29 Mar 2021 15:20:10 -0600 Subject: Revert "RAA489000: Clear debug accessory state on init" This reverts commit 756dc91b0a57aa6d557ee4cb486972d8a7b8106a. There will now be a general debug detach on entry to Unattached.SNK/SRC every boot and so this should no longer be necessary. BRANCH=None BUG=b:177870522 TEST=on galith, ensure that refresh+power resets with servo_v4 sink plugged in results in a successful connection with the servo_v4 Signed-off-by: Diana Z Change-Id: I991ca7fbb5d51d2d8fc07d3a8d8e885e43e94b44 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2792691 Reviewed-by: Aseda Aboagye --- driver/tcpm/raa489000.c | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/driver/tcpm/raa489000.c b/driver/tcpm/raa489000.c index abb92e656f..c4c535a074 100644 --- a/driver/tcpm/raa489000.c +++ b/driver/tcpm/raa489000.c @@ -60,7 +60,6 @@ int raa489000_init(int port) int device_id; int i2c_port; struct charge_port_info chg = { 0 }; - static bool first_init_done[CONFIG_USB_PD_PORT_MAX_COUNT]; /* Perform unlock sequence */ rv = tcpc_write16(port, 0xAA, 0xDAA0); @@ -213,20 +212,6 @@ int raa489000_init(int port) if (rv) CPRINTS("c%d: failed to set Vbus Target Voltage", port); - if (!first_init_done[port]) { - first_init_done[port] = true; - - /* - * Ensure we detach from any previous debug connections if we - * have enough battery to survive Vbus loss. The PD task will - * often start with open CC lines, however this is not the case - * for ITE ECs when they are reset by the security chip since - * they cannot distinguish power on from a reset. - */ - if (pd_is_battery_capable()) - tcpm_debug_detach(port); - } - return rv; } -- cgit v1.2.1