Add missing initialization of queue btm_cb.sec_pending_q
am: 16c97b9

* commit '16c97b9a1732558868901f4315f4f0b3850be2a1':
  Add missing initialization of queue btm_cb.sec_pending_q

Add missing initialization of queue btm_cb.sec_pending_q
am: 6aef394

* commit '6aef394995e2c99bb29be05fa63298ea56ea3f60':
  Add missing initialization of queue btm_cb.sec_pending_q

Add missing initialization of queue btm_cb.sec_pending_q

Bug: 27763878
Change-Id: I93001a679e22b744c3feafda1997e5f7e707db4e

Log error code on wakelock acquisition/release failure
am: b71bdd6

* commit 'b71bdd6f5bef9514eb358c4279dca57ae8c1df70':
  Log error code on wakelock acquisition/release failure

Log error code on wakelock acquisition/release failure

Bug: 27721443
Change-Id: I8ac9e738d916bfdcf125e33c0e09f90cc8d26d55

Cleanup after p_mcb if L2CA_ConnectReq() failed
am: 757047a

* commit '757047a147582d6c5be22f0fb5148005f345fc9b':
  Cleanup after p_mcb if L2CA_ConnectReq() failed

Add missing initialization of queue btm_cb.sec_pending_q

Bug: 27763878
Change-Id: I93001a679e22b744c3feafda1997e5f7e707db4e

Cleanup after p_mcb if L2CA_ConnectReq() failed

If L2CA_ConnectReq() failed, make sure that
we cleanup the lcid cache state after the
affected p_mcb.

Also, extra check and a log message inside function
rfc_check_send_cmd().

Bug: 27334916
Change-Id: Ib2950d12ce456d74355f4bcc0e3c4d87603f8f91

Don\'t clear the btsnoop log ringbuffer on compress.
am: b35c234

* commit 'b35c23483633826617d2312be805df93aeb9d8ca':
  Don't clear the btsnoop log ringbuffer on compress.

Add config source to dumpsys output
am: 63ddbe8

* commit '63ddbe8813aac242071dffc0bd413ba60e6dd7bf':
  Add config source to dumpsys output

Don't clear the btsnoop log ringbuffer on compress.

Change-Id: I0bf4279104e87b151553eba913491d934f3cde41

Add config source to dumpsys output

This logs where the config file was loaded from in the Bluetooth
Manager dumpsys output.

Bug: 27354612
Change-Id: I50d4aaa0be4f4d1d890580b03742713f4345c80f

Add config source to dumpsys output
am: 190cf46

* commit '190cf46ff6937bceff146aec026cf9f53c40e84d':
  Add config source to dumpsys output

Add config source to dumpsys output

This logs where the config file was loaded from in the Bluetooth
Manager dumpsys output.

Bug: 27354612
Change-Id: I50d4aaa0be4f4d1d890580b03742713f4345c80f

Do not leak eventfd\'s for RFCOMM/AVDT/L2CAP
am: 072bd75

* commit '072bd753e6ac2a011421cd7e4a2e622ee7d052fc':
  Do not leak eventfd's for RFCOMM/AVDT/L2CAP

Do not leak eventfd\'s for RFCOMM/AVDT/L2CAP
am: 9ccca2b

* commit '9ccca2b5937ec42409d8b6354884f1f4b6d524f7':
  Do not leak eventfd's for RFCOMM/AVDT/L2CAP

Do not leak eventfd's for RFCOMM/AVDT/L2CAP

Various users of fixed_queue's do not free the lists properly after
using them. As a result two event fd's leak for each fixed_queue that
is not free'd, as fixed_queue contains two semaphores.

This can lead to run-away file descriptor allocation and an eventual
hard limit leading to various undefined behaviours.

Bug: 27731859
Change-Id: If9ce64b711083bf474d217aa537220065722ad3a

Do not leak eventfd's for RFCOMM/AVDT/L2CAP

Various users of fixed_queue's do not free the lists properly after
using them. As a result two event fd's leak for each fixed_queue that
is not free'd, as fixed_queue contains two semaphores.

This can lead to run-away file descriptor allocation and an eventual
hard limit leading to various undefined behaviours.

Bug: 27731859
Change-Id: If9ce64b711083bf474d217aa537220065722ad3a

Log error code on wakelock acquisition/release failure
am: f4f3b60

