Fail to include more than one service data in advertising

[diode]

It seems to be a straightforward BLE app but I cannot make it work.

I start fresh from the “ble_app_template” in nRF51 SDK 8.0.0, and try to add two service data (battery level and temperature) in advertising packet. I make those following changes

1. Change the clock source to the internal RC oscillator
2. Add two custom functions “battery_level_get()” and “temperature_data_get()”. I can confirm those two functions work in my other BLE beacon app.
3. Modify the advertising packet in “advertising_init()” to include battery level and temperature information.

However, it is not quite working. If I just add one service data (battery level), things are fine. If I add two service data (battery level and temperature), the code can compile but I cannot see the BLE device advertising from my BLE client (Nordic Master Control Panel).

The main.c is attached here main.c. What do I miss?

[RK]

Are you in the error handler? I assume somewhere you call ble_advdata_set(), what was the result of that call?

Usually "cannot see the BLE device advertising" means "it crashed already". Put a breakpoint in your error handler or make it turn all the LEDs on or something so you can easily tell when this happens.

[Chris]

Your advertising packet might be too big (limit is 31 bytes).

[RK]

That was my point in asking him to check what the ble_advdata_set() returned - the advertising packet is definitely too long by 3 or 4 bytes, but I thought it might be helpful if he learned what the error handler is and worked out how to do some simple debugging.

[diode]

I took a deeper look at the BLE advertising packet format, and found my previous code created a ADV data beyond limit. Problem solved. Thanks!