commit c7aa813e0866fc7f027592d0d0522552fb125fec
author Michael Bestas <mkbestas@lineageos.org> Wed Mar 23 00:01:10 2022 +0200
committer Michael Bestas <mkbestas@lineageos.org> Tue Oct 24 17:43:04 2023 +0300
tree 7034b7b11d61e2747f47891b5ffe2ae3400c89ad
parent b901ba3e25fcd7ae30905fa026d51a0e744db7b2

enableUsbDataSignal: Handle failures in optional sysfs gracefully

* ID and VBUS paths are not present on all devices,
  depending on the device dwc3 setup.
* Gadget pull is optional.

Change-Id: I235284ad7bad9fbf34568958fd94e32b2787a1c5

hal/Usb.cpp

diff --git a/hal/Usb.cpp b/hal/Usb.cpp
index e30eaff..d79cd1c 100644
--- a/hal/Usb.cpp
+++ b/hal/Usb.cpp
@@ -68,18 +68,15 @@
     }
 
     if (!WriteStringToFile(mGadgetName, PULLUP_PATH)) {
-      ALOGE("Gadget cannot be pulled up");
-      result = false;
+      ALOGW("Gadget cannot be pulled up");
     }
   } else {
     if (!WriteStringToFile("1", mDevicePath + ID_PATH)) {
-      ALOGE("Not able to turn off host mode");
-      result = false;
+      ALOGW("Not able to turn off host mode");
     }
 
     if (!WriteStringToFile("0", mDevicePath + VBUS_PATH)) {
-      ALOGE("Not able to set Vbus state");
-      result = false;
+      ALOGW("Not able to set Vbus state");
     }
 
     if (!WriteStringToFile("0", mDevicePath + USB_DATA_PATH)) {
@@ -88,8 +85,7 @@
     }
 
     if (!WriteStringToFile("none", PULLUP_PATH)) {
-      ALOGE("Gadget cannot be pulled down");
-      result = false;
+      ALOGW("Gadget cannot be pulled down");
     }
   }