Remove extraneous field inherited from parent makefile

Change-Id: I0d0bc0c5ed508f2d94c5644cb3b0f823bae5d83a

LE: Add connection parameter update request API (2/4)

Change-Id: I131bfa7fb0a8cc970e1ac3e26e9d86ed2bee80a7

Fix builds: Revert "Added code to publish ble adv tx power in dBm"

This reverts commit 347a1d64c7a9c345793c61b03bdcaa31ad8e5678.

Change-Id: If62571a3f5bd87d04e5643855c9fea1b8048aa5a

Added code to publish ble adv tx power in dBm

Bug 15412969

Fix build warning about unused argument.

Change-Id: I3ae26dfc4ffd97d039386f55e3290a6cf36ae17d

Fix string format error for 64-bit builds

Change-Id: Idefad9d9ea3a54f7f412e80c4998137d15a1b23c

Merge "A simple, thread-safe timer API for bluedroid." into lmp-dev

Merge "Revert "Revert "Clean up messy dispatch in bt_hci_bdroid by using new primitives.""" into lmp-dev

Revert "Revert "Clean up messy dispatch in bt_hci_bdroid by using new primitives.""

Original code broke the build for razor-eng, updated the code to fix break.

This reverts commit dd29f036a8cde94b3c296fb08ee3e9b2d65c15e0.

Change-Id: I2d2f9b5b51b923e3f0e8092b3169ea447193887d

A simple, thread-safe timer API for bluedroid.

Change-Id: I5fa21b48506e4d1d65da4b8bedbbf39e03dad559

Merge "Add audio_input_flags_t to HAL open_input_stream" into lmp-dev

Add audio_input_flags_t to HAL open_input_stream

Change-Id: I5023c0fdb8069668ca2b84096277d6abdbf7d729

Merge "Fix GKI_add_to_timer_list to handle middle-of-list insertions correctly" into lmp-dev

Fix GKI_add_to_timer_list to handle middle-of-list insertions correctly

The newly added timer_list code didn't handle middle-of-list insertions
correctly, leading to problems pairing certain Bluetooth devices as well
as causing 100% CPU usage for the Bluetooth task.

Change-Id: I3d4c22b775a91da2a3f983e8c78facbd91d4cb7d

Revert "Clean up messy dispatch in bt_hci_bdroid by using new primitives."

This change broke the build. Reverting to unblock people waiting for a green build to start working.

This reverts commit 3f9fbe74c1d6e774d3f21275515c7b895715aec5.

Change-Id: I58c3bdcc2f5f3fe744b8d3f80753566cf1d22bd1

Clean up messy dispatch in bt_hci_bdroid by using new primitives.

Instead of dispatching onto a worker thread by setting bits of a
bitfield and then having the thread pull data from shared data
structures, this code uses the previously defined primitives to
write to a synchronized thread event queue. It's advantageous because
it's more direct dispatch (you specify which function you want to
call on the worker thread instead of mapping bitfields to a function),
it avoids a lot of boilerplate code, function calls have predictable
ordering, and it uses generic infrastructure instead of building a
custom dispatch + synchronized data strucures.

Change-Id: If938e4fd47c8ddf328a29d104151f900ec659588

Extend the threading library to back thread with an event queue.

This change allows arbitrary functions to be called on a given
thread which will clean up much of the dispatch code in bluedroid.
Looking forward, this code will be extended to allow additional objects
and queues to be attached to a thread for more customizable dispatch.

Change-Id: Id3a16256c264e3d35e6db5a562cb0e7762676457

Expose enqueue/dequeue file descriptors for fixed_queue.

These file descriptors can be used with a reactor to perform
non-blocking enqueue/dequeue operations.

Change-Id: If75730f9093e8d20a789a7bd2bde92e019922e5a

Eliminate last_ticks from TIMER_LIST_ENT.

It's unnecessary, potentially calculated incorrectly, and the insertion
logic that depended on it was fragile. I've rewritten the list
insertion routine so it's easier to follow. This should also fix
http://b/16259295.

