fix(ble): fix race condition for MAC addr device ID in sample

[gminn]

Summary

The device ID when using the BLE MAC address is Unknown after the system
boots because the device has not been given a MAC address yet at the
time the device ID is set (system init). This change fixes that by setting a
static device id for the peripheral_mds sample app.

Bonus - enabled logging at the INFO level on the peripheral_mds
app so we can run commands like mflt get_device_info and get print-outs.

Test Plan

• Tested on a nRF52840 DK with the peripheral_mds sample app

uart:~$ *** Booting nRF Connect SDK v3.5.99-ncs1-4965-g3733e7097909 ***
Starting Bluetooth Memfault example
Bluetooth initialized
Advertising successfully started
uart:~$ mflt get_device_info
[00:00:03.995,544] <inf> mflt: S/N: nrf-ble-testdevice
[00:00:03.995,605] <inf> mflt: SW type: nrf-ble-fw
[00:00:03.995,666] <inf> mflt: SW version: 0.0.1+688c47
[00:00:03.995,727] <inf> mflt: HW version: nrf52840dk_nrf52840

---

Resolves: MCU-433

[gminn]

• fix(ble): fix race condition for MAC addr device ID in sample #24 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gminn and the rest of your teammates on Graphite

[ejohnso49]

Thought about this more, and I think I've sadly come to the conclusion that we just shouldn't try to use this for the BLE MAC case. I think that IMEI and network MAC are fine to keep in. Maybe we add a little blurb to the Kconfig to indicate this is for a network MAC (ie nrf7002).

Also I realized that the device serial isn't needed for reboot events 😑 should have remembered that. There's a Memfault config to include device serial with events but it's disabled by default because nearly all devices use the device serial in the chunks POST url.

[gminn]

Thought about this more, and I think I've sadly come to the conclusion that we just shouldn't try to use this for the BLE MAC case.

To clarify -- you're thinking we remove the example of using the BLE MAC addr for the device id?

[gminn]

To clarify -- you're thinking we remove the example of using the BLE MAC addr for the device id?

@ejohnso49 This is back on my radar and I applied your suggestions -- let me know what you think!

[ejohnso49]

Thanks for tackling this, at least we won't have blank device serial's by default

[gminn]

Merged upstream in nrfconnect@f3f5ff6 !