* commit 'f4f3b602395c8961fce5937f8e97a57db276b978':
  Log error code on wakelock acquisition/release failure

Log error code on wakelock acquisition/release failure

Bug: 27721443
Change-Id: I8ac9e738d916bfdcf125e33c0e09f90cc8d26d55

Merge "Use audio_session_t consistently" into nyc-dev
am: 1b1bcbb

* commit '1b1bcbb3b0bc812410a3ee4f798570d40e0463bc':
  Use audio_session_t consistently

DO NOT MERGE Check size of pin before replying
am: fcc536a

* commit 'fcc536af189dfe534e5e7c6b352ee114de71b3a1':
  DO NOT MERGE Check size of pin before replying

Merge "Use audio_session_t consistently" into nyc-dev

Merge "Wait for UHID interface on boot"
am: 1302ca7

* commit '1302ca714a1fc812c392ab841253b5731e3c110b':
  Wait for UHID interface on boot

Merge "Wait for UHID interface on boot"

Don\'t send AT+BCC if local device doesn\'t support codec negotiation.
am: e79eaef

* commit 'e79eaef05e0778917dcc77d95292370d5aa4491e':
  Don't send AT+BCC if local device doesn't support codec negotiation.

Don't send AT+BCC if local device doesn't support codec negotiation.

The current code only checked if the peer supported codec negotiation
before sending AT+BCC. But if codec negotation is disabled on the
local device, we should definitely not send AT+BCC.

Bug: 27213013
Change-Id: Id372a32a61a3d96d5c093bc50594ef0b8c2eff2e

HID service discovery refactor
am: e8187e5

* commit 'e8187e565382ae9afb52a7992e9a6ab1e3108999':
  HID service discovery refactor

Restructure GATTC
am: 5204fd7

* commit '5204fd71b3ca799942c45a36548faeba4d89477d':
  Restructure GATTC

HID service discovery refactor

This patch changes how HID is initialized.
Up till now, HID discovery was intricately chained,
to make sure two operations are not scheduled at same time.
From now on, all discovery is done and all operations are
scheduled for execution right away. There is separate set
of functions making sure all operations will be executed
without collisions.

This patch also removes BTA_HH_SCPP_INST_DEF, which was
used as hardcoded service id.

Bug: 27698756
Change-Id: I30a7388e129844c5e4727b2ed2b7c05867cc73a8

Restructure GATTC

This patch changes how GATTC is structured. Up till now, it
contained tBTA_GATTC_CACHE and tBTA_GATTC_CACHE_ATTR. Those
structures were so ugly, someone hide them inside internal
header, to make sure noone would ever access them from outside
GATTC. They are now replaced with:
tBTA_GATTC_SERVICE,
tBTA_GATTC_CHARACTERISTIC,
tBTA_GATTC_DESCRIPTOR and
tBTA_GATTC_INCLUDED_SVC.

Those looks much better, and were made globally avaliable.
Thanks to way they're build, we no longer need set of access
methods, which were also very ugly:
BTA_GATTC_GetFirstChar
BTA_GATTC_GetNextChar
BTA_GATTC_GetFirstIncludedService
etc.

This patch breaks HID, DO NOT SUBMIT without HID refactor.

Bug: 27455533
Change-Id: Ic42cfff175e0cc1a0d8e1a1216e2b4b756cbf77d

Add LE L2CAP Connection Oriented Channel
am: 6721232

* commit '6721232129f137ab024d9b95fc1094a714bc4c01':
  Add LE L2CAP Connection Oriented Channel

Start listener thread only after successful open call
am: 95b9bc0

* commit '95b9bc0e533545a14cf12877c72e0d96ab4370fd':
  Start listener thread only after successful open call

DO NOT MERGE Check size of pin before replying

If a malicious client set a pin that was too long it would overflow
the pin code memory.

Bug: 27411268
Change-Id: I9197ac6fdaa92a4799dacb6364e04671a39450cc

Add LE L2CAP Connection Oriented Channel

This implementation for LE L2CAP Connection Oriented Channel (a Bluetooth
Core Spec 4.1 feature) has been tested on the following formal and informal
IOP events conducted by BT SIG Sport and Fitness Working Group.

* Informal IOP – Tokyo 2014
* UPF 49 – Berlin 2014
* Formal IOP – Tokyo 2014
* Formal IOP – Santa Clara 2015

