usb_usb: power saving during suspend without remote wakeup

#769 (comment)
tmk · Dec 20, 2023 · 0966463 · 0966463
1 parent ea27f71
commit 0966463
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 0 deletions.
diff --git a/converter/usb_usb/config.h b/converter/usb_usb/config.h
@@ -53,4 +53,7 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #define LOCKING_SUPPORT_ENABLE
 #define LOCKING_RESYNC_ENABLE
 
+// power saving during suspended without remote wakeup disabled
+#define UHS2_POWER_SAVING
+
 #endif
diff --git a/converter/usb_usb/usb_usb.cpp b/converter/usb_usb/usb_usb.cpp
@@ -242,15 +242,38 @@ void hook_usb_suspend_loop(void)
 static uint8_t _led_stats = 0;
 void hook_usb_suspend_entry(void)
 {
+    dprintf("[S]");
     matrix_clear();
     clear_keyboard();
 
     usb_host.suspend();
+
+#ifdef UHS2_POWER_SAVING
+    // power down when remote wake is not enabled
+    if (!USB_Device_RemoteWakeupEnabled) {
+        dprintf("[p]");
+        usb_host.powerDown();
+    }
+#endif
 }
 
 void hook_usb_wakeup(void)
 {
+    dprintf("[W]");
     suspend_wakeup_init();
 
+#ifdef UHS2_POWER_SAVING
+    // power down when remote wake is not enabled
+    if (!USB_Device_RemoteWakeupEnabled) {
+        dprintf("[P]");
+        usb_host.powerUp();
+
+        // USB state cannot be retained through power down/up cycle
+        // device should be enumerated and initialize from the beginning
+        usb_host.ReleaseAllDevices();
+        usb_host.setUsbTaskState(USB_STATE_DETACHED);
+    }
+#endif
+
     usb_host.resume();
 }