S310 v2 Example Code does not follow Migration Guide advice

[awneil]

The s310_nrf51422 migration document states:

Due to an issue present in this release, the application is required to set the device address to the factory default right after calling sd_ble_enable(), this can be achieved with the following lines: sd_ble_enable(p_ble_enable_params); sd_ble_gap_address_get(&addr); sd_ble_gap_address_set(BLE_GAP_ADDR_CYCLE_MODE_NONE, &addr);

But the BLE-ANT HRM Relay example (SDK 7.2.0) does not do this! It calls sd_ble_enable(), but never calls sd_ble_gap_address_get() nor sd_ble_gap_address_set()

Is the example wrong? Or is the migration document wrong?

[Hung Bui]

@Awneil: Seems that it's a bug with our migration document. The issue with the address when pairing was fixed on S310 v2.0 so the workaround is not needed anymore. You can find this information in the S310 v2.0 release note at bugfixes section, DRGN-4521 was fixed.

I will report this internally for a fix.

[awneil]

OK - thanks.

It would have been more helpful if the Migration Document had given that reference (DRGN-4521) - instead of just saying, "an issue present in this release". That way, we would have been able to see that the issue was fixed - otherwise, we can't be sure which "issue" the Document is talking about!