| 18a2f354f4bd711d3027dfecfbbebc5eca479761 |
20-Jul-2017 |
jruthe <jruthe@google.com> |
Merge "test_vendor: Fragment tests" am: 7453a5d341 am: b8b17af6b8 am: d0e79aecb5
am: afd2d51955
Change-Id: I1ad4959fcf41c8102c768f073f8ed2ba2bafcc27
|
| 7453a5d3418db79fefe4988c0acfb2725f52a73d |
20-Jul-2017 |
Treehugger Robot <treehugger-gerrit@google.com> |
Merge "test_vendor: Fragment tests"
|
| 3869309f5995599e9254851660fd5f7e826b6aa6 |
19-Jul-2017 |
Myles Watson <mylesgw@google.com> |
test_vendor: Move include to .cc file am: 8769e8e923 am: 8b95e6e958 am: 2e703ac4ad
am: 9c959368a3
Change-Id: I26eaff4a561dd728785b94850c24cb3a1bffa8e0
|
| 8769e8e923b2ef633151421f6edd75a433319afb |
07-Mar-2017 |
Myles Watson <mylesgw@google.com> |
test_vendor: Move include to .cc file
Test: compile
Change-Id: Ib13830f92c305bcd9d7e688d0d4dba78499b1f5b
nclude/async_manager.h
rc/async_manager.cc
|
| a1d82718e0723f569ce1d8479ba7ac6513dbdc3f |
19-Jul-2017 |
jruthe <jruthe@google.com> |
Merge "test_vendor: Implement L2CAP packet fragmentation" am: 3dc90dd77e am: 967f9908d7 am: 525753fab5
am: 91ea3dbd7c
Change-Id: Id548a020d1260652e524875702a82208b32b008b
|
| 365f3a3a327051a07b1ce9f0abd91c71a93be245 |
09-Jun-2017 |
jruthe <jruthe@google.com> |
test_vendor: Implement L2CAP packet fragmentation
Added functions to fragment an L2CAP packet into SDUs of variable size.
Test: Passes all of the tests in test/l2cap_test.cc
Change-Id: Id39042c32594c2e9f54e98def432f8d1655a3b41
nclude/l2cap_packet.h
nclude/l2cap_sdu.h
rc/l2cap_packet.cc
rc/l2cap_sdu.cc
|
| 4e0fd1b388c4412501f3b816cc964f81a1605e44 |
08-Jul-2017 |
jruthe <jruthe@google.com> |
test_vendor: Fragment tests
Test: compiles
Change-Id: I55c121b5ca97801b37315126164aa01ef3db45d0
est/l2cap_test.cc
|
| 9dbec4867e9a8ad1c2c282ef53b6064d04078598 |
18-Jul-2017 |
jruthe <jruthe@google.com> |
test_vendor: Add LOG_DEBUG to L2cap::assemble am: 8d604d6ab0 am: 8fb366ebdc am: b521bff69c
am: 0edb81ada7
Change-Id: I6d699624a71389f699949200136bd5092ae23710
|
| e8bd07fe7641123b5770b9cd5495dad7da36bee8 |
17-Jul-2017 |
TreeHugger Robot <treehugger-gerrit@google.com> |
Merge "Make system/bt build with the VNDK."
|
| 8d604d6ab0543c9328b19cb8c554b81890fd335b |
17-Jul-2017 |
jruthe <jruthe@google.com> |
test_vendor: Add LOG_DEBUG to L2cap::assemble
Test: compiles
Change-Id: I8753215d00610dfe3ef43a09108362f94402c941
rc/l2cap_packet.cc
|
| 41401a5916f750f29af61bae35ca99f7e851333f |
17-Jul-2017 |
Steven Moreland <smoreland@google.com> |
Make system/bt build with the VNDK.
When the vndk is enabled (BOARD_VNDK_VERSION=current),
vendor libraries must only link against vendor variants
in the build system. This also means that they receive
restricted sets of headers and that automatic global
headers are restricted.
Test: build system/bt with BOARD_VNDK_VERSION=current
Bug: 37342627
Change-Id: Ie1507e9b5bdb626fa0049faadd225c2c1f451661
ndroid.bp
|
| 7d267611a407e2247024982a8395afa625fca303 |
14-Jul-2017 |
jruthe <jruthe@google.com> |
test_vendor: Move SDU checks to L2capSdu
Move functions that check the Segmentation and Reassembly bits of the
control field of SDU packets into the L2capSdu class from the
L2capPacket class.
Test: compiles
Change-Id: I0afc60fcf30d29b5021ddd9afa1c7320fca64969
nclude/l2cap_packet.h
nclude/l2cap_sdu.h
rc/l2cap_packet.cc
rc/l2cap_sdu.cc
|
| 7552c35aec3d261d6af51db74dc7e5a431736dfc |
14-Jun-2017 |
johnshamoon <johnshamoon@google.com> |
test_vendor: Add L2cap and L2capSdu assemble tests
Test: Run the unit tests:
out/host/linux-x86/nativetest64/test-vendor_test_host/\
test-vendor_test_host --gtest_filter=L2capTest.*
Change-Id: I89ff2e6bbcfb58c53d872bfef007e487c2f4e3ae
ndroid.bp
nclude/l2cap_packet.h
nclude/l2cap_sdu.h
nclude/l2cap_test_packets.h
rc/l2cap_sdu.cc
est/l2cap_sdu_test.cc
est/l2cap_test.cc
|
| 16995c3a50a4b74e6e2f233207fdea0d94f240da |
11-Jul-2017 |
jruthe <jruthe@google.com> |
test_vendor: L2cap Assemble additional error check
Assemble now properly checks the proposed length of a segmented SDU that
is given in the first packet of any segmented SDU in accordance with
Bluetooth Core Specification Version 4.2 Volume 3 Part A 3.3.2
Test: Passed in test/l2cap_test.cc
Change-Id: Idbbd1c014bac8928affab581118f73a7d9126824
rc/l2cap_packet.cc
|
| d930fac3dfa31ce5761cd2106e73bac2340d8f60 |
14-Jul-2017 |
jruthe <jruthe@google.com> |
test_vendor: Rename L2cap class to L2capPacket
Test: Compiles
Change-Id: I851d2af8bfeb906cee754531e72e94a832413194
ndroid.bp
nclude/l2cap.h
nclude/l2cap_packet.h
rc/l2cap.cc
rc/l2cap_packet.cc
|
| 96ad84508fbdb33667d2801ce69112edfb397fd0 |
09-Jun-2017 |
jruthe <jruthe@google.com> |
test_vendor: Implement packet assembly for SDUs
Created l2cap.* and l2cap_sdu.* for assembling an L2CAP packet from a
stream of SDU packets.
Test: Passed all tests in l2cap_test.cc and l2cap_sdu_test.cc
Change-Id: I64487ee67170b1dd4eda9555321b1ecf7ed57040
ndroid.bp
nclude/l2cap.h
nclude/l2cap_sdu.h
rc/l2cap.cc
rc/l2cap_sdu.cc
|
| b707f447cbb916de2e0dfd2b4e9bf15818376e64 |
04-Jul-2017 |
Jakub Pawlowski <jpawlowski@google.com> |
Make RawAddress into a class (1/3)
* Add libbluetooth-types - library containing types implementation, that
is common between stystem/bt and packages/apps/Bluetooth. It must be
included in every project using btif interface.
* Put Raw Address implementation into libbluetooth-types
* Unify all "to/from string" helper methods into ToString and FromString
* bd_addr_empty -> RawAddress::kEmpty
* bd_addr_any -> RawAddress::kAny
Also fix leaks in jni str2addr by adding ReleaseStringUTFChars
Test: types_unittest
Change-Id: Ie0694843ad5fbd2a80b310c5f532e5e5a9548043
Merged-In: Ie0694843ad5fbd2a80b310c5f532e5e5a9548043
ndroid.bp
|
| 10b43d2fa0f408dbe465febb27aace0e111b01ff |
17-Apr-2017 |
Myles Watson <mylesgw@google.com> |
test_vendor: Remove unused libbt-vendor files
Bug: 36810308
Test: builds
Change-Id: I50b4a3efaac622959dbf25c0f854d455057e6f3e
ndroid.bp
nclude/dual_mode_controller.h
nclude/hci_transport.h
nclude/vendor_manager.h
rc/dual_mode_controller.cc
rc/hci_transport.cc
est/hci_transport_unittest.cc
|
| e59cbd20e14618888908547d58767aa5fc3b35d7 |
07-Mar-2017 |
Myles Watson <mylesgw@google.com> |
Finish conversion to Android.bp
Bug: 35651902, 35672576, 36810308
Test: build, Android Cloud Bluetooth net_test_bluetooth
Change-Id: Ie99966d610ea35ab8caf8a084d92c43618e90b94
ndroid.mk
UILD.gn
rc/bt_vendor.cc
rc/vendor_manager.cc
|
| 73ad0310835f00d6c725091de990c271544c49a9 |
07-Mar-2017 |
Myles Watson <mylesgw@google.com> |
test: Add a simulated Bluetooth HAL implementation
Bug: 35672576
Test: Angler Bluetooth starts/stops with test HAL
Android Cloud net_test_bluetooth tests pass
Change-Id: I104e0dd3946f0efff9709e05c056bee3538394f7
ndroid.bp
|
| 77c6bc5886da34011fc6b6bfcc15cf19dc4b5b68 |
28-Mar-2017 |
Jorge E. Moreira <jemoreira@google.com> |
test-vendor-lib: Enable building for 64 bits
Removes LOCAL_MODULE_PATH which is not supported for 64 bit builds.
Uses LOCAL_PROPRIETARY_MODULE to let the build system know it should
go in the vendor directory.
Test: builds
Change-Id: Iba55d497d8cbe97911a41a5623248306a8954ec2
ndroid.mk
|
| 01ce9e4233561dc55e05389959ad9d5a5db25d44 |
15-Mar-2017 |
Myles Watson <mylesgw@google.com> |
test_vendor: Stop watching before closing the FD
Test: net_test_bluetooth BluetoothTest.AdapterRepeatedEnableDisable
Change-Id: I72f5c9105b535ad5a5c41bc15dc9a3ad5f4e023f
rc/vendor_manager.cc
|
| 8cd930cc9d2ba9abeaf15ee764b738144084504c |
13-Dec-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Replace __FUNCTION__ with __func__
Test: Bluetooth settings still works
Change-Id: I4de24cceb9beb262ce7f96e7291288b70618f694
rc/async_manager.cc
|
| 21b6e60b93e9b9f270ea9a07b8e1e2377897e2b5 |
17-Feb-2017 |
Myles Watson <mylesgw@google.com> |
test_vendor: Return success for no-ops
Bug: 35408667
Test: Marlin with debug library BT starts and stops
Change-Id: Idd95115c17ca2fed983526ea47a44799e7de6e0c
rc/bt_vendor.cc
|
| e36d707b00e5a6c011e5ba41d67a82374a5cfab5 |
27-Jan-2017 |
Myles Watson <mylesgw@google.com> |
test_vendor: Test fixes for AsyncManager
Test: unit tests pass
Change-Id: I5c3f9f4a1fa86e97fa9e6cc26160fd1e3e854925
est/async_manager_unittest.cc
|
| a72fc53164bc66b44e1a5d04b741df5247b85b17 |
19-Jan-2017 |
Myles Watson <mylesgw@google.com> |
test_vendor: Remove extra byte in LE Features
Bug: 34357009
Test: push test-vendor lib as libbt-vendor.so, Bluetooth starts
Change-Id: Id3a5f85f6b3da5a723dba04527db8b66de0e84a8
rc/event_packet.cc
|
| f563ba49a3bc20ab7e99762506e4dbc8a866552c |
19-Jan-2017 |
Jorge E. Moreira <jemoreira@google.com> |
test_vendor: Make tryStartThread return 0 on success
Test: Start the test vendor library with no error message
Bug: 34357009
Change-Id: Id2821199ad675e2b95a9f333afa803dcced811a3
rc/async_manager.cc
|
| e2eeff4f7f149c59ea8cb67392f287317f40f9bd |
08-Dec-2016 |
Jack He <siyuanh@google.com> |
Convert build system to soong using Android.bp
* Convert top level Android.mk into build templates in build/Android.bp
and build/fluoride.go
* Initial conversion is done by "androidmk Android.mk > Android.bp"
* Android.bp does not allow source inclusion from external directories
and therefore they have to be made in to cc_library_static in their
respective sub-directories and linked using whole_static_libs in the
modules where they are used
* As Android.bp does not allow multiple modules of the same name,
same-name mudules for different target are merged into one definition
with target specific setup
* Generated proto header path has to be changed in osi/src/metrics.cc as
Android.bp only generate header path relative to the Android.bp file
instead of top-level directory such as system/bt
* Android.bp does not support resource copying yet and hence conf files
are left un-touched.
* Android.bp does support conditional module declaration and therefore
test-vendor libs are left untouched except for unit tests
* The goal of this CL is to direct (almost) translate Android.mk to
Android.bp first with Android.bp specific optimizations coming later
Bug: 32958753
Test: Code compilation, manual testing by test team
Change-Id: I5249e1f2135c4121205619b1d735ce448feb7499
ndroid.bp
ndroid.mk
|
| 15915b14150e34bc14d0eb1ace0330129f4c9f8e |
16-Dec-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Use pipe2 to set pipe flags
Simplify the code to fix a typo. F_SETFD should have been
F_SETFL.
Test: run unit tests
out/host/linux-x86/nativetest/test-vendor_test_host/test-vendor_test_host \
--gtest_filter=AsyncManagerSocketTest.*
Change-Id: I215381d6e130428acae7d73d80fefc97cbf64651
rc/async_manager.cc
|
| e1fbea969594b78e404a5aa49aff608a7ed81ee6 |
16-Dec-2016 |
Treehugger Robot <treehugger-gerrit@google.com> |
Merge "Replace assert with CHECK from base/logging.h"
|
| bed69b9bcdae37d6e4ed64194c37e388dadfbcc7 |
08-Dec-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Add more unit tests for async_manager
Test: run the unit tests
out/host/linux-x86/nativetest/test-vendor_test_host/test-vendor_test_host \
--gtest_filter=AsyncManagerSocketTest.*
Change-Id: I8c5e05c5506a067a6148d54dc23bbb931ad612e7
est/async_manager_unittest.cc
|
| f2af1c42ccb2f642b241c2261b42d0be61d45438 |
13-Dec-2016 |
Jack He <siyuanh@google.com> |
Replace assert with CHECK from base/logging.h
* Replace assert with CHECK
* Remove all NDEBUG definitions
* Remove hacks for BT_LIBCHROME_NDEBUG
* Removed some removed directories from Makefile such as hcis, brcm, rpc
Coccinelle-assisted:
@@
@@
- #include "base/logging.h"
+ #include <base/logging.h>
@ assert_included @
@@
@ base_logging_included @
@@
@ depends on (assert_included && !(base_logging_included)) @
@@
- #include <assert.h>
+ #include <base/logging.h>
@ depends on (assert_included && base_logging_included) @
@@
- #include <assert.h>
@@
expression E;
@@
- assert(E);
+ CHECK(E);
And a bash script:
for file in $(find . -name "*.cc"); do
spatch --sp-file replace_assert_with_CHECK.cocci --in-place $file
done
The following files are maually edited:
btif/src/btif_config.cc
btif/src/btif_avrcp_audio_track.cc
btif/src/btif_gatt_client.cc
osi/src/data_dispatcher.cc
osi/src/reactor.cc
osi/src/thread.cc
osi/src/fixed_queue.cc
osi/src/list.cc
osi/src/allocation_tracker.cc
osi/src/alarm.cc
osi/test/wakelock_test.cc
Bug: 31781465
Test: Code compilation, Unit Tests, BtStressTest, BtFunhausMetricsTest
Change-Id: I21dc10a45be31665e41441b75b0515ed87523988
rc/bt_address.cc
rc/bt_vendor.cc
rc/dual_mode_controller.cc
rc/hci_transport.cc
rc/packet.cc
rc/packet_stream.cc
rc/test_channel_transport.cc
rc/vendor_manager.cc
|
| 8b893585da5a112fe6210aace8886134e55d088c |
11-Dec-2016 |
Jorge E. Moreira <jemoreira@google.com> |
test-vendor: Fixed a race condition on the destructor
of AsyncManager.
Test: Run the unit tests
Change-Id: Icfbaf96b40f10f57e7c925422985ed7004adea64
nclude/async_manager.h
rc/async_manager.cc
|
| 911d1ae03efec2d54c3b1b605589d790d1745488 |
29-Nov-2016 |
Myles Watson <mylesgw@google.com> |
Apply clang-format to the rest of the tree
find * | grep "\.[ch]" | xargs clang-format --style=file -i
Test: mma -j32
Change-Id: I6fcc9862bb7bc07c2a367ca58fef2b3cd27a6f05
nclude/bt_address.h
est/hci_transport_unittest.cc
|
| 029a43b78ecb2ad1d400ba8c708e18f2f56f0db4 |
20-Oct-2016 |
Jakub Pawlowski <jpawlowski@google.com> |
Various small fixes for Linux build
* add "-latomic" which was missing since we moved to clang
* empty LOG_EVENT_INT stub added for Linux
* add link libraries to fix stack_unittest
* add missing headers
Test: build with ninja
Change-Id: I1fe0b0b615db1da5ad083c162cb207b08979ce15
UILD.gn
nclude/bt_address.h
rc/async_manager.cc
est/hci_transport_unittest.cc
|
| 015834df156be6b7365ce1cf00e31ddf72f2c67b |
30-Sep-2016 |
Myles Watson <mylesgw@google.com> |
vendor_lib: Apply clang-format with the new config
cd vendor_libs/test_vendor_lib/
clang-format ../linux/bt_vendor_linux.c -style=file -i
clang-format include/* test/* src/* -style=file -i
Update the style guide since clang-format help uses the
-style=file option.
Test: mm -j32
Change-Id: I166adfb2bb9a87fefa1242c91e5f180011caa2ad
nclude/dual_mode_controller.h
nclude/event_packet.h
nclude/packet_stream.h
rc/async_manager.cc
rc/bt_address.cc
rc/bt_vendor.cc
rc/command_packet.cc
rc/dual_mode_controller.cc
rc/event_packet.cc
rc/hci_transport.cc
rc/packet.cc
rc/packet_stream.cc
rc/test_channel_transport.cc
rc/vendor_manager.cc
est/hci_transport_unittest.cc
est/packet_stream_unittest.cc
|
| 3c4f59fc27e36e4489c7f7fd6e330819b16fbc4f |
17-Sep-2016 |
Jorge E. Moreira <jemoreira@google.com> |
test_vendor: Refactor scheduling of dual_mode_controller event handling
Moved all controller actions out of read callbacks to task callbacks to
avoid race conditions when modifying internal controller state.
Change-Id: Ie8bb762992755db2dddb6b7bb6a4fe52ab2d0020
nclude/command_packet.h
nclude/dual_mode_controller.h
nclude/hci_transport.h
nclude/packet_stream.h
rc/dual_mode_controller.cc
rc/hci_transport.cc
rc/packet_stream.cc
rc/vendor_manager.cc
est/packet_stream_unittest.cc
|
| b2797fc48b017097823b674095fd828699f8511f |
29-Sep-2016 |
Victor Khimenko <khim@google.com> |
Support TARGET_TRANSLATE_2ND_ARCH case
Build-only change to support the multilib case where the second arch
is translated (and thus bluetooth library is needed).
"True" multilib case is explicitly not supported.
Test: Build system refactoring CL. Existing unit tests still pass.
BUG=31422117
Change-Id: I09f239d39f5dbe0848a89367327db1ea1074ca39
ndroid.mk
|
| 6d835905f00b14eddd57638a07eec9fb4229d7cc |
20-Sep-2016 |
Jorge E. Moreira <jemoreira@google.com> |
test_vendor: Use kInvalidTaskId in the controller
Test: make test-vendor
Change-Id: I6eefd385ecb992f854544ca92c5a8559b6f281f0
nclude/async_manager.h
rc/async_manager.cc
rc/dual_mode_controller.cc
|
| 354a9bf886b2100f484cd0217d00aa4ef008759f |
14-Sep-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Remove unnecessary const from params
Change-Id: I6502e3e78f139722bb887640dcc00ee79f1b0044
nclude/dual_mode_controller.h
nclude/event_packet.h
nclude/packet.h
rc/dual_mode_controller.cc
rc/event_packet.cc
rc/packet.cc
|
| f6c30e115c482425befa343558f1cdd93f1c1eaa |
13-Sep-2016 |
Treehugger Robot <treehugger-gerrit@google.com> |
Merge "test_vendor: White space fix in vendor manager"
|
| 85dba1c476d94468901a1c4dc66a129140b3eab4 |
13-Sep-2016 |
Treehugger Robot <treehugger-gerrit@google.com> |
Merge "test_vendor: Silence packet_stream.cc"
|
| 52498170cf332031f2931f4b71eb43edf2478f84 |
10-Sep-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: White space fix in vendor manager
Change-Id: I00f2e40ba29a84fd3733b99abafc8e5b9961de7d
rc/vendor_manager.cc
|
| d4ff4bb2c7c6d16945456b2ba2ccdeae045e8cbd |
13-Jul-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Silence packet_stream.cc
Only log if Events report the wrong size.
Change-Id: I3c06e94298f7341a940bb6d5106c869746bdaa78
rc/packet_stream.cc
|
| 9cd5cd2856b3f6c2ec8008e6e8e6145af7d253b9 |
13-Sep-2016 |
Jorge E. Moreira <jemoreira@google.com> |
test_vendor: Move checks for vendor_manager_
Only some vendor operations use vendor_manager_. Move the checks into
the operations that use it.
Change-Id: I8bb23d740c94df7e2081c29b1d9e062c5c012714
rc/bt_vendor.cc
|
| b7d64b20717e3f597bb4e3881f08cb5d07fa1b02 |
29-Aug-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Remove using directives from headers
Change-Id: I83358cbb68b10e80e4e553a6657fcb38cfcb928f
Signed-off-by: Myles Watson <mylesgw@google.com>
nclude/command_packet.h
nclude/dual_mode_controller.h
nclude/event_packet.h
nclude/packet.h
nclude/packet_stream.h
nclude/test_channel_transport.h
rc/command_packet.cc
rc/dual_mode_controller.cc
rc/event_packet.cc
rc/packet.cc
rc/packet_stream.cc
rc/test_channel_transport.cc
est/hci_transport_unittest.cc
|
| 3be0c20a70a0891d495916e6b733f7805300db25 |
29-Aug-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Use BtAddress instead of vectors
Change-Id: I229f2a4f93a9219a502960e6d91f514738f3b3c8
Signed-off-by: Myles Watson <mylesgw@google.com>
nclude/dual_mode_controller.h
nclude/event_packet.h
nclude/packet.h
rc/dual_mode_controller.cc
rc/event_packet.cc
rc/packet.cc
|
| 265316f78ca1077216c9d87b2de096894fdd5ee5 |
29-Aug-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Remove unused extern statements
It should be safe to include C header files without
using extern.
Change-Id: I2b562ef3f76df9f9dac69323345ed3d3610e8bc2
Signed-off-by: Myles Watson <mylesgw@google.com>
nclude/packet.h
rc/command_packet.cc
rc/dual_mode_controller.cc
rc/event_packet.cc
rc/packet.cc
rc/packet_stream.cc
rc/test_channel_transport.cc
est/hci_transport_unittest.cc
est/packet_stream_unittest.cc
|
| caa53d094c810a036b594e2ab30a3d01ababed30 |
16-Jun-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Refactor controller timing
Use AsyncManager to implement a timer tick and an event queue.
Change-Id: Iec2a0ef06f17ecce0652ecc52f5ef732fc4af026
Signed-off-by: Myles Watson <mylesgw@google.com>
nclude/dual_mode_controller.h
nclude/hci_transport.h
rc/dual_mode_controller.cc
rc/hci_transport.cc
rc/vendor_manager.cc
est/hci_transport_unittest.cc
|
| e3ab96a689498c14885bfba6635a245d11542311 |
20-May-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Refactor test channel
The test channel can always be enabled when using the AsyncManager.
- Remove enabled_ and port_
- Allow multiple connections
- Pass file descriptors as parameters instead of private variables
- Add a clean up function to the Test Channel
- Add a static class in bt_vendor.cc and always EXPORT the entry point
- Move the vendor callbacks and the global pointer to the vendor
manager object from vendor_manager to bt_vendor
Change-Id: I3e0dee846eb89f434893603a705c2b13219272be
Signed-off-by: Myles Watson <mylesgw@google.com>
Signed-off-by: Jorge E. Moreira <jemoreira@google.com>
ndroid.mk
nclude/test_channel_transport.h
nclude/vendor_manager.h
rc/bt_vendor.cc
rc/test_channel_transport.cc
rc/vendor_manager.cc
|
| b3a80b682f85678da3869c6f0747c7dd6840ffb8 |
22-Jun-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Add BtAddress unit tests
Change-Id: I23d27a1a5c258de330f4eac23d314ef8b9200d2a
Signed-off-by: Myles Watson <mylesgw@google.com>
ndroid.mk
est/bt_address_unittest.cc
|
| 6f8593ec98041ec4fe4406d72d013da05c0ee68e |
07-Jun-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Add a BtAddress class
The class provides IsValidBtAddress(string), functions to convert
from vectors and strings, and some operators.
Change-Id: I553c47e698ec46ed0e24656f89141c23f7369380
Signed-off-by: Myles Watson <mylesgw@google.com>
ndroid.mk
nclude/bt_address.h
rc/bt_address.cc
|
| 5f7514d934703adfc545e241c8d8dacaf1f7f72c |
25-Aug-2016 |
Jakub Pawlowski <jpawlowski@google.com> |
Update linux BUILD.gn files
Change-Id: Iccec195d35b5a62d7b45834670229a68deeb4add
UILD.gn
|
| 8f981328adcf677a51857282f311f5efa3bed936 |
22-Jun-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Build independently from the stack
Remove the dependency on bluetooth_* flags.
Enable more warnings.
(Based on the build flags in system/bt/Android.mk)
Change-Id: I5f480f15c037955404502a0216b61aa407670163
Signed-off-by: Myles Watson <mylesgw@google.com>
ndroid.mk
|
| 98dcd0babf97d7be1f746bb7ac5750386ce4a59e |
24-Aug-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Fix style for UNUSED_ATTR, whitespace
Remove "/* args */" and use UNUSED_ATTR. It's easier to find using grep.
Change-Id: Ic1a95e44b86d019647acafe982cfef5c69e1da1a
Signed-off-by: Myles Watson <mylesgw@google.com>
nclude/dual_mode_controller.h
nclude/hci_transport.h
rc/dual_mode_controller.cc
rc/hci_transport.cc
|
| a11e1c1b71fc0a81a926e8df61f1b3962d10b2ae |
13-Jun-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor: Allow includes from bt_hci_defs
Avoid duplicating header files for BT constants.
Update Android.mk to search $(BT_DIR)/include and
$(BOARD_BLUETOOTH_BDROID_BUILDCFG_INCLUDE_DIR)
Change-Id: I1403724c1cf9cda8156a5b65cf55da8178bd910f
Signed-off-by: Myles Watson <mylesgw@google.com>
ndroid.mk
|
| d4f80ae7ca433e74a1f841e04c66a29ef1830979 |
19-Aug-2016 |
Myles Watson <mylesgw@google.com> |
Add a socket unit test for the async_manager
This reverts commit 10986700c302f79376894ebef9f2ec92a65b8b08.
Which reverted commit 1387d4330da52034af67d1dbc61b870e8f89c820.
Some minor style changes as well.
Change-Id: I83ea40b4cf96adfa58c24d0cd9109ed9974519f1
ndroid.mk
est/async_manager_unittest.cc
|
| b2bbe93c1b8c5f83e8c5f0597a836f339eeea5e8 |
16-Aug-2016 |
Jorge E. Moreira <jemoreira@google.com> |
Added cancelling for fd watching on async manager
Change-Id: I4107b3b50345f81833c47c85c4a322a582973bb2
nclude/async_manager.h
rc/async_manager.cc
|
| 10986700c302f79376894ebef9f2ec92a65b8b08 |
18-Aug-2016 |
Myles Watson <mylesgw@google.com> |
Revert "Added unit test for async_manager with sockets"
This reverts commit 1387d4330da52034af67d1dbc61b870e8f89c820.
Change-Id: I4af4e0d5ac5a108f3de44d186db97592a70c729e
ndroid.mk
est/async_manager_unittest.cc
|
| 1387d4330da52034af67d1dbc61b870e8f89c820 |
17-Aug-2016 |
Jorge E. Moreira <jemoreira@google.com> |
Added unit test for async_manager with sockets
Change-Id: I8d31f766722e4b4a6c6fe00e30a957ac43176cf1
ndroid.mk
est/async_manager_unittest.cc
|
| 5dc0d15e7865104d13be75c06de4b9158ce7dff4 |
17-Aug-2016 |
Chih-Hung Hsieh <chh@google.com> |
Fix clang-tidy warnings in bt.
* Add explicit keyword to conversion constructors,
or add NOLINT to implicit conversion constructors.
Bug: 28341362
* Use const reference type for read-only parameters,
or add NOLINT to keep same callback function signature.
Bug: 30407689
* Use const reference type for loop index variables to avoid unnecessary copy.
Bug: 30413223
* Use const reference type to avoid unnecessary copy.
Bug: 30413862
Test: build with WITH_TIDY=1
Change-Id: I8237d332f32d7be09ab72a8de82160fdb776dcd1
nclude/dual_mode_controller.h
nclude/event_packet.h
nclude/hci_transport.h
nclude/test_channel_transport.h
rc/async_manager.cc
rc/dual_mode_controller.cc
rc/event_packet.cc
rc/hci_transport.cc
rc/test_channel_transport.cc
|
| 7225e499e81c266f4abba84e2c574cb12eaa2d21 |
17-Aug-2016 |
Jorge E. Moreira <jemoreira@google.com> |
Fix incorrect fd used for select() call in vendor lib async_manager
Change-Id: Iabb8389ee6fcef0f9c6a17838d6a3a9a517befc6
rc/async_manager.cc
|
| 0b32c8d8c1ad2e85d21d00fcd4c270e1ee7f8e25 |
08-Aug-2016 |
Jorge E. Moreira <jemoreira@google.com> |
Refactor vendor library to use async manager class
Homogenizes the use of lambdas/binds (uses c++ lambdas only)
Uses STL's time library implementation (std::chrono)
Change-Id: I6194b26c0a7fabffddf96acb5c2379ff52026a84
nclude/dual_mode_controller.h
nclude/hci_transport.h
nclude/test_channel_transport.h
nclude/vendor_manager.h
rc/dual_mode_controller.cc
rc/hci_transport.cc
rc/test_channel_transport.cc
rc/vendor_manager.cc
est/hci_transport_unittest.cc
|
| 08c214154db84c1d7d3cb8a4bd81e34dd71a2068 |
08-Aug-2016 |
Jorge E. Moreira <jemoreira@google.com> |
Added the async manager
Change-Id: I968fee42fd610e33e13bf358f54bec1fe7af4a7d
ndroid.mk
nclude/async_manager.h
rc/async_manager.cc
|
| 65ee728284db16e1562fbbd879f6ccae3ef7d0fa |
10-Aug-2016 |
Jorge E. Moreira <jemoreira@google.com> |
Added missing #include directives
Change-Id: Ia26cd6775699c65a75d94003c09730926835f7a3
nclude/event_packet.h
nclude/test_channel_transport.h
|
| 97980c25a1160f77309361a885d9888f0d85bae7 |
05-Aug-2016 |
Jorge E. Moreira <jemoreira@google.com> |
Replace calls to std::bind by lambda expressions
Change-Id: I5fbe6ae6d5172e87428322084276e593a67fd7d1
rc/dual_mode_controller.cc
rc/vendor_manager.cc
est/hci_transport_unittest.cc
|
| 83cf95e0a1f79c2e18aa3f5fc65c71ecb2665dbc |
05-Aug-2016 |
Jorge E. Moreira <jemoreira@google.com> |
No longer uses the macro DISALLOW_COPY_AND_ASSIGN
Change-Id: I3403cdfa38fabf56ffe0854064eeb0ae38eb5a4a
nclude/command_packet.h
nclude/dual_mode_controller.h
nclude/hci_transport.h
nclude/test_channel_transport.h
nclude/vendor_manager.h
|
| ea3ede2474a8980387b67bc7db2b661e7219b468 |
05-Aug-2016 |
Jorge E. Moreira <jemoreira@google.com> |
Using include paths relative to vendor library dir
Change-Id: I6b5dfa4098cdf2eda21d3fae60729b765532c2f9
nclude/command_packet.h
nclude/dual_mode_controller.h
nclude/event_packet.h
nclude/hci_transport.h
nclude/packet_stream.h
nclude/vendor_manager.h
rc/command_packet.cc
rc/dual_mode_controller.cc
rc/event_packet.cc
rc/hci_transport.cc
rc/packet.cc
rc/packet_stream.cc
rc/test_channel_transport.cc
est/hci_transport_unittest.cc
est/packet_stream_unittest.cc
|
| 03c2c7c22ad965802bb7bd64528b7d111a204270 |
26-Jul-2016 |
Greg Hartman <ghartman@google.com> |
Implements some additional commands
- Added/Improved responses to certain opcodes on bt_vendor.cc
- Added handlers for a couple of commands used in L
Change-Id: I85c02553057e4ea3a3e51795c2e1a2bb679fd5e1
nclude/dual_mode_controller.h
rc/bt_vendor.cc
rc/dual_mode_controller.cc
|
| 6bfe283db7287624b3333fde2a9b48f5bdb913d1 |
26-Jul-2016 |
Greg Hartman <ghartman@google.com> |
Allow the Bluetooth test library to be used as a vendor library
Change-Id: I479fd72482399f878ef045271c7b493c94aa5ab7
ndroid.mk
rc/bt_vendor.cc
|
| 3398730eac95049f01da0da53629ae77d212857a |
01-Jul-2016 |
Chih-Hung Hsieh <chh@google.com> |
Fix google-explicit-constructor warnings in system/bt.
* Declare conversion constructors as 'explicit'.
* Add NOLINT to implicit converters.
Bug: 28341362
Change-Id: I81fecf19c0ee2e64dd6778b3dea78e04f111bfb4
Test: build with clang-tidy
nclude/command_packet.h
nclude/dual_mode_controller.h
nclude/event_packet.h
nclude/hci_transport.h
|
| 99195bc17fe1f7c4a7e43590d9bce550b177a931 |
26-May-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor_lib: refactor packet constructors
Make it simpler to append bytes to payloads, since that's a common
operation.
Change-Id: I7fc975fb999add8e1a360b91bdae869d6d7d2029
Signed-off-by: Myles Watson <mylesgw@google.com>
nclude/command_packet.h
nclude/dual_mode_controller.h
nclude/event_packet.h
nclude/packet.h
nclude/packet_stream.h
nclude/test_channel_transport.h
rc/command_packet.cc
rc/dual_mode_controller.cc
rc/event_packet.cc
rc/packet.cc
rc/packet_stream.cc
rc/test_channel_transport.cc
est/hci_transport_unittest.cc
est/packet_stream_unittest.cc
|
| cb3c6f3cd0b908e36443d41c2d736b88d742ee1a |
01-Jun-2016 |
Luis Hector Chavez <lhchavez@google.com> |
Merge "bt: Update libchrome APIs to r395517"
|
| 63b0519c3f0d1c0cc523e0e9553c60fc68651acc |
20-May-2016 |
Chih-Hung Hsieh <chh@google.com> |
Fix misc-macro-parentheses warnings
When compiled with WITH_TIDY=1, clang-tidy warns about
missing parameters around macro parameters.
Bug: 28705665
Change-Id: I89c23cfe6eaefae5ef3c648a7c123c0501662c13
rc/event_packet.cc
|
| 0fb335fc21296b2489e03eead682efba55c5690e |
21-May-2016 |
Luis Hector Chavez <lhchavez@google.com> |
bt: Update libchrome APIs to r395517
The new libchrome has been ported from Chromium and some APIs have
changed. Make necessary changes at call sites.
Change-Id: Idb7365dd5b0b66c3926fc07b2264c9bc3477ae20
rc/dual_mode_controller.cc
rc/hci_transport.cc
|
| f9b477fca5b1518d0171a8dae2146e95dd26eb25 |
24-May-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor_lib: Add initial support for BLE
Change-Id: If0d8e07ee0c76ed5f22c5f111b7e7be7f28c7bc3
nclude/dual_mode_controller.h
rc/dual_mode_controller.cc
|
| e49120d722142f593687856a71cbffadc3caa530 |
25-May-2016 |
Myles Watson <mylesgw@google.com> |
test_vendor_lib: run clang-format with brillo opts
Change-Id: Ie2ead0a020e2358696a2f0e26f3c76a9dd29e266
Signed-off-by: Myles Watson <mylesgw@google.com>
nclude/dual_mode_controller.h
nclude/event_packet.h
nclude/hci_transport.h
nclude/packet_stream.h
nclude/test_channel_transport.h
rc/bt_vendor.cc
rc/dual_mode_controller.cc
rc/event_packet.cc
rc/hci_transport.cc
rc/packet_stream.cc
rc/test_channel_transport.cc
rc/vendor_manager.cc
est/hci_transport_unittest.cc
est/packet_stream_unittest.cc
|
| c3763020edb86027a5627154e9fec5d85e26e33e |
22-Apr-2016 |
Dan Albert <danalbert@google.com> |
Fix 64-bit build break.
Master (aosp and internal) has been broken for nearly two days now.
Was broken by https://android-review.googlesource.com/#/c/212244/
because no one checked 64-bit before merging.
No one noticed post-submit because the build servers aren't
checkbuilding any 64-bit targets (aside from mips64, which is broken
for other reasons).
I've uploaded a build change to get treehugger building a 64-bit
target (http://cr/120564724), and also to make all the targets
checkbuilds again (http://cr/120568746).
Change-Id: I195000284de5b52a539a47ad8b308990e72eea96
rc/test_channel_transport.cc
|
| aabb3008aa03ad95339cf1a818c025301cbc14d3 |
20-Apr-2016 |
Bluetooth Build Test <test@build.com> |
Enable test_vendor_lib compilation on Linux
UILD.gn
rc/test_channel_transport.cc
|
| 1c64d03f760cba282e0570f2d4f34fa86b79f309 |
18-Apr-2016 |
Cristian Iorga <cristian.iorga@intel.com> |
vendor_libs: test_vendor_lib: fix build
Change-Id: I1ba531c5289adea3fa5d6f9e0f68cf241a127219
Signed-off-by: Cristian Iorga <cristian.iorga@intel.com>
ndroid.mk
nclude/test_channel_transport.h
rc/dual_mode_controller.cc
rc/hci_transport.cc
rc/test_channel_transport.cc
rc/vendor_manager.cc
|
| d7a05a0c5bead54b50a70ffe50d3f371a57ae7ff |
23-Feb-2016 |
Jakub Pawlowski <jpawlowski@google.com> |
Update build files for Fluoride on Linux
uprev build/secondary/third_party/**/BUILD.gn"
Other BUILD.gn files were updated to match current state of project.
Change-Id: Ie895d25680d510f5e69e6f15553311f6671cdce3
UILD.gn
|
| 1377f938abf30ab3104571fb8372c657e65237d0 |
12-Feb-2016 |
Pavlin Radoslavov <pavlin@google.com> |
Cleanup C and C++ compiler flags
Cleanup the setting of C and C++ compiler flags:
* (Almost) all compiler flags are set uniformly
in the system/bt/Android.mk file.
* Enable by default breaking the compilation if there is a
compilation warning: -Werror
* Enable most compilation warnings: -Wall -Wextra
* Renamed Android.mk related flags:
- bdroid_C_INCLUDES -> bluetooth_C_INCLUDES
- bdroid_CFLAGS -> bluetooth_CFLAGS
* Introduce variables for C-only and C++ only compiler:
- bluetooth_CFLAGS: common C and C++ compiler flags
- bluetooth_CONLYFLAGS: C only compiler flags
- bluetooth_CPPFLAGS: C++ only compiler flags
* Disable warnings for existing issues - to be removed as issues are
resolved
* Add a workaround for libchrome and -DNDEBUG usage.
Bug: 26879229
Change-Id: Ie7595965ca0c8ead0e95e983e76c327e7891b2c3
ndroid.mk
|
| 32579680eb9c52fa58b3eb7e3fd8a7de8f264983 |
06-Jan-2016 |
Andreas Gampe <agampe@google.com> |
Bluetooth: Remove std=c++11
It's the default now.
Change-Id: Ice463e6e74d56bffd00a80824721dbe57241ea06
ndroid.mk
|
| f0a6988499b592cc959b88d573cf8876e9b3753d |
10-Dec-2015 |
Alex Deymo <deymo@google.com> |
Rename libchrome-host to libchrome.
Update the name of the libchrome-host library to the match the one used
in the target.
Bug: 24619596
TEST=make dist
Change-Id: I73e906a0683142cc7e43237d4c5fa20c27acc6cc
ndroid.mk
|
| c196f214c5ae349ec2022f8d3cbaf56910b3b9f8 |
24-Sep-2015 |
Pavlin Radoslavov <pavlin@google.com> |
GKI cleanup - Moved GKI_get_os_tick_count() to OSI
* Renamed function GKI_get_os_tick_count() to time_get_os_boottime_ms()
and moved it to the OSI module: to the new file osi/src/time.c .
The corresponding header file is osi/include/time.h
* Added unit tests for function time_get_os_boottime_ms() in file
osi/test/time_test.cpp
* Removed "osi/include" from the list of paths to search for include
files. This is needed, because file name collision of "time.h"
in osi/include and the system <time.h>
Change-Id: I934be0c8f392150a352947326546bcf8aa070f97
ndroid.mk
UILD.gn
|
| 6f5f0540f1557c87349d9f3378a90979c175ed1d |
21-Aug-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Clean up controller documentation and address style
issues
This upload cleans up the command documentation in the controller's
header file and also fixes a few logical errors related to command
status event packets. Some style fixes are also included, such as
removing braces for single line conditionals to conform to remain
consistent with the rest of the project.
Bug: 21586676
Change-Id: Ife5e608e43c3030e76eff8ea79ce6cad0b2c10a3
nclude/command_packet.h
nclude/dual_mode_controller.h
nclude/event_packet.h
nclude/packet.h
nclude/packet_stream.h
rc/bt_vendor.cc
rc/dual_mode_controller.cc
rc/event_packet.cc
rc/hci_transport.cc
rc/packet.cc
rc/packet_stream.cc
rc/test_channel_transport.cc
rc/vendor_manager.cc
est/packet_stream_unittest.cc
|
| 2e00ba9b74536ce9a0d1adf78f92e6478563e49b |
19-Aug-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Add tests
This upload adds unit tests for the packet_stream and hci_transport
objects.
Bug: 21586676
Change-Id: Ica99be959cdd801ea10631d31fc0f0b8ea1d8456
ndroid.mk
UILD.gn
nclude/hci_transport.h
rc/hci_transport.cc
rc/packet.cc
rc/packet_stream.cc
rc/vendor_manager.cc
est/hci_transport_unittest.cc
est/packet_stream_unittest.cc
|
| 7a87e719982e874c1d23e106a4dbc8dd2ede97bd |
14-Aug-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Add working post delayed task
This upload fixes the post delayed task functionality so that it uses
a custom time stamped event object instead of posting callbacks to the
vendor manager's task runner (which does not support delayed tasks for
MessageLoopForIO).
Bug: 21586676
Change-Id: Ib5d9019d8c8dd5c9ad865df82eac16b0ecf50268
nclude/dual_mode_controller.h
nclude/hci_transport.h
cripts/test_channel.py
rc/dual_mode_controller.cc
rc/hci_transport.cc
rc/vendor_manager.cc
|
| bf7db3143fda4dc8391c92ec25dc93b9725ec225 |
10-Aug-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Read controller properties from file.
This upload lets the controller be configured via a JSON file that
specifies the controller attributes. A default configuration is not
currently provided but will be included in a future upload. Also fixes
a few mistakes in test_channel.py.
Bug: 21586676
Change-Id: I9d0e65a9664e179ddb97664a748c766f4f8a554e
ndroid.mk
ata/controller_properties.json
nclude/dual_mode_controller.h
nclude/hci_transport.h
nclude/vendor_manager.h
cripts/build_and_run.sh
cripts/test_channel.py
rc/dual_mode_controller.cc
rc/hci_transport.cc
rc/test_channel_transport.cc
rc/vendor_manager.cc
|
| 9cd603f04a5478d88da1882ab46f226a5bf4a2d1 |
09-Aug-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Remove handlers
This upload removes the HciHandler and TestChannelHandler classes.
Removing these classes simplifies logic and also allows the
controller to do preprocessing before executing commands.
Bug: 21586676
Change-Id: I3f2711349dd2c86794538edc632b4f5f49b5169b
ndroid.mk
UILD.gn
nclude/dual_mode_controller.h
nclude/hci_handler.h
nclude/test_channel_handler.h
nclude/test_channel_transport.h
nclude/vendor_manager.h
rc/dual_mode_controller.cc
rc/hci_handler.cc
rc/test_channel_handler.cc
rc/test_channel_transport.cc
rc/vendor_manager.cc
|
| f1548336bf79a1e5eade187b5f483726d3439639 |
08-Aug-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vnd_lib: Discover devices
This upload allows the user to send fake inquiry responses to the
controller at run-time via the test channel.
Bug: 21586676
Change-Id: Ieae24d496115f19716fcaaffa627a9c22ffade21
nclude/dual_mode_controller.h
nclude/test_channel_handler.h
nclude/test_channel_transport.h
cripts/test_channel.py
rc/dual_mode_controller.cc
rc/test_channel_handler.cc
rc/test_channel_transport.cc
|
| 8bb61fe28ac339276acf1b5b1616a6e56e2cccac |
05-Aug-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Add test channel command transport and handler
This upload adds the transport and handler functionality for the test
channel. Test channel commands will be sent by the test channel
program and processed by library objects in a manner analogous to how
packets are received and handled from the HCI. The actual
implementation for the 4 test channel commands included here will be
in the next upload.
Bug: 21586676
Change-Id: I9354baedcbd6e48b8920a3ebf1afa69eb89834d9
ndroid.mk
nclude/dual_mode_controller.h
nclude/hci_handler.h
nclude/test_channel_handler.h
nclude/test_channel_transport.h
nclude/vendor_manager.h
cripts/build_and_run.sh
cripts/test_channel.py
rc/dual_mode_controller.cc
rc/hci_transport.cc
rc/test_channel_handler.cc
rc/test_channel_transport.cc
rc/vendor_manager.cc
|
| 2d96c88025c80d4d7e0e341cfdbf278596340b66 |
03-Aug-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Add test channel for run-time input
This change adds a test channel for receiving user-specified commands
and data at run-time. Specifically, the channel is intended to be used
for additional debugging capabilities and for fine-tuned control over
the test controller. Implementation-wise, the test channel is simply
another socket that the vendor manager watches on. The handling of test
channel input goes through the pipeline as normal HCI data after it is
received by the HciTransport.
A script for building and running the test vendor library and
(optionally) the test channel has been added in scripts/.
Bug: 21586676
Change-Id: I55bdeedbcd81effbe009aa62a19031637374a1e6
nclude/hci_transport.h
nclude/packet_stream.h
nclude/vendor_manager.h
cripts/build_and_run.sh
cripts/test_channel.py
rc/hci_transport.cc
rc/packet_stream.cc
rc/vendor_manager.cc
|
| 2fc0eba951081bddd15a3482f2189a9125dac48a |
30-Jul-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Implement event loop with libbase
Adds event loop functionality by handling IO in the Watcher class,
HciTransport. Also fixes a bug where the vendor's file descriptor was
closed early and another where an object was mistakingly being passed
by value.
Bug: 21586676
Change-Id: I2ec30b68c0ede47bf1ad78ec13beee93d0f8ab9d
nclude/dual_mode_controller.h
nclude/hci_handler.h
nclude/hci_transport.h
nclude/packet.h
nclude/packet_stream.h
nclude/vendor_manager.h
rc/bt_vendor.cc
rc/hci_handler.cc
rc/hci_transport.cc
rc/packet_stream.cc
rc/vendor_manager.cc
|
| b2d45eb7cbd4f2de5f0101915981a87392ccf625 |
29-Jul-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Use base::Thread
This upload starts the replacement of the event loop in HciTransport
with libbase's MessageLoopForIO. A global instance of the new
VendorManager class now manages the trio of previously global objects
(i.e. HciTransport, HciHandler, and BREDRController). Note that the new
libbase event loop structure doesn't actually handle anything right now
and the functionality will come in a future change.
Bug: 21586676
Change-Id: Id74392e2a566e06445eb65269f0142265087d47d
ndroid.mk
UILD.gn
nclude/bredr_controller.h
nclude/command_packet.h
nclude/dual_mode_controller.h
nclude/event_packet.h
nclude/hci_handler.h
nclude/hci_transport.h
nclude/vendor_manager.h
rc/bredr_controller.cc
rc/bt_vendor.cc
rc/dual_mode_controller.cc
rc/hci_handler.cc
rc/hci_transport.cc
rc/vendor_manager.cc
|
| a7077ee9b7482708fa7272837a527f64bb470988 |
28-Jul-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Use libbase's CHECK/DCHECK
This upload pulls in libchrome to the test vendor library and uses CHECK
and DCHECK from base/logging.h.
Bug: 21586676
Change-Id: Ia88b4aa3728493d798e8d36076a351a00fad7c6b
UILD.gn
nclude/bredr_controller.h
nclude/command_packet.h
nclude/event_packet.h
nclude/hci_handler.h
nclude/hci_transport.h
nclude/packet_stream.h
rc/bredr_controller.cc
rc/bt_vendor.cc
rc/command_packet.cc
rc/event_packet.cc
rc/hci_handler.cc
rc/hci_transport.cc
rc/packet.cc
rc/packet_stream.cc
|
| 5f8a46aae144f416d695010ae61c47116562a615 |
24-Jul-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Dummy inquiry result
This upload mainly consists of stub command implementations to get
Bluetooth into the enabled state. The last command handled is the
inquiry command. Upon inquiry, the controller currently sends back a
dummy inquiry result for a fake device. This fake device appears under
the "Available devices" list in the UI. Bluetooth is able to be toggled
off and on from the UI as well (i.e. it doesn't crash after switching to
on for a few seconds).
Bug: 21586676
Change-Id: Ifd42a6ab4b9338afd461d2ef4f809bdd62fcf79f
nclude/bredr_controller.h
nclude/event_packet.h
nclude/hci_transport.h
rc/bredr_controller.cc
rc/event_packet.cc
|
| 3c0ea3e5a5aca269283854d8f2f584ad0699c951 |
16-Jun-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Use EPOLLOUT
HciTransport checks for write-readiness before sending any packets
back to the HCI. Packets are maintained on a queue and only written
when the EPOLLOUT flag is set in the epoll_event.
Bug: 21586676
Change-Id: Iebd9599286b62746e2be7e424c522058ae978bb1
nclude/bredr_controller.h
nclude/hci_handler.h
nclude/hci_transport.h
nclude/packet_stream.h
rc/bredr_controller.cc
rc/bt_vendor.cc
rc/hci_handler.cc
rc/hci_transport.cc
rc/packet_stream.cc
|
| f3b1820e1676261886a6537e1f6abac6a35ed6d4 |
16-Jun-2015 |
Dennis Cheng <dennischeng@google.com> |
test_vendor_lib: Initial commit
Basic vendor library for a test Bluetooth controller. Currently consists
of mostly stub functions and objects and can only handle the reset
command (by immediately responding with a command complete event).
Implemented through a global EventDispatcher object which waits for data
from the HCI and directly returns the appropriate response.
Eventually, the dispatcher will manage a fake controller object that
will provide callbacks to be executed whenever events are processed by
the dispatcher. Unit tests are still in progress and will be in the next
upload.
Bug: 21586676
Change-Id: I1c6746f8b0f1732b89a1da13facecdd49b5ac1b6
ndroid.mk
UILD.gn
nclude/bredr_controller.h
nclude/command_packet.h
nclude/event_packet.h
nclude/hci_handler.h
nclude/hci_transport.h
nclude/packet.h
nclude/packet_stream.h
rc/bredr_controller.cc
rc/bt_vendor.cc
rc/command_packet.cc
rc/event_packet.cc
rc/hci_handler.cc
rc/hci_transport.cc
rc/packet.cc
rc/packet_stream.cc
|