Change-Id: Ib7184c94c495c03d3acbe105955b1bb712dbbfaa
(cherry picked from commit a1bfd3ee95378fab78c931c3360cfbd16de3b617)

Merge "Calculate A2DP frames to send based on elapsed time"

Calculate A2DP frames to send based on elapsed time

This patch is aimed at improving A2DP audio quality by sending the
correct number of frames based on the actual time elapes between "ticks"
in order to account for timer drift.

Change-Id: If5b7cde90f5eb10b926ad247f51cff0eb60bbff7

A2DP: Fix audio stuttering in A2DP sink mode

Change-Id: Ied821aeb14f83e3fd1daa6e175ac995abbfbb198

Merge "Fix GKI buffer leak with discovery information service reading"

Fix GKI buffer leak with discovery information service reading

If the discovery information service of the LE client has
the fields model number, serial number, fw version, etc,
the service would allocate PKI buffer and never do anything
with it, so it would leak.  It looks like it should have
been assigned the a callback string array, however fixing
that still doesn't fix the leak because the code that receives
the string array, bta_hh_le_dis_cback(), never uses it and
never frees it.

I believe the semantic is that the string arrays are kept around
as a cache in the srvc engine connection structure,
so it's the srvc engine dealloc of the callback structure that
needs to also free the string buffers if they have been allocated.
After fixing the string array allocation, add code to free the
string array entries if they are not null.

Also fixed an off by one error in DIS_SrUpdate() that would also lead
to a GKI buffer leak.

Improve two string termination cases to use a simple set of the
last entry in the char array instead of memsetting the whole array
when most of it will be filled by a following memcpy.

Change-Id: I7905cd771dbbe166e3c2b42e019bac9f5a312877
Signed-off-by: Mike J. Chen <mjchen@google.com>

Fix deadlock and non-working socket msg throttling

Two major bugs in btif_config.c.  One is that due to improper
locking order, a deadlock could occur (symptom is generally
out of GKI buffers because BTIF thread stops processing it's
mailbox because it is blocked sending a msg to the socket
thread, which in turn is blocked because BTIF is holding
the lock it needs to do the config save that BTIF requested
in the first place).

The other is also lock related.  The mechanism to throttle
socket messages based on whether there was one already
in flight wasn't working because there wasn't locking
when that variable was being used.  The compiler would be
free to cache it in a register on SMP machines, so it would
never look like there as a value in flight.  This would
cause a bunch of messages to be queued up, and then the
way the handling worked in cfg_cmd_callback(), every
cmd would cause a 3 second sleep, even if there was nothing
to do.  Modified the loop to check if there was nothing
to do before sleeping.

I was running into both bugs when repeatedly pairing and
unpairing a BLE device.  The deadlock would hit first, causing
out of GKI buffers.  Once I fixed the deadlock, I would still
run out of GKI buffers because the socket thread wasn't working
fast enough to clear the backlog of socket messages it had
received.

Also optimized some string termination code which was using
a memset of the full buffer before and then copying over
every entry except the last.  Now we do the copy and just
set the last buffer entry to null.

Change-Id: Ic06ef0b8e15d8f1fe669fb88439851ffbad560de
Signed-off-by: Mike J. Chen <mjchen@google.com>

am 02d9b7b4: Bluetooth: Increase number of allowed notifications

* commit '02d9b7b41f1fcf21e798a4c0aabde89f00729d08':
  Bluetooth: Increase number of allowed notifications

am 7475a509: Bluetooth: Adds SMP response timer for security request

* commit '7475a5098339d763aa6e3a0374ee2f1c16e002c4':
  Bluetooth: Adds SMP response timer for security request

am 156c52bb: Send correct status for SMP AUTH failure

* commit '156c52bb003a14147db99d8a151edc08f591bd34':
  Send correct status for SMP AUTH failure

Bluetooth: Increase number of allowed notifications

This change increase the maximum number of notifications
that can be registered for LE remote devices.
This count is increase from 7 to 15.

