About macro SER_MAX_CONNECTIONS

[Lezard]

Hi,

We use nrf51 SDK v9. I found there are two SER_MAX_CONNECTIONS defined both in app_ble_gap_sec_keys.h and ser_config.h. Their values are different. One is referred in app_ble_gap_sec_keys.c and another in referred in app_ble_user_mem.c. From the code, it seems the two SER_MAX_CONNECTIONS refer to the same thing. So why they are redefined and assigned different values.

[ovrebekk]

Hi

This is an error in the SDK, and I have reported it to the software team.

In later SDK's they are both defined to 8 so it should work fine, but the define should clearly only be defined once, and we will aim to get this fixed in a future SDK release.

Thanks for reporting this :)

Best regards
Torbjørn

[Lezard]

Thank you for reply. For SDK v9, I think it is an old version, how can I modify the code to fix this error temporarily? Which one can I delete and what value to set?

[ovrebekk]

Hi

It's safe to keep both as long as you set them to the same value, but if you want to remove one of them it would make the most sense to keep the one in ser_config.h.

What value to set depends on which SoftDevice you are using, and how many central connections you need.
If you use the S110 you have to set it to 1.
The S120 supports up to 8, while the S130 supports up to 3, but if you don't need the additional connections I would suggest setting it to 1 to save the RAM.

Best regards