This post is older than 2 years and might not be relevant anymore
More Info: Consider searching for newer posts
This discussion has been locked.
You can no longer post new replies to this discussion. If you have a question you can start a new discussion

BLE_DB_DISCOVERY_ERROR When Modifying Battery Service

I'm trying to adapt the BLE_BAS_C library to perform discovery and read/write operations for a custom service. However, when I change the UUID that I register with ble_db_discovery_evt_register(), I see the services start to be discovered with sd_ble_gattc_primary_services_discover(), but when the db_discovery_evt_handler() in ble_bas_c is entered instead of the expected BLE_DB_DISCOVERY_COMPLETE event type I see BLE_DB_DISCOVERY_ERROR.

I'm wondering if this has to do with the fact that it's a vendor specific service rather than a standard BLE one. I've attached the changes I've made- is there something I'm missing, or do I need to do something else with regards to the UUID base.

CENTRAL

static void db_discovery_evt_handler(ble_db_discovery_evt_t * p_evt)
{
    printf("[BAS_C] Discovery event handler \r\n");
    
    // Check if the Battery Service was discovered.
    if (p_evt->evt_type == BLE_DB_DISCOVERY_COMPLETE){
        printf("[BAS_C] UUID 0x%x \r\n", p_evt->params.discovered_db.srv_uuid.uuid);
        
        if (p_evt->params.discovered_db.srv_uuid.uuid == DEVICE_STATUS_SERVICE_UUID){
        
            if (p_evt->params.discovered_db.srv_uuid.type == BLE_UUID_TYPE_VENDOR_BEGIN){

            printf("[BAS_C] Battery Service discovered at peer.\r\n");

            ble_bas_c_evt_t evt;

            evt.evt_type = BLE_BAS_C_EVT_DISCOVERY_COMPLETE;

            mp_ble_bas_c->evt_handler(mp_ble_bas_c, &evt);
            }
        }
    }
    else
    {
        printf("[BAS_C] Battery Service discovery failure at peer. \r\n");
    }
}

uint32_t ble_bas_c_init(ble_bas_c_t * p_ble_bas_c, ble_bas_c_init_t * p_ble_bas_c_init)
{
    if ((p_ble_bas_c == NULL) || (p_ble_bas_c_init == NULL))
    {
        return NRF_ERROR_NULL;
    }

    ble_uuid_t bas_uuid;

    bas_uuid.type                = BLE_UUID_TYPE_VENDOR_BEGIN;
    bas_uuid.uuid                = DEVICE_STATUS_SERVICE_UUID;

    mp_ble_bas_c                 = p_ble_bas_c;

    mp_ble_bas_c->conn_handle    = BLE_CONN_HANDLE_INVALID;
    mp_ble_bas_c->bl_cccd_handle = BLE_GATT_HANDLE_INVALID;
    mp_ble_bas_c->bl_handle      = BLE_GATT_HANDLE_INVALID;
    mp_ble_bas_c->evt_handler    = p_ble_bas_c_init->evt_handler;

    return ble_db_discovery_evt_register(&bas_uuid, db_discovery_evt_handler);
}
PERIPHERAL

/**@brief Function for initializing services that will be used by the application.
 *
 * @details Initialize the multilink custom service.
 */
