You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by cc...@apache.org on 2015/12/19 03:10:58 UTC

[1/2] incubator-mynewt-larva git commit: GAP discoverable and connectable procedures.

Repository: incubator-mynewt-larva
Updated Branches:
  refs/heads/master de6469591 -> 76f96f882


GAP discoverable and connectable procedures.


Project: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/commit/fd3c7220
Tree: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/tree/fd3c7220
Diff: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/diff/fd3c7220

Branch: refs/heads/master
Commit: fd3c7220fd9794b907b532cdc53ca1d0aa3f1023
Parents: de64695
Author: Christopher Collins <cc...@gmail.com>
Authored: Fri Dec 18 18:08:55 2015 -0800
Committer: Christopher Collins <cc...@gmail.com>
Committed: Fri Dec 18 18:10:23 2015 -0800

----------------------------------------------------------------------
 net/nimble/host/include/host/host_hci.h     |   1 +
 net/nimble/host/src/ble_gap_conn.c          | 281 +++++++++++++++--------
 net/nimble/host/src/host_hci_cmd.c          |  15 ++
 net/nimble/host/src/test/ble_hs_conn_test.c |  19 +-
 net/nimble/host/src/test/ble_hs_test_util.c |  58 +++++
 net/nimble/host/src/test/ble_hs_test_util.h |   5 +
 net/nimble/include/nimble/hci_common.h      |  37 +--
 7 files changed, 284 insertions(+), 132 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/blob/fd3c7220/net/nimble/host/include/host/host_hci.h
----------------------------------------------------------------------
diff --git a/net/nimble/host/include/host/host_hci.h b/net/nimble/host/include/host/host_hci.h
index 0ee00f8..dde0f3f 100644
--- a/net/nimble/host/include/host/host_hci.h
+++ b/net/nimble/host/include/host/host_hci.h
@@ -44,6 +44,7 @@ int host_hci_cmd_le_read_whitelist(void);
 int host_hci_cmd_le_add_to_whitelist(uint8_t *addr, uint8_t addr_type);
 int host_hci_cmd_le_rmv_from_whitelist(uint8_t *addr, uint8_t addr_type);
 int host_hci_cmd_reset(void);
+int host_hci_cmd_read_adv_pwr(void);
 
 int host_hci_set_buf_size(uint16_t pktlen, uint8_t max_pkts);
 

http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/blob/fd3c7220/net/nimble/host/src/ble_gap_conn.c
----------------------------------------------------------------------
diff --git a/net/nimble/host/src/ble_gap_conn.c b/net/nimble/host/src/ble_gap_conn.c
index dfa1a6f..d42dd49 100644
--- a/net/nimble/host/src/ble_gap_conn.c
+++ b/net/nimble/host/src/ble_gap_conn.c
@@ -27,23 +27,24 @@
 #include "ble_gatt_priv.h"
 #include "ble_gap_conn.h"
 
-#define BLE_GAP_CONN_STATE_IDLE                             0
+#define BLE_GAP_CONN_STATE_IDLE                             UINT8_MAX
 
-#define BLE_GAP_CONN_STATE_M_GEN_DISC_PENDING               1
-#define BLE_GAP_CONN_STATE_M_GEN_DISC_PARAMS                2
-#define BLE_GAP_CONN_STATE_M_GEN_DISC_PARAMS_ACKED          3
-#define BLE_GAP_CONN_STATE_M_GEN_DISC_ENABLE                4
-#define BLE_GAP_CONN_STATE_M_GEN_DISC_ENABLE_ACKED          5
+#define BLE_GAP_CONN_M_STATE_GEN_DISC_PENDING               0
+#define BLE_GAP_CONN_M_STATE_GEN_DISC_PARAMS                1
+#define BLE_GAP_CONN_M_STATE_GEN_DISC_PARAMS_ACKED          2
+#define BLE_GAP_CONN_M_STATE_GEN_DISC_ENABLE                3
+#define BLE_GAP_CONN_M_STATE_GEN_DISC_ENABLE_ACKED          4
 
-#define BLE_GAP_CONN_STATE_M_DIRECT_PENDING                 6
-#define BLE_GAP_CONN_STATE_M_DIRECT_UNACKED                 7
-#define BLE_GAP_CONN_STATE_M_DIRECT_ACKED                   8
+#define BLE_GAP_CONN_M_STATE_DIRECT_PENDING                 5
+#define BLE_GAP_CONN_M_STATE_DIRECT_UNACKED                 6
+#define BLE_GAP_CONN_M_STATE_DIRECT_ACKED                   7
 
-#define BLE_GAP_CONN_STATE_S_PENDING                        1
-#define BLE_GAP_CONN_STATE_S_PARAMS                         2
-#define BLE_GAP_CONN_STATE_S_PARAMS_ACKED                   3
-#define BLE_GAP_CONN_STATE_S_ENABLE                         4
-#define BLE_GAP_CONN_STATE_S_ENABLE_ACKED                   5
+#define BLE_GAP_CONN_S_STATE_PARAMS                         0
+#define BLE_GAP_CONN_S_STATE_POWER                          1
+#define BLE_GAP_CONN_S_STATE_ADV_DATA                       2
+#define BLE_GAP_CONN_S_STATE_RSP_DATA                       3
+#define BLE_GAP_CONN_S_STATE_ENABLE                         4
+#define BLE_GAP_CONN_S_STATE_MAX                            5
 
 /** 30 ms. */
 #define BLE_GAP_ADV_FAST_INTERVAL1_MIN      (30 * 1000 / BLE_HCI_ADV_ITVL)
@@ -72,24 +73,35 @@
 /** 10.24 seconds. */
 #define BLE_GAP_GEN_DISC_SCAN_MIN           (10.24 * 1000)
 