Change-Id: Ia6de62f9321a78b98930629f65078884157a9c0d
Signed-off-by: Navin Kochar <navin.kochar@intel.com>

Start listener thread only after successful open call

Use case: Stress tests ON\OFF with different concurrent
scenarios

Failure: Crash in btsnoop_net_listen thread

Root cause: While starting and shutting down the module
we are calling same function update_logging(). While at
the start it is creating listen thread and throwing error
and exiting without closing the listen thread. While
shutdown of module it "should_log" matches with "is_logging"
and it considers cleanup is not required and returns,
leaving Behind the listen thread open. In the next BT ON,
another listen thread is created and causes the crash.

Fix: Started listener thread after prerequisite operations
are successful.

Change-Id: I92ef293d9b061d0ea65a0c41b506870152114bb0

Wait for UHID interface on boot

Poll and sleep to wait for UHID interface on boot to avoid losing
initial input event after boot.

Bug: 27442119
Change-Id: Ie2141c94624363385951c8316ad854d6ae2e82ec

Allocate sufficient memory for FCS
am: f6a64de84f

* commit 'f6a64de84ff3045d6b4e5a9de44da947f597fbb7':
  Allocate sufficient memory for FCS

Allocate sufficient memory for FCS

Use Case: Hit LeakCanary detection during HDP

Failure: Crash while running LeakCanary detection.

Root cause: Did not have sufficient memory for data frame
which lead to crash.

Fix: Allocating sufficient memory for the data frame
which includes FCS in the header field.

Change-Id: I3bb862f29ecc854423796dc89820ba21c5db6050

osi: add osi_rand()
am: 093f013ab8

* commit '093f013ab883ed2ffeef875b17fd940dd92af659':
  osi: add osi_rand()

osi: add osi_rand()

Provide a replacement for rand() which has a better random source.

Change-Id: I38a8c74d86b89ec160b43b0f648f53b463be89bb

Removed unused function bta_sys_vs_hdl()
am: bea3130893

* commit 'bea31308930f20aac1f7422f262e509e9f51f084':
  Removed unused function bta_sys_vs_hdl()

Removed unused function bta_sys_vs_hdl()

Change-Id: Iaf199d418aafc0b518df2f7febfa1c9e29f1fa42

Removed duplicate (re)define of BTA_DM_SDP_DB_SIZE
am: f958ee655d

* commit 'f958ee655d67b78018a56a593b6e669ca543991e':
  Removed duplicate (re)define of BTA_DM_SDP_DB_SIZE

Removed duplicate (re)define of BTA_DM_SDP_DB_SIZE

Removed conditional duplicate re-define of BTA_DM_SDP_DB_SIZE.
BTA_DM_SDP_DB_SIZE has been already (conditionally) defined to 8000
in file include/bt_target.h

The conditional re-define of BTA_DM_SDP_DB_SIZE to a smaller
value (250) in file bta/dm/bta_dm_int.h should be removed, because
it is confusing and error-prone.

Change-Id: Id10a45562c7c57fd0a9131ef50b7b7a5c2801ec5

Free allocation tracker hash map entries to reduce memory pressure
am: 2bea757aba

* commit '2bea757aba0d0d84865d7d0d675d8bcf31f90b22':
  Free allocation tracker hash map entries to reduce memory pressure

Free allocation tracker hash map entries to reduce memory pressure
am: bb9148eda2

* commit 'bb9148eda25d7d39399fd837d4b13be9b4ecccb4':
  Free allocation tracker hash map entries to reduce memory pressure

Free allocation tracker hash map entries to reduce memory pressure

Use case:
A2DP playback becomes choppy after long hours of testing.

Steps:
1. installed 3rd party app that performs recurring LE scans.
2. Leave phone to run overnight.
3. After 24h+, connect a Bluetooth headset and music is very choppy.

Root cause:
The memory allocation tracker has support for double free check
where it stores all allocations in hash map but it never get
removed to track double free issue, A2DP memory allocation is
taking more time and leading to choppyness.

Fix:
Free the hash map entry to avoid memory leak from
allocation tracker.

Bug: 27621407
Change-Id: If57bd0b89c1d9b99c4f066c537a9e0fa49bc093f

Free allocation tracker hash map entries to reduce memory pressure

Use case:
A2DP playback becomes choppy after long hours of testing.

