commit 24c0f58c67c569e92e7b7d46e2d0cd29ebb64eb7
author Zhihai Xu <zhihaixu@google.com> Tue Apr 16 17:58:43 2013 -0700
committer Zhihai Xu <zhihaixu@google.com> Tue Apr 16 17:58:43 2013 -0700
tree 5ee4700b46516aeccc19d1db44c7defdc35b3597
parent 33af35c49731bc8e305c862c8e9f3e42a1ab0dc9

Audio glitches while playing songs over MOTO S305 headset

The MOTO S305 doesn't support read remote extended features command.
The fix is we should always use ReadRemoteSupportedFeaturesCommand at first,
Only send ReadRemoteExtendedFeaturesCommand if extend features are supported
by remote device.

issue 8618753

Change-Id: Id6d9c0e712dc892bcfca20336d1290b9443213ac

stack/btm/btm_acl.c

diff --git a/stack/btm/btm_acl.c b/stack/btm/btm_acl.c
index 21307c1..ea1c68e 100644
--- a/stack/btm/btm_acl.c
+++ b/stack/btm/btm_acl.c
@@ -1336,16 +1336,9 @@
     p_acl_cb->num_read_pages = 0;
     memset (p_acl_cb->peer_lmp_features, 0, sizeof(p_acl_cb->peer_lmp_features));
 
-    /* If this BT controller supports Read Extended Feature */
-    if (btm_cb.devcb.local_version.hci_version >= HCI_PROTO_VERSION_2_0)
-    {
-        btm_read_remote_ext_features(handle, HCI_EXT_FEATURES_PAGE_0);
-    }
-    /* else, if this is a very old BT controller */
-    else
-    {
-        btsnd_hcic_rmt_features_req (handle);
-    }
+    /* first send read remote supported features HCI command */
+    /* because we don't know whether the remote support extended feature command */
+    btsnd_hcic_rmt_features_req (handle);
 }