-#define BLE_GAP_CONN_S_TYPE_NONE                            (-1)
-#define BLE_GAP_CONN_S_TYPE_NON_DISC                        0
-#define BLE_GAP_CONN_S_TYPE_LTD_DISC                        1
-#define BLE_GAP_CONN_S_TYPE_GEN_DISC                        2
-#define BLE_GAP_CONN_S_TYPE_NON_CONN                        3
-#define BLE_GAP_CONN_S_TYPE_DIR_CONN                        4
-#define BLE_GAP_CONN_S_TYPE_UND_CONN                        5
-#define BLE_GAP_CONN_S_TYPE_MAX                             6
-
-#define BLE_GAP_CONN_M_TYPE_NONE                            (-1)
-#define BLE_GAP_CONN_M_TYPE_GEN_DISC                        1
-#define BLE_GAP_CONN_M_TYPE_DIR_CONN                        2
-
-static int ble_gap_conn_master_type;
-static int ble_gap_conn_slave_type;
+#define BLE_GAP_CONN_S_MODE_NONE                            (-1)
+#define BLE_GAP_CONN_S_MODE_NON_DISC                        0
+#define BLE_GAP_CONN_S_MODE_LTD_DISC                        1
+#define BLE_GAP_CONN_S_MODE_GEN_DISC                        2
+#define BLE_GAP_CONN_S_MODE_NON_CONN                        3
+#define BLE_GAP_CONN_S_MODE_DIR_CONN                        4
+#define BLE_GAP_CONN_S_MODE_UND_CONN                        5
+#define BLE_GAP_CONN_S_MODE_MAX                             6
+
+static int ble_gap_conn_adv_params_tx(void *arg);
+static int ble_gap_conn_adv_power_tx(void *arg);
+static int ble_gap_conn_adv_data_tx(void *arg);
+static int ble_gap_conn_adv_rsp_data_tx(void *arg);
+static int ble_gap_conn_adv_enable_tx(void *arg);
+
+static ble_hci_sched_tx_fn * const
+    ble_gap_conn_slave_dispatch[BLE_GAP_CONN_S_STATE_MAX] = {
+
+    [BLE_GAP_CONN_S_STATE_PARAMS]   = ble_gap_conn_adv_params_tx,
+    [BLE_GAP_CONN_S_STATE_POWER]    = ble_gap_conn_adv_power_tx,
+    [BLE_GAP_CONN_S_STATE_ADV_DATA] = ble_gap_conn_adv_data_tx,
+    [BLE_GAP_CONN_S_STATE_RSP_DATA] = ble_gap_conn_adv_rsp_data_tx,
+    [BLE_GAP_CONN_S_STATE_ENABLE]   = ble_gap_conn_adv_enable_tx,
+};
+
+static int ble_gap_conn_slave_mode;
 
 static struct hci_adv_params
-    ble_gap_conn_adv_params[BLE_GAP_CONN_S_TYPE_MAX];
+    ble_gap_conn_adv_params[BLE_GAP_CONN_S_MODE_MAX];
 
 static ble_gap_connect_fn *ble_gap_conn_cb;
 static void *ble_gap_conn_arg;
@@ -101,6 +113,8 @@ static uint8_t ble_gap_conn_slave_addr[BLE_DEV_ADDR_LEN];
 static struct os_callout_func ble_gap_conn_master_timer;
 static struct os_callout_func ble_gap_conn_slave_timer;
 
+/* XXX: Need to clear slave mode when we join a master. */
+
 /*****************************************************************************
  * $misc                                                                     *
  *****************************************************************************/
@@ -172,14 +186,13 @@ static void
 ble_gap_conn_master_reset_state(void)
 {
     ble_gap_conn_master_state = BLE_GAP_CONN_STATE_IDLE;
-    ble_gap_conn_master_type = BLE_GAP_CONN_M_TYPE_NONE;
 }
 
 static void
 ble_gap_conn_slave_reset_state(void)
 {
     ble_gap_conn_slave_state = BLE_GAP_CONN_STATE_IDLE;
-    ble_gap_conn_slave_type = BLE_GAP_CONN_S_TYPE_NONE;
+    ble_gap_conn_slave_mode = BLE_GAP_CONN_S_MODE_NONE;
 }
 
 /**
@@ -200,18 +213,18 @@ ble_gap_conn_master_failed(uint8_t status)
     ble_gap_conn_master_reset_state();
 
     switch (old_state) {
-    case BLE_GAP_CONN_STATE_M_GEN_DISC_PENDING:
-    case BLE_GAP_CONN_STATE_M_GEN_DISC_PARAMS:
-    case BLE_GAP_CONN_STATE_M_GEN_DISC_PARAMS_ACKED:
-    case BLE_GAP_CONN_STATE_M_GEN_DISC_ENABLE:
-    case BLE_GAP_CONN_STATE_M_GEN_DISC_ENABLE_ACKED:
+    case BLE_GAP_CONN_M_STATE_GEN_DISC_PENDING:
+    case BLE_GAP_CONN_M_STATE_GEN_DISC_PARAMS:
+    case BLE_GAP_CONN_M_STATE_GEN_DISC_PARAMS_ACKED:
+    case BLE_GAP_CONN_M_STATE_GEN_DISC_ENABLE:
+    case BLE_GAP_CONN_M_STATE_GEN_DISC_ENABLE_ACKED:
         event.type = BLE_GAP_CONN_EVENT_TYPE_SCAN_DONE;
         ble_gap_conn_call_cb(&event);
         break;
 
-    case BLE_GAP_CONN_STATE_M_DIRECT_PENDING:
-    case BLE_GAP_CONN_STATE_M_DIRECT_UNACKED:
-    case BLE_GAP_CONN_STATE_M_DIRECT_ACKED:
+    case BLE_GAP_CONN_M_STATE_DIRECT_PENDING:
+    case BLE_GAP_CONN_M_STATE_DIRECT_UNACKED:
+    case BLE_GAP_CONN_M_STATE_DIRECT_ACKED:
         ble_gap_conn_notify_connect(status, NULL);
         break;
 
@@ -260,7 +273,7 @@ ble_gap_conn_rx_disconn_complete(struct hci_disconn_complete *evt)
 int
 ble_gap_conn_master_in_progress(void)
 {
-    return ble_gap_conn_master_type != BLE_GAP_CONN_M_TYPE_NONE;
+    return ble_gap_conn_master_state != BLE_GAP_CONN_STATE_IDLE;
 }
 
 /**
@@ -269,14 +282,14 @@ ble_gap_conn_master_in_progress(void)
 int
 ble_gap_conn_slave_in_progress(void)
 {
-    return ble_gap_conn_slave_type != BLE_GAP_CONN_S_TYPE_NONE;
+    return ble_gap_conn_slave_state != BLE_GAP_CONN_STATE_IDLE;
 }
 
 static int
 ble_gap_conn_accept_new_conn(uint8_t *addr)
 {
     switch (ble_gap_conn_master_state) {
-    case BLE_GAP_CONN_STATE_M_DIRECT_ACKED:
+    case BLE_GAP_CONN_M_STATE_DIRECT_ACKED:
         if (memcmp(ble_gap_conn_master_addr, addr, BLE_DEV_ADDR_LEN) == 0) {
             os_callout_stop(&ble_gap_conn_master_timer.cf_c);
             ble_gap_conn_master_reset_state();
@@ -285,7 +298,7 @@ ble_gap_conn_accept_new_conn(uint8_t *addr)
     }
 
     switch (ble_gap_conn_slave_state) {
-    case BLE_GAP_CONN_STATE_S_ENABLE_ACKED:
+    case BLE_GAP_CONN_S_STATE_MAX:
         if (memcmp(ble_gap_conn_slave_addr, addr, BLE_DEV_ADDR_LEN) == 0) {
             os_callout_stop(&ble_gap_conn_master_timer.cf_c);
             ble_gap_conn_slave_reset_state();
@@ -303,7 +316,7 @@ ble_gap_conn_rx_adv_report(struct ble_gap_conn_adv_rpt *rpt)
     struct ble_gap_conn_event event;
 
     switch (ble_gap_conn_master_state) {
-    case BLE_GAP_CONN_STATE_M_GEN_DISC_ENABLE_ACKED:
+    case BLE_GAP_CONN_M_STATE_GEN_DISC_ENABLE_ACKED:
         event.type = BLE_GAP_CONN_EVENT_TYPE_ADV_RPT;
         event.adv = *rpt;
         ble_gap_conn_call_cb(&event);
@@ -377,7 +390,7 @@ ble_gap_conn_master_timer_exp(void *arg)
     assert(ble_gap_conn_master_in_progress());
 
     switch (ble_gap_conn_master_state) {
-    case BLE_GAP_CONN_STATE_M_GEN_DISC_ENABLE_ACKED:
+    case BLE_GAP_CONN_M_STATE_GEN_DISC_ENABLE_ACKED:
         /* When a discovery procedure times out, it is not a failure. */
         status = 0;
         break;