Steps:
1. installed 3rd party app that performs recurring LE scans.
2. Leave phone to run overnight.
3. After 24h+, connect a Bluetooth headset and music is very choppy.

Root cause:
The memory allocation tracker has support for double free check
where it stores all allocations in hash map but it never get
removed to track double free issue, A2DP memory allocation is
taking more time and leading to choppyness.

Fix:
Free the hash map entry to avoid memory leak from
allocation tracker.

Bug: 27621407
Change-Id: If57bd0b89c1d9b99c4f066c537a9e0fa49bc093f

Add pair event device class and fix timestamp
am: fecf606348

* commit 'fecf606348a0751e0be30e376c9817195afe68ca':
  Add pair event device class and fix timestamp

Add pair event device class and fix timestamp

Added a function to get the device class to use with pair events.
Also update check_cod_hid as it was only being used with COD_HID_MAJOR.
Also fixed an issue with pair event timestamps where they were
using the wrong clock.

Bug: 27499938
Change-Id: I0805d7a3f82b28e7d7e73d80818948843f51a82b

Merge "Add pair event device class and fix timestamp"
am: 00ada513e7

* commit '00ada513e7a034ac372557de5ef8097fc99a9109':
  Add pair event device class and fix timestamp

Merge "Add pair event device class and fix timestamp"

Add pair event device class and fix timestamp

Added a function to get the device class to use with pair events.
Also update check_cod_hid as it was only being used with COD_HID_MAJOR.
Also fixed an issue with pair event timestamps where they were
using the wrong clock.

Bug: 27499938
Change-Id: I0805d7a3f82b28e7d7e73d80818948843f51a82b

Fix startup error when no config file provided
am: e919eda232

* commit 'e919eda23208efd6c8d94e27d973632551412b95':
  Fix startup error when no config file provided

Fix startup error when no config file provided

If bluetoothtbd was started without config file
present, it would die on asserts about missing config.
From now it'll proceed with default settings.

Change-Id: I46c35fb2ed71efa49430206da13a19ca7df44a64

Merge "Linux vendor driver linkage fixes"
am: a15f218c9c

* commit 'a15f218c9c3164d943bbf32b463d6a9d866e76bd':
  Linux vendor driver linkage fixes

Merge "Linux vendor driver linkage fixes"

Linux vendor driver linkage fixes

Linux driver depends on osi, so it must be linked with it.
It also must export bt_vendor_interface_t, and named properly,
otherwise it is not being recognized as vendor library.

Bug: 27589244
Change-Id: Ib6817b11e4226f2e289ff0d521daca059ca9aca6

Use audio_session_t consistently

Bug: 27562099
Change-Id: I10780c8bfc84af568795a2c15557020fbe89a1c4

Merge "Fix build error caused by missing LOG_TAG"
am: a34a404e3d

* commit 'a34a404e3d59b7c25a64471b4502f37c2c75d4de':
  Fix build error caused by missing LOG_TAG

Merge "Add missing \'cd third_party\' step"
am: 8b1e8c3154

* commit '8b1e8c3154884451a4d71db4583334ba1c09729b':
  Add missing 'cd third_party' step

Add missing log tag
am: 85e22b32ec

* commit '85e22b32ec3482a09dc019ee6fd096a31fd1bf64':
  Add missing log tag

Merge "Fix build error caused by missing LOG_TAG"

Merge "Add missing 'cd third_party' step"

Add missing log tag

Change-Id: Icae6cf63de8d298fb12451887594abe24b026c77

Remove bta_gattc_id2handle
am: 14a73093ba

* commit '14a73093ba78d237959f625fd27bd4edd9f0ff48':
  Remove bta_gattc_id2handle

Use attribute handle instead of id in GATT cache
am: bc5d65496d

* commit 'bc5d65496d7dc9d87d4fe3e3c92dadb043559786':
  Use attribute handle instead of id in GATT cache

Remove bta_gattc_id2handle

Now that the instance id is equal to handle, we no longer need this
mapping function.

Bug: 27455533
Change-Id: I6760f8fc81460df8520d4ebf1899ed256ce5ad1e

Use attribute handle instead of id in GATT cache

Currently we generate an id for each GATT attribute. Those ids
are used to differentiate between attributes that have the same
UUID. We need to get rid of those ids because it won't work with
partial GATT database updates due to "service change"
notifications. Instead, attribute handles, which are unique
across the whole GATT database will be used.