static void services_init(void)
{
    uint32_t            err_code;
    ble_uuid_t          uuid;
    ble_gatts_char_md_t char_md;
    ble_gatts_attr_t    attr;
    ble_gatts_attr_md_t attr_md;
    ble_gatts_attr_md_t cccd_md;
    ble_gatts_attr_md_t char_ud_md;
    uint16_t            svc_test;

    /* Add device status service */
    ble_uuid128_t base_uuid = DEVICE_STATUS_BASE_UUID;

    err_code = sd_ble_uuid_vs_add(&base_uuid, &m_base_uuid_type);
    APP_ERROR_CHECK(err_code);

    uuid.type = m_base_uuid_type;
    uuid.uuid = DEVICE_STATUS_SERVICE_UUID;

    err_code = sd_ble_gatts_service_add(BLE_GATTS_SRVC_TYPE_PRIMARY, &uuid, &svc_test);
    APP_ERROR_CHECK(err_code);

    /* Add device status -> Side characteristic */
    static uint8_t char_side_data;
    static uint8_t char_side_ud[] = "Side Characteristic";
    uuid.uuid = DEVICE_STATUS_SIDE_CHAR_UUID;

    memset(&attr, 0, sizeof(ble_gatts_attr_t));
    attr.p_uuid    = &uuid;
    attr.p_attr_md = &attr_md;
    attr.max_len   = 1;
    attr.p_value   = &char_side_data;
    attr.init_len  = sizeof(char_side_data);

    memset(&attr_md, 0, sizeof(ble_gatts_attr_md_t));
    BLE_GAP_CONN_SEC_MODE_SET_OPEN(&attr_md.read_perm);
    BLE_GAP_CONN_SEC_MODE_SET_ENC_NO_MITM(&attr_md.write_perm);
    attr_md.vloc = BLE_GATTS_VLOC_STACK;
    attr_md.vlen = 0;

    memset(&cccd_md, 0, sizeof(ble_gatts_attr_md_t));
    BLE_GAP_CONN_SEC_MODE_SET_OPEN(&cccd_md.read_perm);
    BLE_GAP_CONN_SEC_MODE_SET_ENC_NO_MITM(&cccd_md.write_perm);
    cccd_md.vloc = BLE_GATTS_VLOC_STACK;

    memset(&char_md, 0, sizeof(ble_gatts_char_md_t));
    char_md.p_cccd_md               = &cccd_md;
    char_md.char_props.notify       = 1;
    char_md.char_props.indicate     = 1;
    char_md.char_props.read         = 1;
    char_md.char_props.write        = 1;
    char_md.char_ext_props.wr_aux   = 1;
    char_md.p_user_desc_md          = &char_ud_md;
    char_md.p_char_user_desc        = char_side_ud;
    char_md.char_user_desc_size     = (uint8_t)strlen((char *)char_side_ud);
    char_md.char_user_desc_max_size = (uint8_t)strlen((char *)char_side_ud);

    memset(&char_ud_md, 0, sizeof(ble_gatts_attr_md_t));
    char_ud_md.vloc = BLE_GATTS_VLOC_STACK;
    char_ud_md.vlen = 1;
    BLE_GAP_CONN_SEC_MODE_SET_OPEN(&char_ud_md.read_perm);
    BLE_GAP_CONN_SEC_MODE_SET_OPEN(&char_ud_md.write_perm);

    err_code = sd_ble_gatts_characteristic_add(BLE_GATT_HANDLE_INVALID,
                                               &char_md,
                                               &attr,
                                               &m_side_handle);
    APP_ERROR_CHECK(err_code);
    
    /* Add device status -> MODE characteristic */
    static uint8_t char_mode_data;
    static uint8_t char_mode_ud[] = "Mode Characteristic";    
    uuid.uuid = DEVICE_STATUS_MODE_CHAR_UUID;

    attr.p_uuid    = &uuid;
    attr.p_value   = &char_mode_data;
    attr.init_len  = sizeof(char_mode_data);

    char_md.p_char_user_desc        = char_mode_ud;
    char_md.char_user_desc_size     = (uint8_t)strlen((char *)char_mode_ud);
    char_md.char_user_desc_max_size = (uint8_t)strlen((char *)char_mode_ud);

    err_code = sd_ble_gatts_characteristic_add(BLE_GATT_HANDLE_INVALID,
                                               &char_md,
                                               &attr,
                                               &m_mode_handle);
    APP_ERROR_CHECK(err_code);
    
    /* Add device status -> STATUS characteristic */
    static uint8_t char_status_data;
    static uint8_t char_status_ud[] = "Status Characteristic";    
    uuid.uuid = DEVICE_STATUS_STATUS_CHAR_UUID;

    attr.p_uuid    = &uuid;
    attr.p_value   = &char_status_data;
    attr.init_len  = sizeof(char_status_data);

    char_md.p_char_user_desc        = char_status_ud;
    char_md.char_user_desc_size     = (uint8_t)strlen((char *)char_status_ud);
    char_md.char_user_desc_max_size = (uint8_t)strlen((char *)char_status_ud);

    err_code = sd_ble_gatts_characteristic_add(BLE_GATT_HANDLE_INVALID,
                                               &char_md,
                                               &attr,
                                               &m_stream_status_handle);
    APP_ERROR_CHECK(err_code);
}
Parents Reply Children
No Data
Related