@@ -402,72 +415,144 @@ ble_gap_conn_slave_timer_exp(void *arg)
  *****************************************************************************/
 
 static void
-ble_gap_conn_adv_ack_enable(struct ble_hci_ack *ack, void *arg)
+ble_gap_conn_adv_next_state(void)
 {
-    assert(ble_gap_conn_slave_state == BLE_GAP_CONN_STATE_S_ENABLE);
+    ble_hci_sched_tx_fn *tx_fn;
+    int rc;
+
+    ble_gap_conn_slave_state++;
+    if (ble_gap_conn_slave_state != BLE_GAP_CONN_S_STATE_MAX) {
+        tx_fn = ble_gap_conn_slave_dispatch[ble_gap_conn_slave_state];
+        assert(tx_fn != NULL);
+
+        rc = ble_hci_sched_enqueue(tx_fn, NULL);
+        if (rc != 0) {
+            ble_gap_conn_slave_failed(rc);
+        }
+    }
+}
 
+static void
+ble_gap_conn_adv_ack(struct ble_hci_ack *ack, void *arg)
+{
     if (ack->bha_status != BLE_ERR_SUCCESS) {
         ble_gap_conn_slave_failed(ack->bha_status);
     } else {
-        ble_gap_conn_slave_state = BLE_GAP_CONN_STATE_S_ENABLE_ACKED;
+        ble_gap_conn_adv_next_state();
     }
 }
 
 static int
-ble_gap_conn_adv_tx_enable(void *arg)
+ble_gap_conn_adv_enable_tx(void *arg)
 {
     int rc;
 
-    assert(ble_gap_conn_slave_state == BLE_GAP_CONN_STATE_S_PARAMS_ACKED);
+    assert(ble_gap_conn_slave_state == BLE_GAP_CONN_S_STATE_ENABLE);
 
+    ble_hci_ack_set_callback(ble_gap_conn_adv_ack, NULL);
     rc = host_hci_cmd_le_set_adv_enable(1);
     if (rc != BLE_ERR_SUCCESS) {
         ble_gap_conn_slave_failed(rc);
         return 1;
     }
 
-    ble_hci_ack_set_callback(ble_gap_conn_adv_ack_enable, NULL);
-    ble_gap_conn_slave_state = BLE_GAP_CONN_STATE_S_ENABLE;
+    return 0;
+}
+
+static int
+ble_gap_conn_adv_rsp_data_tx(void *arg)
+{
+    uint8_t rsp_data[BLE_HCI_MAX_SCAN_RSP_DATA_LEN] = { 0 }; /* XXX */
+    int rc;
+
+    assert(ble_gap_conn_slave_state == BLE_GAP_CONN_S_STATE_RSP_DATA);
+
+    ble_hci_ack_set_callback(ble_gap_conn_adv_ack, NULL);
+    rc = host_hci_cmd_le_set_scan_rsp_data(rsp_data, sizeof rsp_data);
+    if (rc != 0) {
+        ble_gap_conn_slave_failed(rc);
+        return 1;
+    }
 
     return 0;
 }
 
-static void
-ble_gap_conn_adv_ack_params(struct ble_hci_ack *ack, void *arg)
+static int
+ble_gap_conn_adv_data_tx(void *arg)
 {
+    uint8_t adv_data[BLE_HCI_MAX_ADV_DATA_LEN] = { 0 }; /* XXX */
     int rc;
 
-    assert(ble_gap_conn_slave_state == BLE_GAP_CONN_STATE_S_PARAMS);
+    assert(ble_gap_conn_slave_state == BLE_GAP_CONN_S_STATE_ADV_DATA);
+
+    ble_hci_ack_set_callback(ble_gap_conn_adv_ack, NULL);
+    rc = host_hci_cmd_le_set_adv_data(adv_data, sizeof adv_data);
+    if (rc != 0) {
+        ble_gap_conn_slave_failed(rc);
+        return 1;
+    }
+
+    return 0;
+}
+
+static void
+ble_gap_conn_adv_power_ack(struct ble_hci_ack *ack, void *arg)
+{
+    int8_t power_level;
 
     if (ack->bha_status != BLE_ERR_SUCCESS) {
         ble_gap_conn_slave_failed(ack->bha_status);
         return;
     }
 
-    ble_gap_conn_slave_state = BLE_GAP_CONN_STATE_S_PARAMS_ACKED;
+    if (ack->bha_params_len != BLE_HCI_ADV_CHAN_TXPWR_ACK_PARAM_LEN) {
+        ble_gap_conn_slave_failed(1 /*XXX*/);
+        return;
+    }
+
+    power_level = ack->bha_params[1];
+    if (power_level < BLE_HCI_ADV_CHAN_TXPWR_MIN ||
+        power_level > BLE_HCI_ADV_CHAN_TXPWR_MAX) {
+
+        /* XXX: Probably can do something nicer than abort the entire
+         * procedure.
+         */
+        ble_gap_conn_slave_failed(1 /*XXX*/);
+        return;
+    }
+
+    /* XXX: Save power level value so it can be put in the adv. data. */
+
+    ble_gap_conn_adv_next_state();
+}
+
+static int
+ble_gap_conn_adv_power_tx(void *arg)
+{
+    int rc;
 
-    rc = ble_hci_sched_enqueue(ble_gap_conn_adv_tx_enable,
-                               NULL);
+    ble_hci_ack_set_callback(ble_gap_conn_adv_power_ack, NULL);
+    rc = host_hci_cmd_read_adv_pwr();
     if (rc != 0) {
         ble_gap_conn_slave_failed(rc);
-        return;
+        return 1;
     }
+
+    return 0;
 }
 
 static int