This is first patch in series to achieve that. It gets rid
of id generation, and just assigns handle in place of ids inside
of gatt cache. Upcoming patches will change the GATT cache API
and eventually get rid of "id" completly.

Bug: 27455533
Change-Id: I74c10872ecc223329b8ac7fef4085963bf272624

HID multi-device refactoring
am: 22aaded594

* commit '22aaded594fab42d5b30354a94c51d0b8666089b':
  HID multi-device refactoring

SDP: Restrict attribute length to SDP_MAX_ATTR_LEN
am: 15b8ab4b33

* commit '15b8ab4b338eed3fa35d8f4751eeab2a6cdcd3ef':
  SDP: Restrict attribute length to SDP_MAX_ATTR_LEN

HID multi-device refactoring

We don't really support multiple HID services in same device.
We shouldn't use default value 0 for service id.
We shouldn't merge service id and characteristic id into one field,
as it's done not right right now.

Bug: 27455533
Change-Id: I46cc1b62bb2c7393e62f3b1a8e4650706954a066

SDP: Restrict attribute length to SDP_MAX_ATTR_LEN

Use case:
Bluetooth crash during the SNS testing.

Root Cause:
In sdpu_build_partial_attrib_entry there is no length check, it can
exceed the maximum allowable value and lead to a crash in memcpy.

Fix:
Restrict remaining attribute length to SDP_MAX_ATTR_LEN
in sdpu_build_partial_attrib_entry.

Change-Id: I5bde6868b82e50b3c09cf94277dabb4862f0a2a8

bluetooth-cli: add support for --exec am: 5d80cb938d
am: e228c9dbcd

* commit 'e228c9dbcd98a6bdfd972410d8eaeab885294acc':
  bluetooth-cli: add support for --exec

bluetooth-cli: add support for --exec
am: 5d80cb938d

* commit '5d80cb938d7fb3312870a047749cc839ba5bafca':
  bluetooth-cli: add support for --exec

bluetooth-cli: add support for --exec

Adds the --exec and -e arguments, which execute commands when the
cli starts.

This also adds support for chaining commands at the command line by
separating them with ';' characters.

Change-Id: Ic2bf8a908fbb7fb4cf775810754571fa96d75e87

Fix the build
am: 597ce3bd08

* commit '597ce3bd08533c24cd61be0ef4ad8a237571770f':
  Fix the build

Fix the build

Change-Id: Ib226f087a76ddd426e40f4d6d94f2ed758660c36

Move focus handling entirely to component calling fluoride.
am: 547b7b5257

* commit '547b7b52578fddbd9e42ce1cc7e7bba9dded1d7d':
  Move focus handling entirely to component calling fluoride.

Move focus handling entirely to component calling fluoride.

Audio focus handling for A2DP sink currently split between Fluoride and
the component sitting on top of it. Since this is not the best place to
decide policies, we move all such decisions to the upper layer.

Bug: b/26928143

Change-Id: I67a1a2a50782289e7e31447b01ee29a21bc8dac2

GATT cache storage refactoring am: c10b209f62
am: d7fba0cad0

* commit 'd7fba0cad007a5d31798983660b25b03e6794a49':
  GATT cache storage refactoring

Ignore LCB entries that migth have been released while queued am: fb2105f348
am: 64a81c78c5

* commit '64a81c78c58dfe165ee4de77b7694bde8a86f48e':
  Ignore LCB entries that migth have been released while queued

GATT cache storage refactoring
am: c10b209f62

* commit 'c10b209f62a52d2a782d4d8c97734b6c142f2567':
  GATT cache storage refactoring

Ignore LCB entries that migth have been released while queued
am: 0c33fde894

* commit '0c33fde894b566484b9956560e8b701b82fa04cc':
  Ignore LCB entries that migth have been released while queued

GATT cache storage refactoring

Currently when storing GATT cache into file, we save it piece
after piece, max 10 elements at once. This is not necessary,
we have enough space to just save all at once.
Also the logic doesn't need to be so complicated - no need
to state machine for such simple task.

Bug: 27455533
Change-Id: I82923dbedfb317835e45e13e0ea28151eb524d37

Ignore LCB entries that migth have been released while queued

