Don't wake the host from suspend on mouse movement

Only wake the host when we have a report with keys or buttons pressed.

This matches real device behavior.

Author: jfedor2

firmware/src/main.cc

@@ -72,7 +72,9 @@ void __no_inline_not_in_flash_func(sof_handler)(uint32_t frame_count) {
 }
 
 bool do_send_report(uint8_t interface, const uint8_t* report_with_id, uint8_t len) {
-    if (tud_suspended()) {
+    if (tud_suspended() &&
+        (our_descriptor->should_cause_wakeup != nullptr) &&
+        our_descriptor->should_cause_wakeup(report_with_id[0], report_with_id + 1, len - 1)) {
         tud_remote_wakeup();
     } else {
         tud_hid_n_report(interface, report_with_id[0], report_with_id + 1, len - 1);

firmware/src/our_descriptor.cc

@@ -541,6 +541,26 @@ uint16_t kb_mouse_handle_get_report(uint8_t report_id, uint8_t* buffer, uint16_t
     return 0;
 }
 
+bool kb_mouse_should_cause_wakeup(uint8_t report_id, const uint8_t* buffer, uint16_t len) {
+    if ((report_id == REPORT_ID_KEYBOARD) || (report_id == REPORT_ID_CONSUMER)) {
+        for (uint16_t i = 0; i < len; i++) {
+            if (buffer[i] != 0) {
+                return true;
+            }
+        }
+        return false;
+    }
+
+    if (report_id == REPORT_ID_MOUSE) {
+        if ((len > 0) && (buffer[0] != 0)) {
+            return true;
+        }
+        return false;
+    }
+
+    return false;
+}
+
 static const uint8_t horipad_neutral[] = { 0x00, 0x00, 0x0F, 0x80, 0x80, 0x80, 0x80, 0x00 };
 
 void horipad_clear_report(uint8_t* report, uint8_t report_id, uint16_t len) {
@@ -618,6 +638,7 @@ const our_descriptor_def_t our_descriptors[] = {
         .handle_get_report = kb_mouse_handle_get_report,
         .handle_set_report = kb_mouse_handle_set_report,
         .set_report_synchronous = kb_mouse_set_report_synchronous,
+        .should_cause_wakeup = kb_mouse_should_cause_wakeup,
     },
     {
         .idx = 1,
@@ -627,6 +648,7 @@ const our_descriptor_def_t our_descriptors[] = {
         .handle_get_report = kb_mouse_handle_get_report,
         .handle_set_report = kb_mouse_handle_set_report,
         .set_report_synchronous = kb_mouse_set_report_synchronous,
+        .should_cause_wakeup = kb_mouse_should_cause_wakeup,
     },
     {
         .idx = 2,

firmware/src/our_descriptor.h

@@ -27,6 +27,7 @@ typedef void (*handle_set_report_complete_t)(uint16_t interface, uint8_t report_
 typedef void (*clear_report_t)(uint8_t* report, uint8_t report_id, uint16_t len);
 typedef int32_t (*default_value_t)(uint32_t usage);
 typedef void (*sanitize_report_t)(uint8_t report_id, uint8_t* buffer, uint16_t len);
+typedef bool (*should_cause_wakeup_t)(uint8_t report_id, const uint8_t* buffer, uint16_t len);
 
 struct our_descriptor_def_t {
     uint8_t idx;
@@ -46,6 +47,7 @@ struct our_descriptor_def_t {
     clear_report_t clear_report = nullptr;
     default_value_t default_value = nullptr;
     sanitize_report_t sanitize_report = nullptr;
+    should_cause_wakeup_t should_cause_wakeup = nullptr;
 };
 
 extern const our_descriptor_def_t our_descriptors[];