-ble_gap_conn_adv_tx_params(void *arg)
+ble_gap_conn_adv_params_tx(void *arg)
 {
     struct hci_adv_params hap;
     int rc;
 
-    assert(ble_gap_conn_slave_state == BLE_GAP_CONN_STATE_S_PENDING);
+    assert(ble_gap_conn_slave_state == BLE_GAP_CONN_S_STATE_PARAMS);
 
-    hap = ble_gap_conn_adv_params[ble_gap_conn_slave_type];
+    hap = ble_gap_conn_adv_params[ble_gap_conn_slave_mode];
     memcpy(hap.peer_addr, ble_gap_conn_slave_addr, sizeof hap.peer_addr);
 
-    ble_gap_conn_slave_state = BLE_GAP_CONN_STATE_S_PARAMS;
-    ble_hci_ack_set_callback(ble_gap_conn_adv_ack_params, NULL);
-
+    ble_hci_ack_set_callback(ble_gap_conn_adv_ack, NULL);
     rc = host_hci_cmd_le_set_adv_params(&hap);
     if (rc != 0) {
         ble_gap_conn_slave_failed(rc);
@@ -482,11 +567,11 @@ ble_gap_conn_adv_initiate(void)
 {
     int rc;
 
-    assert(ble_gap_conn_slave_in_progress());
+    assert(!ble_gap_conn_slave_in_progress());
 
-    ble_gap_conn_slave_state = BLE_GAP_CONN_STATE_S_PENDING;
+    ble_gap_conn_slave_state = BLE_GAP_CONN_S_STATE_PARAMS;
 
-    rc = ble_hci_sched_enqueue(ble_gap_conn_adv_tx_params, NULL);
+    rc = ble_hci_sched_enqueue(ble_gap_conn_adv_params_tx, NULL);
     if (rc != 0) {
         ble_gap_conn_slave_reset_state();
         return rc;
@@ -515,7 +600,7 @@ ble_gap_conn_non_discoverable(void)
         return BLE_HS_EALREADY;
     }
 
-    ble_gap_conn_slave_type = BLE_GAP_CONN_S_TYPE_NON_DISC;
+    ble_gap_conn_slave_mode = BLE_GAP_CONN_S_MODE_NON_DISC;
 
     rc = ble_gap_conn_adv_initiate();
     return rc;
@@ -541,7 +626,7 @@ ble_gap_conn_limited_discoverable(void)
         return BLE_HS_EALREADY;
     }
 
-    ble_gap_conn_slave_type = BLE_GAP_CONN_S_TYPE_LTD_DISC;
+    ble_gap_conn_slave_mode = BLE_GAP_CONN_S_MODE_LTD_DISC;
 
     rc = ble_gap_conn_adv_initiate();
     return rc;
@@ -567,7 +652,7 @@ ble_gap_conn_general_discoverable(void)
         return BLE_HS_EALREADY;
     }
 
-    ble_gap_conn_slave_type = BLE_GAP_CONN_S_TYPE_GEN_DISC;
+    ble_gap_conn_slave_mode = BLE_GAP_CONN_S_MODE_GEN_DISC;
 
     rc = ble_gap_conn_adv_initiate();
     return rc;
@@ -592,7 +677,7 @@ ble_gap_conn_non_connectable(void)
         return BLE_HS_EALREADY;
     }
 
-    ble_gap_conn_slave_type = BLE_GAP_CONN_S_TYPE_NON_CONN;
+    ble_gap_conn_slave_mode = BLE_GAP_CONN_S_MODE_NON_CONN;
 
     rc = ble_gap_conn_adv_initiate();
     return rc;
@@ -625,7 +710,7 @@ ble_gap_conn_direct_connectable(int addr_type, uint8_t *addr)
         return BLE_HS_EALREADY;
     }
 
-    ble_gap_conn_slave_type = BLE_GAP_CONN_S_TYPE_DIR_CONN;
+    ble_gap_conn_slave_mode = BLE_GAP_CONN_S_MODE_DIR_CONN;
     memcpy(ble_gap_conn_slave_addr, addr, BLE_DEV_ADDR_LEN);
 
     rc = ble_gap_conn_adv_initiate();
@@ -652,7 +737,7 @@ ble_gap_conn_undirect_connectable(void)
         return BLE_HS_EALREADY;
     }
 
-    ble_gap_conn_slave_type = BLE_GAP_CONN_S_TYPE_UND_CONN;
+    ble_gap_conn_slave_mode = BLE_GAP_CONN_S_MODE_UND_CONN;
 
     rc = ble_gap_conn_adv_initiate();
     return rc;
@@ -665,12 +750,12 @@ ble_gap_conn_undirect_connectable(void)
 static void
 ble_gap_conn_gen_disc_ack_enable(struct ble_hci_ack *ack, void *arg)
 {
-    assert(ble_gap_conn_master_state == BLE_GAP_CONN_STATE_M_GEN_DISC_ENABLE);
+    assert(ble_gap_conn_master_state == BLE_GAP_CONN_M_STATE_GEN_DISC_ENABLE);
 
     if (ack->bha_status != 0) {
         ble_gap_conn_master_failed(ack->bha_status);
     } else {
-        ble_gap_conn_master_state = BLE_GAP_CONN_STATE_M_GEN_DISC_ENABLE_ACKED;
+        ble_gap_conn_master_state = BLE_GAP_CONN_M_STATE_GEN_DISC_ENABLE_ACKED;
     }
 }
 
