tools/mgmt-tester: Update to support multiple extended adv instances
authorTedd Ho-Jeong An <tedd.an@intel.com>
Wed, 23 Jun 2021 04:49:44 +0000 (21:49 -0700)
committerAyush Garg <ayush.garg@samsung.com>
Fri, 11 Mar 2022 13:38:35 +0000 (19:08 +0530)
This patch updates to align with the multiple extended advertising
support.

Signed-off-by: Anuj Jain <anuj01.jain@samsung.com>
Signed-off-by: Ayush Garg <ayush.garg@samsung.com>
tools/mgmt-tester.c

index 8cae376..d2ded57 100755 (executable)
@@ -5871,9 +5871,10 @@ static void setup_bthost(void)
 
        if (data->hciemu_type == HCIEMU_TYPE_LE ||
                test->client_enable_adv) {
-               if (data->hciemu_type >= HCIEMU_TYPE_BREDRLE50)
+               if (data->hciemu_type >= HCIEMU_TYPE_BREDRLE50) {
+                       bthost_set_ext_adv_params(bthost);
                        bthost_set_ext_adv_enable(bthost, 0x01);
-               else
+               else
                        bthost_set_adv_enable(bthost, 0x01);
        } else
                bthost_write_scan_enable(bthost, 0x03);
@@ -7554,7 +7555,7 @@ static const uint8_t read_adv_features_rsp_3[] =  {
        0xff, 0xff, 0x01, 0x00, /* supported flags */
        0x1f,                   /* max_adv_data_len */
        0x1f,                   /* max_scan_rsp_len */
-       0x01,                   /* max_instances */
+       0x03,                   /* max_instances */
        0x00,                   /* num_instances */
 };
 
@@ -8077,9 +8078,6 @@ static const struct generic_data add_ext_advertising_timeout_expired = {
        .expect_alt_ev = MGMT_EV_ADVERTISING_REMOVED,
        .expect_alt_ev_param = advertising_instance1_param,
        .expect_alt_ev_len = sizeof(advertising_instance1_param),
-       .expect_hci_command = BT_HCI_CMD_LE_SET_EXT_ADV_ENABLE,
-       .expect_hci_param = set_ext_adv_disable_param,
-       .expect_hci_len = sizeof(set_ext_adv_disable_param),
 };
 
 static const struct generic_data remove_ext_advertising_fail_1 = {
@@ -8120,7 +8118,7 @@ static const struct generic_data remove_ext_advertising_success_2 = {
 };
 
 static const uint8_t set_ext_adv_data_test2[] = {
-       0x01,                           /* handle */
+       0x02,                           /* handle */
        0x03,                           /* complete data */
        0x01,                           /* controller should not fragment */
        0x07,                           /* adv data len */
@@ -8129,13 +8127,10 @@ static const uint8_t set_ext_adv_data_test2[] = {
        0x74, 0x65, 0x73, 0x74, 0x32,   /* "test2" */
 };
 
-static const struct generic_data multi_ext_advertising_switch = {
+static const struct generic_data multi_ext_advertising = {
        .expect_alt_ev = MGMT_EV_ADVERTISING_REMOVED,
        .expect_alt_ev_param = advertising_instance1_param,
        .expect_alt_ev_len = sizeof(advertising_instance1_param),
-       .expect_hci_command = BT_HCI_CMD_LE_SET_EXT_ADV_DATA,
-       .expect_hci_param = set_ext_adv_data_test2,
-       .expect_hci_len = sizeof(set_ext_adv_data_test2),
 };
 
 static const struct generic_data multi_ext_advertising_add_second = {
@@ -9654,6 +9649,7 @@ static void trigger_device_found(void *user_data)
 
                bthost_set_adv_enable(bthost, 0x01);
        } else if (data->hciemu_type >= HCIEMU_TYPE_BREDRLE50) {
+               bthost_set_ext_adv_params(bthost);
                if (test->set_adv)
                        bthost_set_ext_adv_data(bthost, test->adv_data,
                                                        test->adv_data_len);
@@ -9818,21 +9814,16 @@ static void le_connected_event(uint16_t index, uint16_t length,
 
        tester_print("Device connected");
 
-       test_add_condition(data);
-
-       if (data->hciemu_type >= HCIEMU_TYPE_BREDRLE50)
-               hciemu_add_hook(data->hciemu, HCIEMU_HOOK_POST_CMD,
-                                       BT_HCI_CMD_LE_SET_EXT_ADV_ENABLE,
-                                       test_adv_enable_hook, data);
-       else
+       if (data->hciemu_type < HCIEMU_TYPE_BREDRLE50) {
+               test_add_condition(data);
                hciemu_add_hook(data->hciemu, HCIEMU_HOOK_POST_CMD,
                                        BT_HCI_CMD_LE_SET_ADV_ENABLE,
                                        test_adv_enable_hook, data);
+       }
 
        /* Make sure we get not disconnected during the testaces */
        mgmt_register(data->mgmt_alt, MGMT_EV_DEVICE_DISCONNECTED,
                                data->mgmt_index, disconnected_event,
-
                                NULL, NULL);
 
        test_condition_complete(data);
@@ -11067,7 +11058,6 @@ int main(int argc, char *argv[])
                                         &add_advertising_name_data_appear,
                                         setup_command_generic,
                                         test_command_generic);
-
        test_le_full("Adv. connectable & connected (slave) - Success",
                                        &conn_slave_adv_conneactable_test,
                                        setup_advertise_while_connected,
@@ -11406,8 +11396,8 @@ int main(int argc, char *argv[])
        /* When advertising two instances, the instances should be
         * advertised in a round-robin fashion.
         */
-       test_bredrle50("Multi Ext Advertising - Success 1 (Instance Switch)",
-                                       &multi_ext_advertising_switch,
+       test_bredrle50("Multi Ext Advertising - Success 1",
+                                       &multi_ext_advertising,
                                        setup_multi_adv,
                                        test_command_generic);
 
@@ -11430,7 +11420,7 @@ int main(int argc, char *argv[])
                                        NULL, test_command_generic);
 
        test_bredrle50("Set PHY 1m 2m coded Succcess", &set_phy_all_success,
-                                        NULL, test_command_generic);
+                                       NULL, test_command_generic);
 
        test_bredrle50("Set PHY 2m tx success", &set_phy_2m_tx_success,
                                        NULL, test_command_generic);