Change-Id: I422704e9bae69c48b731757526ed9ee4055aa643

Bluetooth: Adds SMP response timer for security request

This adds timer to cancel security request from remote if there
is no response from user for pairing popup.

Change-Id: I18694dcc5275a1eb10de4ad921b6baca189b42fa

Merge "Fix deadlock and non-working socket msg throttling"

Fix GKI buffer leak in GATT event handling

The GATT event handler was telling the BTA dispatcher that
it would always free the GKI msg buffers itself, but in fact
this wasn't true.  There were some cases where the buffer is
queued and freed later (queued in p_clcb->p_q_cmd for later
access and freeing), but many times the event handlers
(like bta_gattc_ci_save()) don't queue the buffer and don't
free it, causing a GKI buffer leak.

Also, prevent a double free case (not certain it happens, but
in theory it could) if bta_gattc_sm_execute() queues a buffer
in bta_gattc_disc_cmpl().

Lastly, remove held flag since it's redundant with whether
p_q_cmd is null or not.  Just go with one source of truth and
not have one shadow the other.

Change-Id: I1bd43919b8a2e125076e26cd6a3a08480b3e7e2d
Signed-off-by: Mike J. Chen <mjchen@google.com>

Send correct status for SMP AUTH failure

Send correct corresponding status for SMP AUTH failure to JAVA layer.
This will enable BluetoothEventManager to display correct error message
for bond state changed callback.
Bug 15677623

Change-Id: I6a8742926bac862f960890417b5d68c23b33e010

Merge "Fix GKI timers to take execution time into account"

Fix GKI timers to take execution time into account

Previously, the GKI timer system would not take the time taken for
updating the timers into account when re-scheduling the next timer. This
would by design introduce timer drift into the system.

This patch measures the time taken for an alarm to be triggered to
report the correct number of ticks to the GKI timer subsystem, as well
as measuring the time taken for updating the timers to adjust the next
alarm accordingly.

This patch dramatically improves A2DP audio quality on various devices
which currently suffer timer drift problems.

Change-Id: Ia99454abcb2a6889c47b1d8e267705f29ebe7b64

Add template doc for Android properties used by bluedroid.

Change-Id: Idafbaa69161a7fa01a381d9b4fc5d9f41bc5695f

Merge "Prevent A2DP bit pool calculation endless loops."

Expose semaphore's file descriptor.

|semaphore_get_fd| can be used by callers to determine if it's safe
to call |semaphore_wait| and |semaphore_post| without blocking.

Change-Id: Icf758f03ea2aa85cddcb4ab3f7906b2c587fb8ac

Start of threading library

So far it's a thin shim around pthreads which allows setting thread
name and querying tids from any thread.

Change-Id: Id156f662778806a54a8a302be424ee051fac4710

Prevent A2DP bit pool calculation endless loops.

This prevents log spam and endless loops in the A2DP bit pool
calculation. Protecting against the endless loop with prevent battery
draining issues and phone unresponsiveness.

This patch addresses the symptom. A separate patch will be submitted
once root cause is determined to address the problem.

Change-Id: I045b724c1c1638551b9333249f39eee109fa9ba1

Bluetooth: Rename BTA_AVK_INCLUDED flag to BTA_AV_SINK_INCLUDED (bt_target.h)

Now OEM could enable/disable A2DP SINK role

Change-Id: I62cf17b0397a86c5f886a2c1ca724ca77a7f9bd4
Signed-off-by: Dmitry Shmidt <dimitrysh@google.com>

Merge "DO NOT MERGE: Send to local UUID for AV connection" into klp-wireless-dev

Implement the reactor pattern in C.

This code will form the basis of most select-based event loops in
bluedroid. It provides a thread-safe abort routine and a separation
between the dispatcher and event handler code.

Change-Id: I6f1c033d18f045ba273187dab607c209dfe32d30

DO NOT MERGE: Send to local UUID for AV connection

BTA expects local UUID to be passed into BTA_AvOpen. The value of source/sink
was reversed in the btif layer thus causing connection failure
bug 15837817