@@ -680,9 +765,9 @@ ble_gap_conn_gen_disc_tx_enable(void *arg)
     int rc;
 
     assert(ble_gap_conn_master_state ==
-           BLE_GAP_CONN_STATE_M_GEN_DISC_PARAMS_ACKED);
+           BLE_GAP_CONN_M_STATE_GEN_DISC_PARAMS_ACKED);
 
-    ble_gap_conn_master_state = BLE_GAP_CONN_STATE_M_GEN_DISC_ENABLE;
+    ble_gap_conn_master_state = BLE_GAP_CONN_M_STATE_GEN_DISC_ENABLE;
     ble_hci_ack_set_callback(ble_gap_conn_gen_disc_ack_enable, NULL);
 
     rc = host_hci_cmd_le_set_scan_enable(1, 0);
@@ -699,14 +784,14 @@ ble_gap_conn_gen_disc_ack_params(struct ble_hci_ack *ack, void *arg)
 {
     int rc;
 
-    assert(ble_gap_conn_master_state == BLE_GAP_CONN_STATE_M_GEN_DISC_PARAMS);
+    assert(ble_gap_conn_master_state == BLE_GAP_CONN_M_STATE_GEN_DISC_PARAMS);
 
     if (ack->bha_status != 0) {
         ble_gap_conn_master_failed(ack->bha_status);
         return;
     }
 
-    ble_gap_conn_master_state = BLE_GAP_CONN_STATE_M_GEN_DISC_PARAMS_ACKED;
+    ble_gap_conn_master_state = BLE_GAP_CONN_M_STATE_GEN_DISC_PARAMS_ACKED;
 
     rc = ble_hci_sched_enqueue(ble_gap_conn_gen_disc_tx_enable, NULL);
     if (rc != 0) {
@@ -720,9 +805,9 @@ ble_gap_conn_gen_disc_tx_params(void *arg)
 {
     int rc;
 
-    assert(ble_gap_conn_master_state == BLE_GAP_CONN_STATE_M_GEN_DISC_PENDING);
+    assert(ble_gap_conn_master_state == BLE_GAP_CONN_M_STATE_GEN_DISC_PENDING);
 
-    ble_gap_conn_master_state = BLE_GAP_CONN_STATE_M_GEN_DISC_PARAMS;
+    ble_gap_conn_master_state = BLE_GAP_CONN_M_STATE_GEN_DISC_PARAMS;
     ble_hci_ack_set_callback(ble_gap_conn_gen_disc_ack_params, NULL);
 
     rc = host_hci_cmd_le_set_scan_params(BLE_HCI_SCAN_TYPE_ACTIVE,
@@ -754,13 +839,11 @@ ble_gap_conn_gen_disc(uint32_t duration_ms)
         return BLE_HS_EALREADY;
     }
 
-    ble_gap_conn_master_type = BLE_GAP_CONN_M_TYPE_GEN_DISC;
-
     if (duration_ms == 0) {
         duration_ms = BLE_GAP_GEN_DISC_SCAN_MIN;
     }
 
-    ble_gap_conn_master_state = BLE_GAP_CONN_STATE_M_GEN_DISC_PENDING;
+    ble_gap_conn_master_state = BLE_GAP_CONN_M_STATE_GEN_DISC_PENDING;
     memset(ble_gap_conn_master_addr, 0, BLE_DEV_ADDR_LEN);
 
     rc = ble_hci_sched_enqueue(ble_gap_conn_gen_disc_tx_params, NULL);
@@ -787,14 +870,14 @@ static void
 ble_gap_conn_direct_connect_ack(struct ble_hci_ack *ack, void *arg)
 {
     assert(ble_gap_conn_master_state ==
-           BLE_GAP_CONN_STATE_M_DIRECT_UNACKED);
+           BLE_GAP_CONN_M_STATE_DIRECT_UNACKED);
 
     if (ack->bha_status != 0) {
         ble_gap_conn_master_failed(ack->bha_status);
         return;
     }
 
-    ble_gap_conn_master_state = BLE_GAP_CONN_STATE_M_DIRECT_ACKED;
+    ble_gap_conn_master_state = BLE_GAP_CONN_M_STATE_DIRECT_ACKED;
 }
 
 static int
@@ -803,7 +886,7 @@ ble_gap_conn_direct_connect_tx(void *arg)
     struct hci_create_conn hcc;
     int rc;
 
-    assert(ble_gap_conn_master_state == BLE_GAP_CONN_STATE_M_DIRECT_PENDING);
+    assert(ble_gap_conn_master_state == BLE_GAP_CONN_M_STATE_DIRECT_PENDING);
 
     hcc.scan_itvl = 0x0010;
     hcc.scan_window = 0x0010;
@@ -818,7 +901,7 @@ ble_gap_conn_direct_connect_tx(void *arg)
     hcc.min_ce_len = 0x0010; // XXX
     hcc.max_ce_len = 0x0300; // XXX
 
-    ble_gap_conn_master_state = BLE_GAP_CONN_STATE_M_DIRECT_UNACKED;
+    ble_gap_conn_master_state = BLE_GAP_CONN_M_STATE_DIRECT_UNACKED;
     ble_hci_ack_set_callback(ble_gap_conn_direct_connect_ack, NULL);
 
     rc = host_hci_cmd_le_create_connection(&hcc);
@@ -853,8 +936,7 @@ ble_gap_conn_direct_connect(int addr_type, uint8_t *addr)
         return BLE_HS_EALREADY;
     }
 
-    ble_gap_conn_master_type = BLE_GAP_CONN_M_TYPE_DIR_CONN;
-    ble_gap_conn_master_state = BLE_GAP_CONN_STATE_M_DIRECT_PENDING;
+    ble_gap_conn_master_state = BLE_GAP_CONN_M_STATE_DIRECT_PENDING;
     memcpy(ble_gap_conn_master_addr, addr, BLE_DEV_ADDR_LEN);
 
     rc = ble_hci_sched_enqueue(ble_gap_conn_direct_connect_tx, NULL);
@@ -925,7 +1007,7 @@ ble_gap_conn_terminate(uint16_t handle)
 static void
 ble_gap_conn_init_slave_params(void)
 {
-    ble_gap_conn_adv_params[BLE_GAP_CONN_S_TYPE_NON_DISC] =
+    ble_gap_conn_adv_params[BLE_GAP_CONN_S_MODE_NON_DISC] =
         (struct hci_adv_params) {
 
         .adv_itvl_min = BLE_GAP_ADV_FAST_INTERVAL1_MIN,
@@ -937,7 +1019,7 @@ ble_gap_conn_init_slave_params(void)
         .adv_filter_policy = BLE_HCI_ADV_FILT_DEF,
     };
 
-    ble_gap_conn_adv_params[BLE_GAP_CONN_S_TYPE_LTD_DISC] =
+    ble_gap_conn_adv_params[BLE_GAP_CONN_S_MODE_LTD_DISC] =
         (struct hci_adv_params) {
 
         .adv_itvl_min = BLE_GAP_ADV_FAST_INTERVAL1_MIN,
@@ -949,7 +1031,7 @@ ble_gap_conn_init_slave_params(void)
         .adv_filter_policy = BLE_HCI_ADV_FILT_DEF,
     };
 
-    ble_gap_conn_adv_params[BLE_GAP_CONN_S_TYPE_GEN_DISC] =
+    ble_gap_conn_adv_params[BLE_GAP_CONN_S_MODE_GEN_DISC] =
         (struct hci_adv_params) {
 
         .adv_itvl_min = BLE_GAP_ADV_FAST_INTERVAL1_MIN,
@@ -961,7 +1043,7 @@ ble_gap_conn_init_slave_params(void)
         .adv_filter_policy = BLE_HCI_ADV_FILT_DEF,
     };
 
-    ble_gap_conn_adv_params[BLE_GAP_CONN_S_TYPE_NON_CONN] =
+    ble_gap_conn_adv_params[BLE_GAP_CONN_S_MODE_NON_CONN] =
         (struct hci_adv_params) {
 
         .adv_itvl_min = BLE_GAP_ADV_FAST_INTERVAL1_MIN,
@@ -973,7 +1055,7 @@ ble_gap_conn_init_slave_params(void)
         .adv_filter_policy = BLE_HCI_ADV_FILT_DEF,
     };
 
-    ble_gap_conn_adv_params[BLE_GAP_CONN_S_TYPE_DIR_CONN] =
+    ble_gap_conn_adv_params[BLE_GAP_CONN_S_MODE_DIR_CONN] =
         (struct hci_adv_params) {
 
         .adv_itvl_min = BLE_GAP_ADV_FAST_INTERVAL1_MIN,
@@ -985,7 +1067,7 @@ ble_gap_conn_init_slave_params(void)
         .adv_filter_policy = BLE_HCI_ADV_FILT_DEF,
     };
 
-    ble_gap_conn_adv_params[BLE_GAP_CONN_S_TYPE_UND_CONN] =
+    ble_gap_conn_adv_params[BLE_GAP_CONN_S_MODE_UND_CONN] =
         (struct hci_adv_params) {
 
         .adv_itvl_min = BLE_GAP_ADV_FAST_INTERVAL1_MIN,
@@ -1002,8 +1084,7 @@ int
 ble_gap_conn_init(void)
 {
     ble_gap_conn_cb = NULL;
-    ble_gap_conn_master_type = BLE_GAP_CONN_M_TYPE_NONE;
-    ble_gap_conn_slave_type = BLE_GAP_CONN_S_TYPE_NONE;
+    ble_gap_conn_slave_mode = BLE_GAP_CONN_S_MODE_NONE;
     ble_gap_conn_master_state = BLE_GAP_CONN_STATE_IDLE;
     ble_gap_conn_slave_state = BLE_GAP_CONN_STATE_IDLE;
     memset(ble_gap_conn_master_addr, 0, sizeof ble_gap_conn_master_addr);

http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/blob/fd3c7220/net/nimble/host/src/host_hci_cmd.c
----------------------------------------------------------------------
diff --git a/net/nimble/host/src/host_hci_cmd.c b/net/nimble/host/src/host_hci_cmd.c
index 4b7fb94..ab98071 100644
--- a/net/nimble/host/src/host_hci_cmd.c
+++ b/net/nimble/host/src/host_hci_cmd.c
@@ -495,3 +495,18 @@ host_hci_cmd_reset(void)
                            NULL);
     return rc;
 }
+
+/**
+ * Read the transmit power level used for LE advertising channel packets.
+ *
+ * @return int
+ */
+int
+host_hci_cmd_read_adv_pwr(void)
+{
+    int rc;
+
+    rc = host_hci_cmd_send(BLE_HCI_OGF_LE, BLE_HCI_OCF_LE_RD_ADV_CHAN_TXPWR, 0,
+                           NULL);
+    return rc;
+}

http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/blob/fd3c7220/net/nimble/host/src/test/ble_hs_conn_test.c
----------------------------------------------------------------------
diff --git a/net/nimble/host/src/test/ble_hs_conn_test.c b/net/nimble/host/src/test/ble_hs_conn_test.c
index 6c5d9a6..e9083e5 100644
--- a/net/nimble/host/src/test/ble_hs_conn_test.c
+++ b/net/nimble/host/src/test/ble_hs_conn_test.c
@@ -144,17 +144,7 @@ TEST_CASE(ble_hs_conn_test_direct_connectable_success)
     TEST_ASSERT(!ble_gap_conn_master_in_progress());
     TEST_ASSERT(ble_gap_conn_slave_in_progress());
 
-    /* Receive set-adv-params ack. */
-    ble_hs_test_util_rx_le_ack(BLE_HCI_OCF_LE_SET_ADV_PARAMS, BLE_ERR_SUCCESS);
-    TEST_ASSERT(!ble_gap_conn_master_in_progress());
-    TEST_ASSERT(ble_gap_conn_slave_in_progress());
-
-    ble_hci_sched_wakeup();
-
-    /* Receive set-adv-enable ack. */
-    ble_hs_test_util_rx_le_ack(BLE_HCI_OCF_LE_SET_ADV_ENABLE, BLE_ERR_SUCCESS);
-    TEST_ASSERT(!ble_gap_conn_master_in_progress());
-    TEST_ASSERT(ble_gap_conn_slave_in_progress());
+    ble_hs_test_util_rx_adv_acks();
 
     /* Receive successful connection complete event. */
     memset(&evt, 0, sizeof evt);
@@ -209,11 +199,8 @@ TEST_CASE(ble_hs_conn_test_direct_connectable_hci_errors)
     TEST_ASSERT(rc != 0);
     TEST_ASSERT(ble_gap_conn_slave_in_progress());
 
-    /* Receive success command status events. */
-    ble_hs_test_util_rx_le_ack(BLE_HCI_OCF_LE_SET_ADV_PARAMS, BLE_ERR_SUCCESS);
-    ble_hci_sched_wakeup();
-    ble_hs_test_util_rx_le_ack(BLE_HCI_OCF_LE_SET_ADV_ENABLE, BLE_ERR_SUCCESS);
-    TEST_ASSERT(ble_gap_conn_slave_in_progress());
+    /* Receive necessary ack events. */
+    ble_hs_test_util_rx_adv_acks();
 
     /* Receive failure connection complete event. */
     evt.status = BLE_ERR_UNSPECIFIED;

http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/blob/fd3c7220/net/nimble/host/src/test/ble_hs_test_util.c
----------------------------------------------------------------------
diff --git a/net/nimble/host/src/test/ble_hs_test_util.c b/net/nimble/host/src/test/ble_hs_test_util.c
index 7887dcb..d623f76 100644
--- a/net/nimble/host/src/test/ble_hs_test_util.c
+++ b/net/nimble/host/src/test/ble_hs_test_util.c
@@ -87,6 +87,22 @@ ble_hs_test_util_create_conn(uint16_t handle, uint8_t *addr)
 }
 
 void
+ble_hs_test_util_rx_ack_param(uint16_t opcode, uint8_t status, void *param,
+                              int param_len)
+{
+    uint8_t buf[256];
+    int rc;
+
+    ble_hs_test_util_build_cmd_complete(buf, sizeof buf, param_len + 1, 1,
+                                        opcode);
+    buf[BLE_HCI_EVENT_CMD_COMPLETE_HDR_LEN] = status;
+    memcpy(buf + BLE_HCI_EVENT_CMD_COMPLETE_HDR_LEN + 1, param, param_len);
+
+    rc = host_hci_event_rx(buf);
+    TEST_ASSERT(rc == 0);
+}
+
+void
 ble_hs_test_util_rx_ack(uint16_t opcode, uint8_t status)
 {
     uint8_t buf[BLE_HCI_EVENT_CMD_STATUS_LEN];
@@ -118,6 +134,14 @@ ble_hs_test_util_rx_hci_buf_size_ack(uint16_t buf_size)
 }
 
 void
+ble_hs_test_util_rx_le_ack_param(uint16_t ocf, uint8_t status, void *param,
+                                 int param_len)
+{
+    ble_hs_test_util_rx_ack_param((BLE_HCI_OGF_LE << 10) | ocf, status, param,
+                                  param_len);
+}
+
+void
 ble_hs_test_util_rx_le_ack(uint16_t ocf, uint8_t status)
 {
     ble_hs_test_util_rx_ack((BLE_HCI_OGF_LE << 10) | ocf, status);
@@ -231,6 +255,40 @@ ble_hs_test_util_rx_num_completed_pkts_event(
 }
 
 void
+ble_hs_test_util_rx_adv_acks(void)
+{
+    /* Receive set-adv-params ack. */
+    ble_hs_test_util_rx_le_ack(BLE_HCI_OCF_LE_SET_ADV_PARAMS, BLE_ERR_SUCCESS);
+    TEST_ASSERT(ble_gap_conn_slave_in_progress());
+
+    ble_hci_sched_wakeup();
+
+    /* Receive read-power-level ack. */
+    ble_hs_test_util_rx_le_ack_param(BLE_HCI_OCF_LE_RD_ADV_CHAN_TXPWR,
+                                     BLE_ERR_SUCCESS, (uint8_t[]){0}, 1);
+    TEST_ASSERT(ble_gap_conn_slave_in_progress());
+
+    ble_hci_sched_wakeup();
+
+    /* Receive set-adv-data ack. */
+    ble_hs_test_util_rx_le_ack(BLE_HCI_OCF_LE_SET_ADV_DATA, BLE_ERR_SUCCESS);
+    TEST_ASSERT(ble_gap_conn_slave_in_progress());
+
+    ble_hci_sched_wakeup();
+
+    /* Receive set-scan-response-data ack. */
+    ble_hs_test_util_rx_le_ack(BLE_HCI_OCF_LE_SET_SCAN_RSP_DATA,
+                               BLE_ERR_SUCCESS);
+    TEST_ASSERT(ble_gap_conn_slave_in_progress());
+
+    ble_hci_sched_wakeup();
+
+    /* Receive set-adv-enable ack. */
+    ble_hs_test_util_rx_le_ack(BLE_HCI_OCF_LE_SET_ADV_ENABLE, BLE_ERR_SUCCESS);
+    TEST_ASSERT(ble_gap_conn_slave_in_progress());
+}
+
+void
 ble_hs_test_util_init(void)
 {
     int rc;

http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/blob/fd3c7220/net/nimble/host/src/test/ble_hs_test_util.h
----------------------------------------------------------------------
diff --git a/net/nimble/host/src/test/ble_hs_test_util.h b/net/nimble/host/src/test/ble_hs_test_util.h
index feae544..e8e5999 100644
--- a/net/nimble/host/src/test/ble_hs_test_util.h
+++ b/net/nimble/host/src/test/ble_hs_test_util.h
@@ -37,7 +37,11 @@ void ble_hs_test_util_build_cmd_status(uint8_t *dst, int len,
 struct ble_hs_conn *ble_hs_test_util_create_conn(uint16_t handle,
                                                  uint8_t *addr);
 void ble_hs_test_util_rx_ack(uint16_t opcode, uint8_t status);
+void ble_hs_test_util_rx_ack_param(uint16_t opcode, uint8_t status,
+                                   void *param, int param_len);
 void ble_hs_test_util_rx_le_ack(uint16_t ocf, uint8_t status);
+void ble_hs_test_util_rx_le_ack_param(uint16_t ocf, uint8_t status, void *param,
+                                      int param_len);
 int ble_hs_test_util_l2cap_rx_payload_flat(struct ble_hs_conn *conn,
                                            struct ble_l2cap_chan *chan,
                                            const void *data, int len);
@@ -47,6 +51,7 @@ void ble_hs_test_util_rx_att_err_rsp(struct ble_hs_conn *conn, uint8_t req_op,
 void ble_hs_test_util_rx_startup_acks(void);
 void ble_hs_test_util_rx_num_completed_pkts_event(
     struct ble_hs_test_util_num_completed_pkts_entry *entries);
+void ble_hs_test_util_rx_adv_acks(void);
 void ble_hs_test_util_init(void);
 
 #endif

http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/blob/fd3c7220/net/nimble/include/nimble/hci_common.h
----------------------------------------------------------------------
diff --git a/net/nimble/include/nimble/hci_common.h b/net/nimble/include/nimble/hci_common.h
index f456f9c..7510ade 100644
--- a/net/nimble/include/nimble/hci_common.h
+++ b/net/nimble/include/nimble/hci_common.h
@@ -112,29 +112,17 @@
 #define BLE_HCI_OCF_LE_RD_MAX_DATA_LEN      (0x002F)
 
 /* Command Specific Definitions */
-/*--- BLE_HCI_OCF_LE_SET_EVENT_MASK OCF 0x0001 ---*/
+/* --- LE set event mask (OCF 0x0001) --- */
 #define BLE_HCI_SET_LE_EVENT_MASK_LEN   (8)
 
-/*--- BLE_HCI_OCF_LE_RD_BUF_SIZE  OCF 0x0002  ---*/
+/* --- LE read buffer size (OCF 0x0002) --- */
 #define BLE_HCI_RD_BUF_SIZE_LEN         (0)
 #define BLE_HCI_RD_BUF_SIZE_RSPLEN      (3)
 
-/* Read local supported features*/
+/* --- LE read local supported features (OCF 0x0003) --- */
 #define BLE_HCI_RD_LOC_SUPP_FEAT_RSPLEN (8)
 
-/* Set scan response data */
-#define BLE_HCI_MAX_SCAN_RSP_DATA_LEN   (31)
-
-/* Set advertising data */
-#define BLE_HCI_MAX_ADV_DATA_LEN        (31)
-
-/* Set advertising enable */
-#define BLE_HCI_SET_ADV_ENABLE_LEN      (1)
-
-/* Set scan enable */
-#define BLE_HCI_SET_SCAN_ENABLE_LEN     (2)
-
-/* Set advertising parameters */
+/* --- LE set advertising parameters (OCF 0x0006) */
 #define BLE_HCI_SET_ADV_PARAM_LEN       (15)
 
 /* Advertising types */
@@ -157,6 +145,23 @@
 #define BLE_HCI_ADV_PEER_ADDR_RANDOM        (1)
 #define BLE_HCI_ADV_PEER_ADDR_MAX           (1)
 
+/* --- LE advertising channel tx power (OCF 0x0007) */
+#define BLE_HCI_ADV_CHAN_TXPWR_ACK_PARAM_LEN   (2)  /* Includes status byte. */
+#define BLE_HCI_ADV_CHAN_TXPWR_MIN             (-20)
+#define BLE_HCI_ADV_CHAN_TXPWR_MAX             (10)
+
+/* --- LE set advertising data (OCF 0x0008) */
+#define BLE_HCI_MAX_ADV_DATA_LEN        (31)
+
+/* --- LE set scan response data (OCF 0x0009) */
+#define BLE_HCI_MAX_SCAN_RSP_DATA_LEN   (31)
+
+/* --- LE set advertising enable (OCF 0x000a) */
+#define BLE_HCI_SET_ADV_ENABLE_LEN      (1)
+
+/* --- LE set scan enable (OCF 0x000c) */
+#define BLE_HCI_SET_SCAN_ENABLE_LEN     (2)
+
 /* Connect peer address type */
 #define BLE_HCI_CONN_PEER_ADDR_PUBLIC        (0)
 #define BLE_HCI_CONN_PEER_ADDR_RANDOM        (1)


[2/2] incubator-mynewt-larva git commit: Remove some obsolete defines.

Posted by cc...@apache.org.
Remove some obsolete defines.


Project: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/commit/76f96f88
Tree: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/tree/76f96f88
Diff: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/diff/76f96f88

Branch: refs/heads/master
Commit: 76f96f882d7a90a722233165de4938ddf8ab9775
Parents: fd3c722
Author: Christopher Collins <cc...@gmail.com>
Authored: Fri Dec 18 18:09:35 2015 -0800
Committer: Christopher Collins <cc...@gmail.com>
Committed: Fri Dec 18 18:10:27 2015 -0800

----------------------------------------------------------------------
 project/hostctlrtest/src/main.c | 16 ----------------
 1 file changed, 16 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-mynewt-larva/blob/76f96f88/project/hostctlrtest/src/main.c
----------------------------------------------------------------------
diff --git a/project/hostctlrtest/src/main.c b/project/hostctlrtest/src/main.c
index a455214..3c75de0 100755
--- a/project/hostctlrtest/src/main.c
+++ b/project/hostctlrtest/src/main.c
@@ -72,22 +72,6 @@ struct os_mbuf_pool g_mbuf_pool;
 struct os_mempool g_mbuf_mempool;
 os_membuf_t g_mbuf_buffer[MBUF_MEMPOOL_SIZE];
 
-/* Some application configurations */
-#define HOSTCTLRTEST_CFG_FILT_DUP_ADV        (0)
-#define HOSTCTLRTEST_CFG_ADV_ITVL            (500000 / BLE_HCI_ADV_ITVL)
-#define HOSTCTLRTEST_CFG_ADV_TYPE            BLE_HCI_ADV_TYPE_ADV_IND
-#define HOSTCTLRTEST_CFG_ADV_FILT_POLICY     (BLE_HCI_ADV_FILT_NONE)
-#define HOSTCTLRTEST_CFG_SCAN_ITVL           (700000 / BLE_HCI_SCAN_ITVL)
-#define HOSTCTLRTEST_CFG_SCAN_WINDOW         (650000 / BLE_HCI_SCAN_ITVL)
-#define HOSTCTLRTEST_CFG_SCAN_TYPE           (BLE_HCI_SCAN_TYPE_ACTIVE)
-#define HOSTCTLRTEST_CFG_SCAN_FILT_POLICY    (BLE_HCI_SCAN_FILT_NO_WL)
-#define HOSTCTLRTEST_CFG_CONN_ITVL           (1000)  /* 1250 msecs */           
-#define HOSTCTLRTEST_CFG_SLAVE_LATENCY       (0)
-#define HOSTCTLRTEST_CFG_INIT_FILTER_POLICY  (BLE_HCI_CONN_FILT_NO_WL)
-#define HOSTCTLRTEST_CFG_CONN_SPVN_TMO       (1000)  /* 10 seconds */
-#define HOSTCTLRTEST_CFG_MIN_CE_LEN          (1000)    
-#define HOSTCTLRTEST_CFG_MAX_CE_LEN          (HOSTCTLRTEST_CFG_CONN_ITVL * 2)
-
 /* HOSTCTLRTEST variables */
 #define HOSTCTLRTEST_STACK_SIZE              (256)
 #define HOSTCTLRTEST_TASK_PRIO               (HOST_TASK_PRIO + 1)