Setting MAC address fails with multirole BLE (Central+Peripheral)

RE: Setting MAC address fails with multirole BLE (Central+Peripheral)

Amanda Hsieh — Tue, 21 Feb 2023 16:43:00 GMT

Good to hear that. 🙂

-Amanda H.

RE: Setting MAC address fails with multirole BLE (Central+Peripheral)

wichmann — Tue, 21 Feb 2023 13:41:33 GMT

I tested the changes and it seems to resolve the problem. Thanks.

RE: Setting MAC address fails with multirole BLE (Central+Peripheral)

Amanda Hsieh — Mon, 20 Feb 2023 15:58:23 GMT

Hi,

The engineer found the issue. There is now a bugfix coming in https://github.com/nrfconnect/sdk-nrfxlib/pull/919/files.

Can you check that this works for you?

-Amanda H.

RE: Setting MAC address fails with multirole BLE (Central+Peripheral)

wichmann — Fri, 17 Feb 2023 12:22:07 GMT

The workaround does not seem to work. I use the following code to test this (CONFIG_BT_CENTRAL and CONFIG_BT_PERIPHERAL enabled; I'm currently not using `settings_load`):

void main(void) {
   int err;

   err = bt_enable(NULL);
   if (err) {
      LOG_ERR("Bluetooth init failed (err %d)", err);
   }

   k_sleep(K_MSEC(3000));     // ensure that bt_enable is done; alternatively via bt_ready_cb

   static const uint8_t _dbg_mac[6] = {0x01, 0x00, 0x00, 0x25, 0x80, 0x00};
   bt_ctlr_set_public_addr(_dbg_mac);

   bt_addr_le_t addr[CONFIG_BT_ID_MAX];
   size_t count = CONFIG_BT_ID_MAX;
   char str_buffer[30];
   bt_id_get(addr, &count);
   for (size_t i = 0; i < count; i++) {
      bt_addr_le_to_str(&addr[i], str_buffer, sizeof(str_buffer));
      LOG_INF("ID %d: %s", (int) i, str_buffer);
   }

   static const struct bt_data ble_ad_default[] = {};
   static const struct bt_data ble_sd_default[] = {};
   err = bt_le_adv_start(BT_LE_ADV_CONN_NAME, ble_ad_default, ARRAY_SIZE(ble_ad_default), ble_sd_default, ARRAY_SIZE(ble_sd_default));
   if (err) {
      LOG_ERR("Advertising failed to start (err %d)", err);
   }
}

The id returned from `bt_id_get()` as well as the MAC used in advertising are still the static random address.

RE: Setting MAC address fails with multirole BLE (Central+Peripheral)

Amanda Hsieh — Thu, 16 Feb 2023 19:38:46 GMT

Hi,

The team found that there is a bug in the stack if bt_ctlr_set_public_addr() is called before bt_enable() and will have a bug fixed later. The workaround for this right now is setting the address after the bt_enable(), but before settigs_load (if you have it). Please let me know whether it can work or not. Thanks.

-Amanda H.

RE: Setting MAC address fails with multirole BLE (Central+Peripheral)

wichmann — Thu, 16 Feb 2023 05:12:39 GMT

Exactly. "No error" means the public address is set correctly and "error" means the static random address is used instead. I didn't try a complete communication with all these cases, only checked the output of `bt_enable()`.

RE: Setting MAC address fails with multirole BLE (Central+Peripheral)

Amanda Hsieh — Wed, 15 Feb 2023 18:13:26 GMT

Hi,

Are you able to set the public address as expected for those get "No error" in your tests?

Regards,
Amanda H.

RE: Setting MAC address fails with multirole BLE (Central+Peripheral)

wichmann — Wed, 15 Feb 2023 11:43:46 GMT

I did some more testing and it seems that the error is triggered when using CONFIG_BT_PERIPHERAL with the multirole SoftDevice library. To test this I edited `nrfxlib/softdevice_controller/CMakeLists.txt` to manually select the SoftDevice library (changed `zephyr_link_libraries(...)` at the end). When using CONFIG_BT_PERIPHERAL and the peripheral library (`nrfxlib\softdevice_controller\lib\cortex-m4\soft-float\libsoftdevice_controller_peripheral.a`) the address gets set as expected. If I force the use of the multirole library instead (`nrfxlib\softdevice_controller\lib\cortex-m4\soft-float\libsoftdevice_controller_multirole.a`) the error occurs and the random address is used instead. I could not reproduce the error in other constellations (e.g. CONFIG_BT_CENTRAL and multirole lib). I also tried to both use CONFIG_BT_CENTRAL and CONFIG_BT_PERIPHERAL, while using both central and peripheral libraries at the same time, instead of the multirole lib. This seems to work as well (although I did not test full communication).

To sum up my tests:

* CONFIG_BT_CENTRAL + libsoftdevice_controller_central ==> No error

* CONFIG_BT_PERIPHERAL + libsoftdevice_controller_peripheral ==> No error

* CONFIG_BT_CENTRAL + libsoftdevice_controller_multirole ==> No error

* CONFIG_BT_PERIPHERAL + libsoftdevice_controller_multirole ==> Error!

* CONFIG_BT_CENTRAL + CONFIG_BT_PERIPHERAL + libsoftdevice_controller_multirole ==> Error!

* CONFIG_BT_CENTRAL + CONFIG_BT_PERIPHERAL + libsoftdevice_controller_central + libsoftdevice_controller_peripheral ==> No error