AN36 modifications

[michurin]

I want to add to lbs service battery service(bas) as well. One of the changes is in the advertising_init() function:

static void advertising_init(void)
{
    uint32_t      err_code;
    ble_advdata_t advdata;
    uint8_t       flags = BLE_GAP_ADV_FLAGS_LE_ONLY_LIMITED_DISC_MODE;
    
    ble_uuid_t adv_uuids[] = 
    {
        {BLE_UUID_BATTERY_SERVICE, m_lbs.uuid_type},
        {LBS_UUID_SERVICE, m_lbs.uuid_type}
    };

    // Build and set advertising data
    memset(&advdata, 0, sizeof(advdata));
    
    advdata.name_type               = BLE_ADVDATA_FULL_NAME;
    advdata.include_appearance      = true;
    advdata.flags.size              = sizeof(flags);
    advdata.flags.p_data            = &flags;
    advdata.uuids_complete.uuid_cnt = sizeof(adv_uuids) / sizeof(adv_uuids[0]);
    advdata.uuids_complete.p_uuids  = adv_uuids;
    
    err_code = ble_advdata_set(&advdata, NULL);
    APP_ERROR_CHECK(err_code);
}

As a result I have NRF_ERROR_DATA_SIZE I did use blood pressure service as an example. What did I miss?

Thank you.

[michurin]

Dear Ole,

Thank you very much for your help, support and patience. Could you please add more details and extend this paragraph:

Based on my understanding of the spec however, I don't think it would normally be legal to have uuids_complete split between advertisement data and scan response. The reason is that each packet should be valid in itself, but if you have only some services in a field marked as uuid_complete, your field doesn't actually contain what the type implies. If you want to share the list between the packets, I'd therefore recommend you to use more_available.

What confuses me is if I do have a proprietary device with some unique service(s) but also want to have same defined services like the battery, temperature, proximity. The list can become lengthy and may not fit, so how to do it the right way?

Thank you.

[Ole Morten]

Each field in the advertisement and scan response packets have a type, and when you use the complete UUID list as type, I'd say that you are required to actually have all services in this exact field. If you have one field in the advertisement packet which have the complete type, and one field in the scan response with the complete type, neither of them are actually complete, and I'd hence say that you don't fulfill the specification.

This is exactly why you have the more available type, and are also not required to have all services in the advertisement packet. I'd normally recommend you to have the primary service of the device in the advertisement, and leave out all others. In some cases you may have multiple equally important services, and if so, you may need/want to split them between the advertisement and scan response, but then by using the more available type for both fields.