Change-Id: If159ac0491afcb75659336c98caba3cefe71ee7b

Merge "Code cleanup - remove unnecessary commented out code" into klp-wireless-dev

Code cleanup - remove unnecessary commented out code

Change-Id: I86e78d1b0470b9fda7455ae8ef5788814cf5c780

Merge "audio: deprecate audio_stream_frame_size()"

am 85b4ee5d: (-s ours) Merge "DO NOT MERGE: Send to local UUID for AV connection" into klp-wireless-dev

* commit '85b4ee5df4208fd06e836f40d7a19a84545f4e93':
  DO NOT MERGE: Send to local UUID for AV connection

audio: deprecate audio_stream_frame_size()

Bug: 15000850.
Change-Id: I1edcc78561cdc328ae0d91a1d07b0836ca5e1715

Fix signed - unsigned comparison in bluedroid test.

Change-Id: I702e9406a31424f27de2b600cb55c92e255917dd

am 19084c62: Implement the reactor pattern in C.

* commit '19084c6242d8ddb366a52eba5084c974280cce0f':
  Implement the reactor pattern in C.

am b3c61229: Merge "Code cleanup - remove unnecessary commented out code" into klp-wireless-dev

* commit 'b3c61229ccf38ea3f398f93902d348ba12a7fb07':
  Code cleanup - remove unnecessary commented out code

am a92c04f2: (-s ours) DO NOT MERGE Fix compilation issue

* commit 'a92c04f2dbd7f88853d6380d66c1f4d200674f97':
  DO NOT MERGE Fix compilation issue

DO NOT MERGE Fix compilation issue

This patch fixes broken compilation on KWD branch

Change-Id: I9a1da11e33e938910dc556ed3e015bd3456e21ef

Fix build: change macro call from _DEBUG0 to _DEBUG.

Change-Id: Iacbf4c33a7b690e2065471b85a6a4d82dd4a7fca

am 23ae7c52: Merge "Added code to reuse the signal fds and avoid fd close" into klp-wireless-dev

* commit '23ae7c526b529cf1c76c4a4cc8a341545493404d':
  Added code to reuse the signal fds and avoid fd close

am 359895ef: Fixed deadlock issue for concurrent multiple rfcomm connections.

* commit '359895efb36a7e4f65fbcc9a28221b6ede44aadd':
  Fixed deadlock issue for concurrent multiple rfcomm connections.

Use POSIX timer for non-wake alarms in bluedroid.

After this change, bluedroid will go through AlarmManager for wake
alarms (>= 3s) and POSIX timers + wake lock for short deadlines.
This change allows A2DP to continue streaming while the screen is
off.

Change-Id: I56622a6873aa431842dd1d97e830849ac2fe6773

Remove more unused code from GKI timers.

Change-Id: I4fc3a64312b49a3dd23d80454244f8eb6340d8e6

Remove more unused code from GKI.

Specifically:
 - ISR code (since bluedroid is entirely userspace)
 - logging macros
 - newlines in log macros
 - GKI_get_time_stamp (unused function)
 - GKI_freeze
 - unused timer variables and macros

Change-Id: I8e0676c28842d87c4148059ebb0320367658bc16

Fix problem with A2DP initialization due to merge error

btif_av_init() would fail when called a second time due to
erroneous nulling of btif_av_cb.sm_handle

Bug: 16014877
Change-Id: I7e6b213759e93372ac8ecaadc2b81452572f3f7d

Merge "Added code to reuse the signal fds and avoid fd close" into klp-wireless-dev

Added code to reuse the signal fds and avoid fd close

Bug 15436535

Change-Id: Ica5b1915dd94699eb19c57b6904154493b0cc08c

Fixed deadlock issue for concurrent multiple rfcomm connections.

Bug 15469934

Change-Id: I77a3dc97c92f0ccd5764220e8e0bf5c0689d7b12

Bluetooth: Support MAP Client role on Bluedroid.

Implementation changes from BTA and BTIF layer to support
MCE role on Bluedroid stack.

