Revert "Usb Gadget HAL: modify monitorfs behavior"

This reverts commit 9cf018372ba8527af4bc54d453f1e2a5d0a9749f.

Reason for revert: using alternate method for original behavior change
Bug: 187261232
Test: build pass

Merged-In: I3de693862ae49d78ca41b09cb7450fb5a8f9d2a3
Change-Id: Idb35b586f39c6e6bf772c219c7e4ed9f3cee4778

2 files changed, 4 insertions, 32 deletions

diff --git a/usb/MonitorFfs.cpp b/usb/MonitorFfs.cpp
index dc25aba..717e6a7 100644
--- a/usb/MonitorFfs.cpp
+++ b/usb/MonitorFfs.cpp
@@ -26,7 +26,7 @@ namespace usb {
 
 static volatile bool gadgetPullup;
 
-MonitorFfs::MonitorFfs(const char *const gadget, const char *const extconUsbState)
+MonitorFfs::MonitorFfs(const char *const gadget)
     : mWatchFd(),
       mEndpointList(),
       mLock(),
@@ -37,7 +37,6 @@ MonitorFfs::MonitorFfs(const char *const gadget, const char *const extconUsbStat
       mCallback(NULL),
       mPayload(NULL),
       mGadgetName(gadget),
-      mExtconUsbState(extconUsbState),
       mMonitorRunning(false) {
     unique_fd eventFd(eventfd(0, 0));
     if (eventFd == -1) {
@@ -119,7 +118,6 @@ void *MonitorFfs::startMonitorFd(void *param) {
     bool writeUdc = true, stopMonitor = false;
     struct epoll_event events[kEpollEvents];
     steady_clock::time_point disconnect;
-    std::string usb_state;
 
     bool descriptorWritten = true;
     for (int i = 0; i < static_cast<int>(monitorFfs->mEndpointList.size()); i++) {
@@ -129,17 +127,8 @@ void *MonitorFfs::startMonitorFd(void *param) {
         }
     }
 
-    if (ReadFileToString(monitorFfs->mExtconUsbState, &usb_state))
-        usb_state = Trim(usb_state);
-    else
-        usb_state = TYPEC_GADGET_MODE_ENABLE;
-
     // notify here if the endpoints are already present.
-    if (usb_state == TYPEC_GADGET_MODE_DISABLE) {
-        gadgetPullup = false;
-        writeUdc = false;
-        ALOGI("pending GADGET pulled up due to USB disconnected");
-    } else if (descriptorWritten) {
+    if (descriptorWritten) {
         usleep(kPullUpDelay);
         if (!!WriteStringToFile(monitorFfs->mGadgetName, PULLUP_PATH)) {
             lock_guard<mutex> lock(monitorFfs->mLock);
@@ -183,17 +172,7 @@ void *MonitorFfs::startMonitorFd(void *param) {
                         }
                     }
 
-                    if (ReadFileToString(monitorFfs->mExtconUsbState, &usb_state))
-                        usb_state = Trim(usb_state);
-                    else
-                        usb_state = TYPEC_GADGET_MODE_ENABLE;
-
-                    if (usb_state == TYPEC_GADGET_MODE_DISABLE) {
-                        if (kDebug)
-                            ALOGI("pending GADGET pulled up due to USB disconnected");
-                        gadgetPullup = false;
-                        writeUdc = false;
-                    } else if (!descriptorPresent && !writeUdc) {
+                    if (!descriptorPresent && !writeUdc) {
                         if (kDebug)
                             ALOGI("endpoints not up");
                         writeUdc = true;
diff --git a/usb/include/pixelusb/UsbGadgetCommon.h b/usb/include/pixelusb/UsbGadgetCommon.h
index fde80b3..964a1d6 100644
--- a/usb/include/pixelusb/UsbGadgetCommon.h
+++ b/usb/include/pixelusb/UsbGadgetCommon.h
@@ -19,7 +19,6 @@
 
 #include <android-base/file.h>
 #include <android-base/properties.h>
-#include <android-base/strings.h>
 #include <android-base/unique_fd.h>
 
 #include <android/hardware/usb/gadget/1.0/IUsbGadget.h>
@@ -75,13 +74,9 @@ constexpr char kVendorRndisConfig[] = "vendor.usb.rndis.config";
 #define FUNCTION_NAME "function"
 #define FUNCTION_PATH CONFIG_PATH FUNCTION_NAME
 #define RNDIS_PATH FUNCTIONS_PATH "gsi.rndis"
-#define TYPEC_GADGET_MODE_DISABLE "0"
-#define TYPEC_GADGET_MODE_ENABLE "1"
 
 using ::android::base::GetProperty;
-using ::android::base::ReadFileToString;
 using ::android::base::SetProperty;
-using ::android::base::Trim;
 using ::android::base::unique_fd;
 using ::android::base::WriteStringToFile;
 using ::android::hardware::usb::gadget::V1_0::GadgetFunction;
@@ -131,13 +126,11 @@ class MonitorFfs {
   void *mPayload;
   // Name of the USB gadget. Used for pullup.
   const char *const mGadgetName;
-  // Extcon USB state from Type-C notification.
-  const char *const mExtconUsbState;
   // Monitor State
   bool mMonitorRunning;
 
  public:
-  MonitorFfs(const char *const gadget, const char *const extconUsbState);
+  MonitorFfs(const char *const gadget);
   // Inits all the UniqueFds.
   void reset();
   // Starts monitoring endpoints and pullup the gadget when