commit c5a560160fd82e94870a2252037d394e1e1f9780
author Kyle Tso <kyletso@google.com> Thu Jun 13 16:54:37 2024 +0800
committer Kyle Tso <kyletso@google.com> Thu Jun 13 17:12:20 2024 +0800
tree 396e2ea7a03e5ccd4c3cd0a29911f220d619cfec
parent 744c86c49a9ff9e29e2193cdc5079b27d5e06ec3

usb: Log for retry on role switch

Add a log for the retry. Also define the sleep time in the header file.

Bug: 341996696
Change-Id: Ib3cc0c6ad576e7cda26b7f50cdc61d52e6349c78
Signed-off-by: Kyle Tso <kyletso@google.com>
(cherry picked from commit d17fdff5f81cff76adaa75d50f26318be8048123)

usb/usb/Usb.cpp

diff --git a/usb/usb/Usb.cpp b/usb/usb/Usb.cpp
index 39cbceb..0736f4c 100644
--- a/usb/usb/Usb.cpp
+++ b/usb/usb/Usb.cpp
@@ -594,7 +594,8 @@
         if (fp != NULL) {
             int ret = fputs(convertRoletoString(in_role).c_str(), fp);
             if (ret == EAGAIN) {
-                std::this_thread::sleep_for(std::chrono::milliseconds(700));
+                ALOGI("role switch busy, retry in %d ms", ROLE_SWAP_RETRY_MS);
+                std::this_thread::sleep_for(std::chrono::milliseconds(ROLE_SWAP_RETRY_MS));
                 ret = fputs(convertRoletoString(in_role).c_str(), fp);
             }
             fclose(fp);

usb/usb/Usb.h

diff --git a/usb/usb/Usb.h b/usb/usb/Usb.h
index facc328..15348af 100644
--- a/usb/usb/Usb.h
+++ b/usb/usb/Usb.h
@@ -55,6 +55,8 @@
 #define VBUS_PATH NEW_UDC_PATH "dwc3_exynos_otg_b_sess"
 #define USB_DATA_PATH NEW_UDC_PATH "usb_data_enabled"
 
+#define ROLE_SWAP_RETRY_MS 700
+
 struct Usb : public BnUsb {
     Usb();