Change-Id: I8547b0f28338e83edabae969121872ca23fdcb36

Fix audio HAL build (make it a shared library again)

Bug: 15978121
Change-Id: I23ba30f27b73b48a1d9128d294641367bfffa810

am 7ae3f9ee: LE Onfound and Onlost feature

* commit '7ae3f9eebc4e930d51e6b61f845ffd0f9a8cb9f8':
  LE Onfound and Onlost feature

am 5e177043: (-s ours) LE ADV Filter changes

* commit '5e177043e3dfcd70676761a40425179d16c4a7ce':
  LE ADV Filter changes

Add BOARD_BLUETOOTH_BDROID_HCILP_INCLUDED

Added CleanSpec.mk change

Signed-off-by: Richie Hsieh <richie.hsieh@mediatek.com>
Change-Id: Ice2a514f3d24859dbf091aae30e5c8781e90b7fd
(cherry picked from commit d343338cb4c42e62ac97c451bc23fb36d9b58d3f)

am b9b1f4e7: (-s ours) LE batch scan feature

* commit 'b9b1f4e7a5cc96c57873a991f4ea3c3c08934c26':
  LE batch scan feature

LE Onfound and Onlost feature

BTIF and stack layer changes for onfound/onlost feature

DO NOT MERGE

Change-Id: Id52cdd5855ca8c0ff276b42613b51c4cd3882bfc

LE ADV Filter changes

BTIF and stack layer changes for ADV filter feature

DO NOT MERGE

Change-Id: I865e57c41802751e172248547b8510598261ec94

LE batch scan feature

BTIF and stack changes for batch scan feature

DO NOT MERGE

Change-Id: I4dd2d551435e9f1f0e43906bd695294e97ed4bba

am 66432dcf: Add WBS support on Bluedroid (2/6)

* commit '66432dcffe211e317d35978283a04a96d5620bdf':
  Add WBS support on Bluedroid (2/6)

Add WBS support on Bluedroid (2/6)

Bug 13764086

Change-Id: I12bf302f7dd19efb497ad49d4d1d638a7a926049

LE Onfound and Onlost feature

BTIF and stack layer changes for onfound/onlost feature

Change-Id: Id52cdd5855ca8c0ff276b42613b51c4cd3882bfc

LE ADV Filter changes

BTIF and stack layer changes for ADV filter feature

Change-Id: I865e57c41802751e172248547b8510598261ec94

LE batch scan feature

BTIF and stack changes for batch scan feature

Change-Id: I4dd2d551435e9f1f0e43906bd695294e97ed4bba

am 021e17a6: Clear background conn list entries during BT power off

* commit '021e17a641e61e3f139717cd755bf6f9b9b9eb12':
  Clear background conn list entries during BT power off

am 9279e000: Merge "Bluetooth: Updates background conn list properly during unpair" into klp-wireless-dev

* commit '9279e0003d43be077d2c2d7a3e5163db960a1150':
  Bluetooth: Updates background conn list properly during unpair

Clear background conn list entries during BT power off

This clears whitelist entries and resets  bgconn device lists

Change-Id: I551791d63fa826071c113abd383540838f6c59b2

LE: Fix HOGP connection issues caused by incorrect HCI packet flags

Do not set the automatically-flushable packet flag for LE bound HCI
packets (Packet_Boundary_Flag 0b10).

Reported-by: Richie Hsieh <richie.hsieh@mediatek.com>
Change-Id: Id71a096acee70582923e652cfef1f4141b721fed

am d9e2d0f1: Bluetooth: Restarting PM timer during incoming rfcomm data.

* commit 'd9e2d0f1fa19f737632e82c24ba278990b8a7548':
  Bluetooth: Restarting PM timer during incoming rfcomm data.

am e146869f: GAP: Remove device info from storage on Pairing Cancel

* commit 'e146869fec903400e601d2ac7b0a59bd05934f0e':
  GAP: Remove device info from storage on Pairing Cancel

am b4801441: GAP:Add authentication failed error code in auth_compl_evt.