Function btm_send_pending_direct_conn() dequeues tL2C_LCB
entries from the conn_pending_q queue and calls the processing
function.
However, there is the possilility of a race condition
and releasing the entry by l2cu_release_lcb() before it is processed.
Now tL2C_LCB that are not in_use are ignored.

Bug: 27418154
Change-Id: I4b5fda605a60c8aebbf5587bb12dd74ac5d28f68

Ignore LCB entries that migth have been released while queued
am: fb2105f348

* commit 'fb2105f3488e48f2d908b13a33f47e34f6e3f78d':
  Ignore LCB entries that migth have been released while queued

Ignore LCB entries that migth have been released while queued

Function btm_send_pending_direct_conn() dequeues tL2C_LCB
entries from the conn_pending_q queue and calls the processing
function.
However, there is the possilility of a race condition
and releasing the entry by l2cu_release_lcb() before it is processed.
Now tL2C_LCB that are not in_use are ignored.

Bug: 27418154
Change-Id: I4b5fda605a60c8aebbf5587bb12dd74ac5d28f68

Fix build error caused by missing LOG_TAG

FAILED: gcc -MMD -MF obj/hci/src/libhci.hci_packet_parser.o.d -D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -DHAS_NO_BDROID_BUILDCFG -DLOG_NDEBUG=1 -DEXPORT_SYMBOL=__attribute__\(\(visibility\(\"default\"\)\)\) -DKERNEL_MISSING_CLOCK_BOOTTIME_ALARM=TRUE -DOS_GENERIC -I../../hci/include -I../.. -I../../include -I../../bta/include -I../../btcore/include -I../../stack/include -I/home/patrick/src/fluoride/libhardware/include -I/home/patrick/src/fluoride/core/include -I/home/patrick/src/fluoride/media/audio/include -g -O0 -fpic -fdata-sections -ffunction-sections -fvisibility=hidden -std=c99 -c ../../hci/src/hci_packet_parser.c -o obj/hci/src/libhci.hci_packet_parser.o
In file included from ../../hci/src/hci_packet_parser.c:28:0:
../../hci/src/hci_packet_parser.c: In function ‘read_command_complete_header’:
../../hci/src/hci_packet_parser.c:239:43: error: expected ‘)’ before ‘__func__’
     LOG_ERROR("%s: return status - 0x%x", __func__, status);

Change-Id: I6ca721ba24ba3170d701f606f0676c5c4e707cf8

Add missing 'cd third_party' step

Update device type in NVRAM correctly during inquiry am: ab7418f58c
am: 68bb3ecf9c

* commit '68bb3ecf9ca7bf59694193e3a8a4b1efe76a47ad':
  Update device type in NVRAM correctly during inquiry

Update device type in NVRAM correctly during inquiry
am: ab7418f58c

* commit 'ab7418f58c84216d57b7c0d553f09f5748badd82':
  Update device type in NVRAM correctly during inquiry

Default missing DevType and DevClass in NVRAM. am: 3b6ef54cd2
am: 4b8cdc78d4

* commit '4b8cdc78d4d665dafc7903877cb7c362807ec782':
  Default missing DevType and DevClass in NVRAM.

GATT: Initiate L2CAP disconnect only when the channel is open am: bf955cb7e5
am: e65e09f115

* commit 'e65e09f11598af404915111cf3b027c6d5890081':
  GATT: Initiate L2CAP disconnect only when the channel is open

Update device type in NVRAM correctly during inquiry

If the NVRAM device type for a device is not dual mode and the inquiry
result is received for BLE or BR/EDR transport, then this patch updates
the NVRAM device type to dual mode.

Change-Id: I688e7c5e3bd8d844288d095cb97c2e55ddf652c6

Default missing DevType and DevClass in NVRAM.
am: 3b6ef54cd2

* commit '3b6ef54cd262da6a41a16dafe361040740024717':
  Default missing DevType and DevClass in NVRAM.

GATT: Initiate L2CAP disconnect only when the channel is open
am: bf955cb7e5

* commit 'bf955cb7e50e1047452e741effa504c8aefc8a4f':
  GATT: Initiate L2CAP disconnect only when the channel is open

Default missing DevType and DevClass in NVRAM.

Missing DevType and DevClass in NVRAM were not properly set to a
default values previously.

Remove btla-specific comments am: 1be7ae1a08
am: a9b36318f2

* commit 'a9b36318f287a1a3ef4a8dc253b4313dadf00721':
  Remove btla-specific comments