Commit: 4d0c625609ece690e705502134f2770c14cf89d0 - system-bt (git)

system/bt

Commit MetaInfo

• Revision: 4d0c625609ece690e705502134f2770c14cf89d0 (tree)
• Time: 2019-06-19 12:30:12
• Author: Andre Eisenbach <eisenbach@goog...>
• Commiter: android-build-merger

Log Message

Limit sniff mode for Nintendo Pro Controller to 11.25ms
am: 85d56a8d38

Change-Id: Ic6398eb054f7bc20c7ab4dc2ee1e603c683c9044

Change Summary

• modified: bta/hh/bta_hh_utils.cc
• modified: device/include/interop.h
• modified: device/include/interop_database.h
• modified: device/src/interop.cc

Incremental Difference

--- a/bta/hh/bta_hh_utils.cc
+++ b/bta/hh/bta_hh_utils.cc
@@ -21,6 +21,7 @@
 #if (BTA_HH_INCLUDED == TRUE)
 
 #include "bta_hh_int.h"
+#include "device/include/interop.h"
 #include "osi/include/osi.h"
 
 /* if SSR max latency is not defined by remote device, set the default value
@@ -393,6 +394,11 @@ tBTA_HH_STATUS bta_hh_read_ssr_param(const RawAddress& bd_addr,
         if (ssr_max_latency > BTA_HH_SSR_MAX_LATENCY_DEF)
           ssr_max_latency = BTA_HH_SSR_MAX_LATENCY_DEF;
 
+        if (interop_match_addr(INTEROP_HID_HOST_LIMIT_SNIFF_INTERVAL,
+                               &bd_addr)) {
+          if (ssr_max_latency > 18 /* slots * 0.625ms */) ssr_max_latency = 18;
+        }
+
         *p_max_ssr_lat = ssr_max_latency;
       } else
         *p_max_ssr_lat = p_cb->kdev[i].dscp_info.ssr_max_latency;

--- a/device/include/interop.h
+++ b/device/include/interop.h
@@ -89,7 +89,11 @@ typedef enum {
   // Disable role switch for headsets/car-kits.
   // Some car kits allow role switch but when the Phone initiates role switch,
   // the Remote device will go into bad state that will lead to LMP time out.
-  INTEROP_DISABLE_ROLE_SWITCH
+  INTEROP_DISABLE_ROLE_SWITCH,
+
+  // Set a very low initial sniff subrating for HID devices that do not
+  // set their own sniff interval.
+  INTEROP_HID_HOST_LIMIT_SNIFF_INTERVAL,
 } interop_feature_t;
 
 // Check if a given |addr| matches a known interoperability workaround as

--- a/device/include/interop_database.h
+++ b/device/include/interop_database.h
@@ -143,6 +143,10 @@ static const interop_addr_entry_t interop_addr_database[] = {
 
     // AirPods 2 - unacceptably loud volume
     {{{0x94, 0x16, 0x25, 0, 0, 0}}, 3, INTEROP_DISABLE_ABSOLUTE_VOLUME},
+
+    // Nintendo Switch Pro Controller - does not set sniff interval dynamically.
+    // Requires custom HID report command to change mode.
+    {{{0x98, 0xB6, 0xE9, 0, 0, 0}}, 3, INTEROP_HID_HOST_LIMIT_SNIFF_INTERVAL},
 };
 
 typedef struct {

--- a/device/src/interop.cc
+++ b/device/src/interop.cc
@@ -128,6 +128,7 @@ static const char* interop_feature_string_(const interop_feature_t feature) {
     CASE_RETURN_STR(INTEROP_DISABLE_AVDTP_RECONFIGURE)
     CASE_RETURN_STR(INTEROP_DYNAMIC_ROLE_SWITCH)
     CASE_RETURN_STR(INTEROP_DISABLE_ROLE_SWITCH)
+    CASE_RETURN_STR(INTEROP_HID_HOST_LIMIT_SNIFF_INTERVAL)
   }
 
   return "UNKNOWN";