* commit 'b4801441a5b7a093bc78f2143a8ed28605c51dc6':
  GAP:Add authentication failed error code in auth_compl_evt.

am aef7a671: [2/2] Bluetooth: Security: Handle authentication rejected error status

* commit 'aef7a6716e72e9304af36469500ee53c97350c21':
  [2/2] Bluetooth: Security: Handle authentication rejected error status

am 59a88ec2: Bluetooth: Handle missing key event

* commit '59a88ec207ead601c7ba5e767fd1373c7f2df70b':
  Bluetooth: Handle missing key event

am 831423ea: Bluetooth: Fix to avoid NULL pointer dereferences

* commit '831423ea1879244e746f39d7696ad4a66681f306':
  Bluetooth: Fix to avoid NULL pointer dereferences

am c8fe2925: GAP: Retain the old COD if already present

* commit 'c8fe2925af4f5b167b615a0677adebbd156a4387':
  GAP: Retain the old COD if already present

am 79f454dc: L2CAP: Send HCI connect with all packet types supported

* commit '79f454dc544280cc1a3ca3acc643f7dddfe91cd9':
  L2CAP: Send HCI connect with all packet types supported

Bluetooth: Restarting PM timer during incoming rfcomm data.

During incoming rfcomm data, we were setting the pm mode to
active, thereby stopping timer and never restarting it again.
So Dut was never going to sniff , if the last packet is incoming
packet. Therefore add change to restart the PM timer during incoming
data.

Change-Id: I4396bb2f267b12cb92a944c15609ddeaa13288ad

GAP: Remove device info from storage on Pairing Cancel

Remove paired device information for SSP devices from the BT-IF
layer storage on pairing cancel.

Change-Id: I4660e3659866bd66339350b5cdb92bf3ad3cc850

GAP:Add authentication failed error code in auth_compl_evt.

There is an unhandle failed HCI error code when pairing with
symbian OS phone nokia n95 which will cause no prompt when
auth failed.

Change-Id: I3a37c7732d1bf71c090bb968380d8c5f0af2ff79

[2/2] Bluetooth: Security: Handle authentication rejected error status

Handle error codes in authentication complete event with failures
like reject pairing , unspecified error.

Change-Id: I392b7531a8625efbf91bf80f7f617a6954229892

Bluetooth: Handle missing key event

This change handles HCI_ERR_KEY_MISSING event
coming to the stack when user doesn't enter any
key and timeout happens.

Change-Id: If530e8acce3f105e4f1e2841bf350ee5330f389e

Bluetooth: Fix to avoid NULL pointer dereferences

This fix for avoiding NULL pointer dereferences:
 - Added check for state checking in btm_remove_acl function
   to avoid sending disconnect command if already in Disconnecting
   state.
 - Added NULL check for 'pin_code' in btif_dm_pin_reply function
 - Null check before accessing in BD interface layer configuration
   node variables.
 - Add NULL check before accessing p_bd_addr variable to avoid null
    pointer exception while referring to invalid memory.

Change-Id: Ib7ed45b6a1692785a45224d739a564f767e5b10f

GAP: Retain the old COD if already present

Retain the old COD as the initial value if the device
entry already present in the list and entry is not in use.

Change-Id: I99b7269b1976ef7458c6568441e6e3c1e015a75f

L2CAP: Send HCI connect with all packet types supported

HCI create connect function is now supporting only single slot packet
during HCI connection initiation. Added support for all basic rate
packet types.

Change-Id: Ieeb892ea43dc9bb801a8f97a644a0f8c06b8184a

am 96cdc153: Merge "GAP: Handle invalid SDP service search request" into klp-wireless-dev

* commit '96cdc153002c82d959f132d6e6804b516228ff32':
  GAP: Handle invalid SDP service search request

am 08b1c8d9: Merge "GAP: Handled collision scenerio with headset" into klp-wireless-dev

* commit '08b1c8d9147c253fecd34686b0c4dcd5b78cc778':
  GAP: Handled collision scenerio with headset