OSDN Git Service
Hansong Zhang [Sat, 26 Sep 2020 01:41:31 +0000 (01:41 +0000)]
HIDD: Treat unacceptable QOS as normal failure am:
3c87d00189 am:
e5c650c227
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1439773
Change-Id: I7a5f7bf0451ce3f9b875cd90b8240a98c183d7a1
Hansong Zhang [Sat, 26 Sep 2020 01:41:29 +0000 (01:41 +0000)]
L2CA_ConfigReq and L2CA_ConfigRsp are internal now am:
acc416288e am:
469629acc2
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1439772
Change-Id: Idfaf325b7d4ef9de3369ebcd7dc1c0c4b8ea3871
Hansong Zhang [Sat, 26 Sep 2020 01:41:27 +0000 (01:41 +0000)]
Remove L2CA_ConfigReq in client code am:
e91542d478 am:
55776d6d6a
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1439451
Change-Id: I798ab375e33ad2ba4dca3c2d006898314ceee523
Hansong Zhang [Sat, 26 Sep 2020 01:41:25 +0000 (01:41 +0000)]
L2CAP module will send ConfigReq for you am:
65acfa23cf am:
f1d3cdc3b3
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438418
Change-Id: I5775f7e30de43213ff13a783e235b572749c1a78
Hansong Zhang [Sat, 26 Sep 2020 01:41:23 +0000 (01:41 +0000)]
L2CAP: Don't allow sending re-config am:
db34984d29 am:
ad09212a89
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438672
Change-Id: Ica13009f4be10e74ddfb74e3177b1529f7c167a2
Hansong Zhang [Sat, 26 Sep 2020 01:41:21 +0000 (01:41 +0000)]
L2CA_ConfigRsp is always invoked with status OK am:
8588d5cdd1 am:
ee1e17e709
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438671
Change-Id: I6a9f1a308b44f8f15f17e80628b58551e31e8707
Hansong Zhang [Sat, 26 Sep 2020 01:41:19 +0000 (01:41 +0000)]
L2CAP: Provide a default ERTM option am:
0640f93856 am:
6721ca6f6d
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438417
Change-Id: I130ad57b81589add56b042d507c16c6a2a2f3981
Hansong Zhang [Sat, 26 Sep 2020 01:41:17 +0000 (01:41 +0000)]
Some ERTM values are constant am:
1b6d60e504 am:
75af19b15d
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438416
Change-Id: I150d26e8c52f2f56749a7ec59dc0586dc5aa366d
Hansong Zhang [Sat, 26 Sep 2020 01:41:15 +0000 (01:41 +0000)]
Remove unnecessary definition am:
989e83b5c1 am:
d675b7bc07
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438415
Change-Id: Ia9b0091b3c17991b3f8df23fc6deeaf2b62f947d
Treehugger Robot [Sat, 26 Sep 2020 01:40:51 +0000 (01:40 +0000)]
Merge "HciCaptures: Add CommandComplete" am:
08fd1d6421 am:
38ec6d219a
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438313
Change-Id: Iac9663d9cb401b260bbdef9dd1edcc38f32c5c9a
Hansong Zhang [Sat, 26 Sep 2020 01:20:36 +0000 (01:20 +0000)]
HIDD: Treat unacceptable QOS as normal failure am:
3c87d00189
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1439773
Change-Id: Ib50488e6972f13614ac9fa352032213221650a62
Hansong Zhang [Sat, 26 Sep 2020 01:20:31 +0000 (01:20 +0000)]
L2CA_ConfigReq and L2CA_ConfigRsp are internal now am:
acc416288e
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1439772
Change-Id: I0cbd5d725f6053d051f35b1e40c98fe6883a25c5
Hansong Zhang [Sat, 26 Sep 2020 01:20:27 +0000 (01:20 +0000)]
Remove L2CA_ConfigReq in client code am:
e91542d478
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1439451
Change-Id: I68652917693600c85d10e10574c5f4f64d7c51f6
Hansong Zhang [Sat, 26 Sep 2020 01:20:24 +0000 (01:20 +0000)]
L2CAP module will send ConfigReq for you am:
65acfa23cf
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438418
Change-Id: I4ea5f0e86c2bf834b348f7f3750517f0fed4550f
Hansong Zhang [Sat, 26 Sep 2020 01:20:20 +0000 (01:20 +0000)]
L2CAP: Don't allow sending re-config am:
db34984d29
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438672
Change-Id: If31d05d271340f2dd41561a4bc7142f223b9927b
Hansong Zhang [Sat, 26 Sep 2020 01:20:15 +0000 (01:20 +0000)]
L2CA_ConfigRsp is always invoked with status OK am:
8588d5cdd1
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438671
Change-Id: I593f15a4d4157df2a7109c2c43d74559a7cbabd8
Hansong Zhang [Sat, 26 Sep 2020 01:20:12 +0000 (01:20 +0000)]
L2CAP: Provide a default ERTM option am:
0640f93856
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438417
Change-Id: I18acbf65f5eab682de8073b98875f828da5056fa
Hansong Zhang [Sat, 26 Sep 2020 01:20:07 +0000 (01:20 +0000)]
Some ERTM values are constant am:
1b6d60e504
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438416
Change-Id: I0803905600edcfd0c0012d51917c584815ef87d8
Hansong Zhang [Sat, 26 Sep 2020 01:20:03 +0000 (01:20 +0000)]
Remove unnecessary definition am:
989e83b5c1
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438415
Change-Id: I82073c462fe23457dc86e2cb3580de5da844b8d0
Treehugger Robot [Sat, 26 Sep 2020 01:19:01 +0000 (01:19 +0000)]
Merge "HciCaptures: Add CommandComplete" am:
08fd1d6421
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438313
Change-Id: Idc051f11ee5ccf06d7b241fb5dc66261ae34bc0f
Hansong Zhang [Fri, 25 Sep 2020 19:17:12 +0000 (12:17 -0700)]
HIDD: Treat unacceptable QOS as normal failure
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I7d37c46f7c6b16a55afafe04b62081d43f63c122
Hansong Zhang [Fri, 25 Sep 2020 18:31:02 +0000 (11:31 -0700)]
L2CA_ConfigReq and L2CA_ConfigRsp are internal now
Only used by l2c_csm, so we make an internal version instead.
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: If725c705bcbabec025cb16d063cd4d4aa4b319f9
Hansong Zhang [Fri, 25 Sep 2020 18:15:00 +0000 (11:15 -0700)]
Remove L2CA_ConfigReq in client code
Let L2CAP do this for you.
HIDD used to send anothe ConfigReq if the remote HIDH wants a different
QOS. We drop this adjustment for now. QOS is not used. Regard this as a
normal config error. This simplifies the workflow.
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: Ic3ff08a4b876462fec697ffa8a09fc8bca943603
Hansong Zhang [Fri, 25 Sep 2020 04:32:08 +0000 (21:32 -0700)]
L2CAP module will send ConfigReq for you
No need to send it in client code.
Will get rid of the fake L2CA_ConfigReq() next.
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I66e2efad144e76eb6abfa6d421e12ae4a60d8e1f
Hansong Zhang [Fri, 25 Sep 2020 07:04:44 +0000 (00:04 -0700)]
L2CAP: Don't allow sending re-config
There is no use case. Remove the support for now. This makes L2CAP state
machine more complicated.
Note that we don't handle re-config request from remote gracefully. For
example, RFCOMM will simply disconnect.
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I1d2a420e4f904b7e935715f327171d133760eab3
Hansong Zhang [Fri, 25 Sep 2020 06:28:56 +0000 (23:28 -0700)]
L2CA_ConfigRsp is always invoked with status OK
So we can move some error processing logic.
Also we never send a config with continuation flag.
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I6c5e05b39a76138f90b41a0e56b7bfd73f056455
Hansong Zhang [Fri, 25 Sep 2020 04:20:37 +0000 (21:20 -0700)]
L2CAP: Provide a default ERTM option
Note that the existing ERTM options for AVCT_BR and GAP OBEX are almost
the same, and the only difference is TxWindow (10 and 20). Providing a
default value of 10 won't make a difference.
Use the same ERTM option so that later L2CAP can send config request for
them.
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I410045f47c643fd4a152ae44722e905b2ce01197
Hansong Zhang [Fri, 25 Sep 2020 03:59:13 +0000 (20:59 -0700)]
Some ERTM values are constant
Remove unnecessary definition.
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I2cb74575a57262e11ebc389a3f9473702dbb13b4
Hansong Zhang [Fri, 25 Sep 2020 03:45:11 +0000 (20:45 -0700)]
Remove unnecessary definition
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I9baf600d22cad78b2b93d29dd7d8975731beb47d
Treehugger Robot [Sat, 26 Sep 2020 00:51:46 +0000 (00:51 +0000)]
Merge "HciCaptures: Add CommandComplete"
Hansong Zhang [Fri, 25 Sep 2020 23:14:36 +0000 (23:14 +0000)]
Let L2CAP send ConfigRsp for you am:
75d1c63ba5 am:
c00171736f
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438234
Change-Id: I2872c55f5637ea68e85c1d01aa3ebc68bded0f15
Chris Manton [Fri, 25 Sep 2020 23:14:21 +0000 (23:14 +0000)]
Straighten out stack/acl/btm_acl::btm_read_remote_version_complete am:
d64f464405 am:
dce4e2f847
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438551
Change-Id: I21a1003225894781a9fc338b9e099ee216381977
Chris Manton [Fri, 25 Sep 2020 23:14:19 +0000 (23:14 +0000)]
Add gd/l2cap::OnReadRemoteVersionInformationComplete am:
11131aa71b am:
cd50f3afcf
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438552
Change-Id: I63c96c495e362e433dcb7b3c35915cba77a3924d
Hansong Zhang [Fri, 25 Sep 2020 22:55:30 +0000 (22:55 +0000)]
Let L2CAP send ConfigRsp for you am:
75d1c63ba5
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438234
Change-Id: I18cd60e397d3d74dbf2827918010b08e443b2f7d
Myles Watson [Thu, 24 Sep 2020 15:51:14 +0000 (08:51 -0700)]
HciCaptures: Add CommandComplete
Add public methods to match or extract events in matchers.py.
Call the same private static methods to extract or match events.
- CommandComplete
- LeMetaEvent
- LeConnectionComplete
Test: cert/run --host
Bug:
145832107
Tag: #gd-refactor
Change-Id: I308e97cfe60dbcd351f211ff836c0516019412db
Chris Manton [Fri, 25 Sep 2020 22:43:09 +0000 (22:43 +0000)]
Straighten out stack/acl/btm_acl::btm_read_remote_version_complete am:
d64f464405
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438551
Change-Id: I39a09c44ed84df9c6f4d5c076d0b7fc171813b35
Chris Manton [Fri, 25 Sep 2020 22:42:24 +0000 (22:42 +0000)]
Add gd/l2cap::OnReadRemoteVersionInformationComplete am:
11131aa71b
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438552
Change-Id: Ie02d5e1824a37937b7b4e8b08dd94738d9ec11b7
Hansong Zhang [Fri, 25 Sep 2020 03:05:07 +0000 (20:05 -0700)]
Let L2CAP send ConfigRsp for you
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I4767af7032b9c22fc297cca0877a1bc7e8c8607b
Chris Manton [Fri, 25 Sep 2020 00:11:21 +0000 (17:11 -0700)]
Straighten out stack/acl/btm_acl::btm_read_remote_version_complete
Towards readable code
Bug:
163134718
Tag: #refactor
Test: ble paired 2 phones
Test: act.py -tc BleCocTest
Change-Id: I87db6ce0106e98684f1582a60f28beed81d66aec
Chris Manton [Fri, 25 Sep 2020 05:12:00 +0000 (22:12 -0700)]
Add gd/l2cap::OnReadRemoteVersionInformationComplete
Provide ability to handle at higher layer
Bug:
166280067
Tag: #refactor
Test: gd/cert/run --host
Change-Id: I717b273623df4a3e21bf07e62b034cd7ad790cd3
Treehugger Robot [Fri, 25 Sep 2020 19:51:10 +0000 (19:51 +0000)]
Merge "Handle remote connection parameter request" am:
827c2217e9 am:
f6f9561687
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1439391
Change-Id: I15bc4a7c7a3c835ed5521f5dd3dd647f2b4d742e
Hansong Zhang [Fri, 25 Sep 2020 19:51:02 +0000 (19:51 +0000)]
Enforce remote MTU requirement in L2CAP directly am:
679dfc5152 am:
3a331729b1
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438233
Change-Id: Ifec682a469df78de30d58917e26b370973d499d2
Hansong Zhang [Fri, 25 Sep 2020 19:51:01 +0000 (19:51 +0000)]
L2CAP: peer_cfg_bits is unused am:
b0d9b4b9b7 am:
ec7778f6c1
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438232
Change-Id: I5b03052f9dfba76068b0b4a10902baea564a3c2b
Hansong Zhang [Fri, 25 Sep 2020 19:51:00 +0000 (19:51 +0000)]
L2CAP: Add a "required_remote_mtu" and store it am:
8ddee4d822 am:
63dab83aef
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438414
Change-Id: Ia70490add3ee2794a6502a5fbe0331f869fe0ab6
Hansong Zhang [Fri, 25 Sep 2020 19:50:58 +0000 (19:50 +0000)]
Remove unused code in BNEP am:
bf88eeb9ea am:
0df9618bc8
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438412
Change-Id: I637409054e4689a90a1132fcb98e9e418af25ba5
Hansong Zhang [Fri, 25 Sep 2020 19:50:14 +0000 (19:50 +0000)]
Profile services have const MTU size now am:
2e15c539cc am:
6a7bc570af
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438411
Change-Id: Ifc88585026028f862bcd378c7090d043b574c3d3
Treehugger Robot [Fri, 25 Sep 2020 19:19:43 +0000 (19:19 +0000)]
Merge "Handle remote connection parameter request" am:
827c2217e9
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1439391
Change-Id: I49d07ef7d59d9b312b01caaa1a17944d77d60212
Hansong Zhang [Fri, 25 Sep 2020 19:19:29 +0000 (19:19 +0000)]
Enforce remote MTU requirement in L2CAP directly am:
679dfc5152
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438233
Change-Id: I54ee513e88222c2119ecc89ef45eb3ad389f912b
Hansong Zhang [Fri, 25 Sep 2020 19:19:25 +0000 (19:19 +0000)]
L2CAP: peer_cfg_bits is unused am:
b0d9b4b9b7
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438232
Change-Id: Ib6ce3a945650bbb9fe1a92d60b92c1f44406c06a
Hansong Zhang [Fri, 25 Sep 2020 19:19:21 +0000 (19:19 +0000)]
L2CAP: Add a "required_remote_mtu" and store it am:
8ddee4d822
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438414
Change-Id: I1b184c4b93e50339d346cfd635a36c007ed944c3
Hansong Zhang [Fri, 25 Sep 2020 19:19:18 +0000 (19:19 +0000)]
Remove unused code in BNEP am:
bf88eeb9ea
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438412
Change-Id: Ied8c5cccc4da6f559095f9f37ae127d9445d7a53
Hansong Zhang [Fri, 25 Sep 2020 19:17:59 +0000 (19:17 +0000)]
Profile services have const MTU size now am:
2e15c539cc
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438411
Change-Id: Ie2fde010c0631b54cc9c8199cb7c91e5bced60cc
Treehugger Robot [Fri, 25 Sep 2020 19:15:18 +0000 (19:15 +0000)]
Merge "Handle remote connection parameter request"
Jakub Pawlowski [Fri, 25 Sep 2020 17:28:07 +0000 (19:28 +0200)]
Handle remote connection parameter request
Bug:
145832107
Test: connect with LE Device that request parameter chagne
Tag: #gd-refactor
Change-Id: I82839b56c58dda4b2f430c212156bf8919d67103
Hansong Zhang [Fri, 25 Sep 2020 02:57:01 +0000 (19:57 -0700)]
Enforce remote MTU requirement in L2CAP directly
AVCT_BR and BNEP don't need to enforce it locally.
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I090d4e31b76e0098fc77e707ebd2eacf4cd78b7b
Hansong Zhang [Fri, 25 Sep 2020 02:28:17 +0000 (19:28 -0700)]
L2CAP: peer_cfg_bits is unused
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: Icec4e536f28a649779f747d34a7d76295f5ef10f
Hansong Zhang [Fri, 25 Sep 2020 00:50:55 +0000 (17:50 -0700)]
L2CAP: Add a "required_remote_mtu" and store it
For AVCT_BR and BNEP, we need a larger required remote mtu than minimal
L2CAP MTU.
So far we enforce this in profile service itself, but later we can let
L2CAP module enforce this for us.
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I0b4bad4049636bcdc52615c000b1fff3acde360c
Hansong Zhang [Fri, 25 Sep 2020 00:27:49 +0000 (17:27 -0700)]
Remove unused code in BNEP
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I51a6edb1d7f94422657128d0b3d8398a4c25c5de
Hansong Zhang [Thu, 24 Sep 2020 23:56:10 +0000 (16:56 -0700)]
Profile services have const MTU size now
For GAP, the value comes from
`btsock_l2cap_listen_or_connect`. For classic dynamic channel, we use
L2CAP_SDU_LENGTH_MAX for now.
For GATT, we use GATT_MAX_MTU_SIZE, which was used when we send
config request.
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I93b82c718866f09c4c209311c93b65f2dc74b193
Treehugger Robot [Fri, 25 Sep 2020 16:17:26 +0000 (16:17 +0000)]
Merge changes I8b3c9cd4,Iedb0a81a,If7612988,I25f7c702,I23eb6baa, ... am:
1cb3744fa9 am:
30e8505d80
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437485
Change-Id: Ieb2ac7b8c0c81730ffafa9d0fdf39be5af6993fc
Treehugger Robot [Fri, 25 Sep 2020 15:40:13 +0000 (15:40 +0000)]
Merge changes I8b3c9cd4,Iedb0a81a,If7612988,I25f7c702,I23eb6baa, ... am:
1cb3744fa9
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437485
Change-Id: I0820a5e1990b2bc8b0dffe87b213212c4c3a602c
Treehugger Robot [Fri, 25 Sep 2020 15:14:56 +0000 (15:14 +0000)]
Merge changes I8b3c9cd4,Iedb0a81a,If7612988,I25f7c702,I23eb6baa, ...
* changes:
Move decls to ble_hci interface
Locally link and remove unused funcs stack/btm/btm_ble_privacy
Remove duplicate decls stack/btm/btm_ble_int
Move function decls with usage stack/btm::
Explictly provide ble extern declarations
Move stack/btm/btm_ble_addr::btm_ble_refresh_raddr_timer_timeout
Chris Manton [Sun, 20 Sep 2020 00:54:59 +0000 (17:54 -0700)]
Move decls to ble_hci interface
From: stack/btm/btm_ble_int
Towards readable code
Bug:
163134718
Tag: #refactor
Test: acts -tc BleCocTest
Test: ble paired 2 phones
Change-Id: I8b3c9cd43fe3cc34a2c41c53252d5f58990b2dbf
Jakub Pawlowski [Fri, 25 Sep 2020 09:59:48 +0000 (09:59 +0000)]
Merge "Properly pass address type when extended advertisement is received" am:
55b44afafd am:
18638ddba3
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437791
Change-Id: I7342213b2b1e33974ac3a3e9dc5746030c60284d
Jakub Pawlowski [Fri, 25 Sep 2020 09:31:19 +0000 (09:31 +0000)]
Merge "Properly pass address type when extended advertisement is received" am:
55b44afafd
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437791
Change-Id: Iac4fc6bc3c4a14b398c1545e3ae49e40cc095017
Jakub Pawlowski [Fri, 25 Sep 2020 09:12:13 +0000 (09:12 +0000)]
Merge "Properly pass address type when extended advertisement is received"
HsingYuan Lo [Fri, 25 Sep 2020 06:01:11 +0000 (06:01 +0000)]
Merge "Expose service changed event to application (1/3)" am:
b87849b7b1 am:
cf3be7220d
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1433963
Change-Id: I65e3fac22e76fd2263acbaaa52b5be7a62b61a9a
HsingYuan Lo [Fri, 25 Sep 2020 05:51:49 +0000 (05:51 +0000)]
Merge "Expose service changed event to application (1/3)" am:
b87849b7b1
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1433963
Change-Id: If50a9798d140cccaa54f29009cea94afeb991a95
Hansong Zhang [Fri, 25 Sep 2020 05:47:25 +0000 (05:47 +0000)]
AVDTP: Use the same MTU in config request am:
1780e37a0b am:
4f611c2569
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437938
Change-Id: Ia699cc19a4c756a00b1ec73a9ceaf561b9fa31cd
Hansong Zhang [Fri, 25 Sep 2020 05:47:24 +0000 (05:47 +0000)]
Remove unused definition in benp_int am:
cac61e8f34 am:
debd13b9e0
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437937
Change-Id: If6086f58be6a6bc22a0e6370586dda382caf5a83
Hansong Zhang [Fri, 25 Sep 2020 05:47:18 +0000 (05:47 +0000)]
We always send default flush timeout am:
31903b6da7 am:
352f84bb96
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437935
Change-Id: I81657696d4bfab394c91c1c922472622c2c326d3
Hansong Zhang [Fri, 25 Sep 2020 05:47:16 +0000 (05:47 +0000)]
AVDT Flush Timeout is fixed am:
655234d56e am:
3a1caba6b7
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437934
Change-Id: Iddf79c9341bb093c4a4eb845d48447a78f373cce
Hansong Zhang [Fri, 25 Sep 2020 05:47:15 +0000 (05:47 +0000)]
AVDT MTU is fixed am:
94722f9372 am:
9b6deb86e6
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438053
Change-Id: I9a24466b9867a676b5fb4d2754b8eb07b3914854
Hansong Zhang [Fri, 25 Sep 2020 05:47:07 +0000 (05:47 +0000)]
AVRC_BR uses the same mtu am:
810f878b5a am:
5e76768733
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438052
Change-Id: If482a10ffc4d2784f060979a6f0c7859ddb17d88
Hansong Zhang [Fri, 25 Sep 2020 05:47:01 +0000 (05:47 +0000)]
AVRC uses the same mtu (part 2) am:
6e25c080da am:
77baf15141
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437714
Change-Id: I2c7e28af065d988265d4ab449cf4b64bebf43532
Hansong Zhang [Fri, 25 Sep 2020 05:46:59 +0000 (05:46 +0000)]
AVRC uses the same mtu am:
0ae5bb34fd am:
c4d63071f4
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437713
Change-Id: Ia30f5648b42541fe10c79fff94c0d0852c4cc199
Hansong Zhang [Fri, 25 Sep 2020 05:46:52 +0000 (05:46 +0000)]
We can't bypass FCS am:
9bc2e8608f am:
6f4ca5bf59
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437473
Change-Id: I35693002a6a616e76a3b8fbb26031112154eff28
Hansong Zhang [Fri, 25 Sep 2020 05:46:50 +0000 (05:46 +0000)]
p_ccb->bypass_fcs is never satisfied am:
772cf45701 am:
2ed2cc8114
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437472
Change-Id: I8c0e1249c280e00fd875600fc4ffef305de96cbd
Hansong Zhang [Fri, 25 Sep 2020 05:46:48 +0000 (05:46 +0000)]
L2CAP: We never explicitly disable FCS am:
be38658455 am:
5a705e6187
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437471
Change-Id: I034b2adcaccf9f21c0785e6cd5a34c6c209d48b1
Treehugger Robot [Fri, 25 Sep 2020 05:46:21 +0000 (05:46 +0000)]
Merge changes Id7a14929,Ib01088d7,I3c46c8ec,Ib71ed35c am:
cd9e8dc741 am:
be7615441f
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437478
Change-Id: Ic9775697156b593f641d2465ad3c03c59ff9663c
Hansong Zhang [Fri, 25 Sep 2020 05:45:02 +0000 (05:45 +0000)]
AVDTP: Use the same MTU in config request am:
1780e37a0b
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437938
Change-Id: I5e3014b2451932a8fde734b13ad840171437835b
Hansong Zhang [Fri, 25 Sep 2020 05:44:58 +0000 (05:44 +0000)]
Remove unused definition in benp_int am:
cac61e8f34
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437937
Change-Id: Ibd1f295a4b10b602044bd4708c8cf74527a81294
Hansong Zhang [Fri, 25 Sep 2020 05:44:48 +0000 (05:44 +0000)]
We always send default flush timeout am:
31903b6da7
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437935
Change-Id: I66644495f589ca8a4a6a9db849018ce21b58102f
Hansong Zhang [Fri, 25 Sep 2020 05:44:45 +0000 (05:44 +0000)]
AVDT Flush Timeout is fixed am:
655234d56e
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437934
Change-Id: Ia4026c0439007a1dd54bcf26bbc5814446b5adae
Hansong Zhang [Fri, 25 Sep 2020 05:44:41 +0000 (05:44 +0000)]
AVDT MTU is fixed am:
94722f9372
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438053
Change-Id: Ib7dd455bf705071c82f31e3ebb51aa0b26ef047d
Hansong Zhang [Fri, 25 Sep 2020 05:44:30 +0000 (05:44 +0000)]
AVRC_BR uses the same mtu am:
810f878b5a
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1438052
Change-Id: Ia81def4d5e9f6026267a6629a415cf7e4482992c
Hansong Zhang [Fri, 25 Sep 2020 05:44:22 +0000 (05:44 +0000)]
AVRC uses the same mtu (part 2) am:
6e25c080da
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437714
Change-Id: I99d3b36e97f06f5f5c0dc60800af0cb09e9d1cf8
Hansong Zhang [Fri, 25 Sep 2020 05:44:17 +0000 (05:44 +0000)]
AVRC uses the same mtu am:
0ae5bb34fd
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437713
Change-Id: Idbbacce00faeb0a1261093887a907ecba6f16f3a
Hansong Zhang [Fri, 25 Sep 2020 05:44:00 +0000 (05:44 +0000)]
We can't bypass FCS am:
9bc2e8608f
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437473
Change-Id: I9fd3781e45446293fa51a0c76d21c34aae0ea146
Hansong Zhang [Fri, 25 Sep 2020 05:43:56 +0000 (05:43 +0000)]
p_ccb->bypass_fcs is never satisfied am:
772cf45701
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437472
Change-Id: I1f52a14411af18bc5df396dd9c5c18bc9f9fb598
Hansong Zhang [Fri, 25 Sep 2020 05:43:01 +0000 (05:43 +0000)]
L2CAP: We never explicitly disable FCS am:
be38658455
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437471
Change-Id: I84852d90b80d5da0066a200326e658782bf681ec
Treehugger Robot [Fri, 25 Sep 2020 05:26:33 +0000 (05:26 +0000)]
Merge changes Id7a14929,Ib01088d7,I3c46c8ec,Ib71ed35c am:
cd9e8dc741
Original change: https://android-review.googlesource.com/c/platform/system/bt/+/
1437478
Change-Id: I91b7c472cc8c6d82f59a708b16314fc66475c97c
Chris Manton [Sun, 20 Sep 2020 01:22:09 +0000 (18:22 -0700)]
Locally link and remove unused funcs stack/btm/btm_ble_privacy
Towards readable code
Bug:
163134718
Tag: #refactor
Test: acts -tc BleCocTest
Test: ble paired 2 phones
Change-Id: Iedb0a81a01f7c1ac24bc35202be32b9799e2e4e4
Chris Manton [Sun, 20 Sep 2020 00:41:57 +0000 (17:41 -0700)]
Remove duplicate decls stack/btm/btm_ble_int
Towards readable code
Bug:
163134718
Tag: #refactor
Test: acts -tc BleCocTest
Test: ble paired 2 phones
Change-Id: If7612988717edca3d2676635783fbff331552268
Chris Manton [Sun, 20 Sep 2020 00:38:17 +0000 (17:38 -0700)]
Move function decls with usage stack/btm::
Towards readable code
Bug:
163134718
Tag: #refactor
Test: acts -tc BleCocTest
Test: ble paired 2 phones
Change-Id: I25f7c7024c29ac461d4f50db365fb28d97f5f407
Chris Manton [Sun, 20 Sep 2020 01:00:52 +0000 (18:00 -0700)]
Explictly provide ble extern declarations
Towards readable code
Bug:
163134718
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: I23eb6baa71324529a413f20614dc9e4e8ab8bc70
Chris Manton [Sun, 20 Sep 2020 00:22:05 +0000 (17:22 -0700)]
Move stack/btm/btm_ble_addr::btm_ble_refresh_raddr_timer_timeout
From: stack/btm/btm_ble_gap::btm_ble_refresh_raddr_timer_timeout
Group functionality within same file
Towards readable code
Bug:
163134718
Tag: #refactor
Test: acts -tc BleCocTest
Test: ble paired 2 phones
Change-Id: Ida81c42548b377ecada80bfe7e94f0e4fa45721a
HsingYuan Lo [Fri, 25 Sep 2020 04:04:19 +0000 (04:04 +0000)]
Merge "Expose service changed event to application (1/3)"
Hansong Zhang [Thu, 24 Sep 2020 23:45:19 +0000 (16:45 -0700)]
AVDTP: Use the same MTU in config request
MTU in config request indicates how many bytes _we_ can process at most.
Instead of using different number for signalling and audio channel, we
use the same number, which is the greater one . This won't require the
remote side to have a larger buffer size, as they can still send the
same packet size as before. We won't need to worry about memory
implication either, because audio buffer will go away quickly.
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: Iab025784699a77d89d784926d1f393a3cc2fe767
Hansong Zhang [Thu, 24 Sep 2020 23:19:19 +0000 (16:19 -0700)]
Remove unused definition in benp_int
Bug:
159815595
Tag: #refactor
Test: compile & verify basic functions working
Change-Id: Id2ba9e6366a2561249ee844b45a118d219d96b16