Only some of the connected thingy are output

RE: Only some of the connected thingy are output

ovrebekk — Mon, 07 Jan 2019 11:09:26 GMT

So you are using the files from the attached zip, but you can not get notifications to work?

Are some of the characteristics working, or don't you get any updates at all?

Best regards
Torbjørn

RE: Only some of the connected thingy are output

yumi — Mon, 24 Dec 2018 00:31:13 GMT

#define TX_BUFFER_MASK 0x0F

I found this code and changed

but this is not solution

RE: Only some of the connected thingy are output

ovrebekk — Fri, 21 Dec 2018 08:41:06 GMT

This file was included in the ble_thingy_tms_c.zip file, that I sent you in my reply 16 days ago.

Please let me know if you still can't find it, and I will send it again.

Best regards
Torbjørn

RE: Only some of the connected thingy are output

yumi — Thu, 20 Dec 2018 04:39:44 GMT

i don't found ble_thingy_tsm_c.c

how to find ble_thingy_tsm_c.c ?

RE: Only some of the connected thingy are output

ovrebekk — Mon, 17 Dec 2018 12:09:47 GMT

Sorry for the late reply. I have been out travelling and haven't had time to follow up.

At first glance your code looks OK, but when enabling that many characteristics at the same time you might have to increase the buffer size inside the service module.

inside the ble_thingy_tsm_c.c file there should be two defines looking like this:
#define TX_BUFFER_MASK 0x07 /**< TX Buffer mask, must be a mask of continuous zeroes, followed by continuous sequence of ones: 000...111. */
#define TX_BUFFER_SIZE (TX_BUFFER_MASK + 1) /**< Size of send buffer, which is 1 higher than the mask. */

Try to change them to this, and it should work better:
#define TX_BUFFER_MASK 0x0F /**< TX Buffer mask, must be a mask of continuous zeroes, followed by continuous sequence of ones: 000...111. */
#define TX_BUFFER_SIZE (TX_BUFFER_MASK + 1) /**< Size of send buffer, which is 1 higher than the mask. */

Best regards
Torbjørn

RE: Only some of the connected thingy are output

yumi — Fri, 07 Dec 2018 06:18:15 GMT

uint32_t ble_thingy_tms_c_quat_notif_enable(ble_thingy_tms_c_t * p_ble_thingy_tms_c)
{
VERIFY_PARAM_NOT_NULL(p_ble_thingy_tms_c);

if (p_ble_thingy_tms_c->conn_handle == BLE_CONN_HANDLE_INVALID)
{
return NRF_ERROR_INVALID_STATE;
}
// NRF_LOG_INFO("quat notif_enable");

return cccd_configure(p_ble_thingy_tms_c->conn_handle,
p_ble_thingy_tms_c->peer_thingy_tms_db.quat_cccd_handle,
true);
}
uint32_t ble_thingy_tms_c_tap_notif_enable(ble_thingy_tms_c_t * p_ble_thingy_tms_c)
{
VERIFY_PARAM_NOT_NULL(p_ble_thingy_tms_c);

if (p_ble_thingy_tms_c->conn_handle == BLE_CONN_HANDLE_INVALID)
{
return NRF_ERROR_INVALID_STATE;
}
// NRF_LOG_INFO("tap notif_enable");

return cccd_configure(p_ble_thingy_tms_c->conn_handle,
p_ble_thingy_tms_c->peer_thingy_tms_db.tap_cccd_handle,
true);
}

uint32_t ble_thingy_tms_c_pedo_notif_enable(ble_thingy_tms_c_t * p_ble_thingy_tms_c)
{
VERIFY_PARAM_NOT_NULL(p_ble_thingy_tms_c);

if (p_ble_thingy_tms_c->conn_handle == BLE_CONN_HANDLE_INVALID)
{
return NRF_ERROR_INVALID_STATE;
}
// NRF_LOG_INFO("pedo notif_enable");

return cccd_configure(p_ble_thingy_tms_c->conn_handle,
p_ble_thingy_tms_c->peer_thingy_tms_db.pedo_cccd_handle,
true);
}
uint32_t ble_thingy_tms_c_raw_notif_enable(ble_thingy_tms_c_t * p_ble_thingy_tms_c)
{
VERIFY_PARAM_NOT_NULL(p_ble_thingy_tms_c);

if (p_ble_thingy_tms_c->conn_handle == BLE_CONN_HANDLE_INVALID)
{
return NRF_ERROR_INVALID_STATE;
}
// NRF_LOG_INFO("raw notif_enable");

return cccd_configure(p_ble_thingy_tms_c->conn_handle,
p_ble_thingy_tms_c->peer_thingy_tms_db.raw_cccd_handle,
true);

}

uint32_t ble_thingy_tms_c_euler_notif_enable(ble_thingy_tms_c_t * p_ble_thingy_tms_c)
{
VERIFY_PARAM_NOT_NULL(p_ble_thingy_tms_c);

if (p_ble_thingy_tms_c->conn_handle == BLE_CONN_HANDLE_INVALID)
{
return NRF_ERROR_INVALID_STATE;
}

return cccd_configure(p_ble_thingy_tms_c->conn_handle,
p_ble_thingy_tms_c->peer_thingy_tms_db.euler_cccd_handle,
true);
}
uint32_t ble_thingy_tms_c_orientation_notif_enable(ble_thingy_tms_c_t * p_ble_thingy_tms_c)
{
VERIFY_PARAM_NOT_NULL(p_ble_thingy_tms_c);

if (p_ble_thingy_tms_c->conn_handle == BLE_CONN_HANDLE_INVALID)
{
return NRF_ERROR_INVALID_STATE;
}
// NRF_LOG_INFO("orientation notif_enable");

return cccd_configure(p_ble_thingy_tms_c->conn_handle,
p_ble_thingy_tms_c->peer_thingy_tms_db.orientation_cccd_handle,
true);
}
uint32_t ble_thingy_tms_c_gravity_notif_enable(ble_thingy_tms_c_t * p_ble_thingy_tms_c)
{
VERIFY_PARAM_NOT_NULL(p_ble_thingy_tms_c);

if (p_ble_thingy_tms_c->conn_handle == BLE_CONN_HANDLE_INVALID)
{
return NRF_ERROR_INVALID_STATE;
}
NRF_LOG_INFO("gravity notif_enable");

return cccd_configure(p_ble_thingy_tms_c->conn_handle,
p_ble_thingy_tms_c->peer_thingy_tms_db.gravity_cccd_handle,
true);
}
uint32_t ble_thingy_tms_c_heading_notif_enable(ble_thingy_tms_c_t * p_ble_thingy_tms_c)
{
VERIFY_PARAM_NOT_NULL(p_ble_thingy_tms_c);

if (p_ble_thingy_tms_c->conn_handle == BLE_CONN_HANDLE_INVALID)
{
return NRF_ERROR_INVALID_STATE;
}
NRF_LOG_INFO("heading notif_enable");
return cccd_configure(p_ble_thingy_tms_c->conn_handle,
p_ble_thingy_tms_c->peer_thingy_tms_db.heading_cccd_handle,
true);
}
uint32_t ble_thingy_tms_c_config_notif_enable(ble_thingy_tms_c_t * p_ble_thingy_tms_c)
{
VERIFY_PARAM_NOT_NULL(p_ble_thingy_tms_c);

if (p_ble_thingy_tms_c->conn_handle == BLE_CONN_HANDLE_INVALID)
{
return NRF_ERROR_INVALID_STATE;
}

return cccd_configure(p_ble_thingy_tms_c->conn_handle,
p_ble_thingy_tms_c->peer_thingy_tms_db.config_cccd_handle,
true);
}
uint32_t ble_thingy_tms_c_rot_notif_enable(ble_thingy_tms_c_t * p_ble_thingy_tms_c)
{
VERIFY_PARAM_NOT_NULL(p_ble_thingy_tms_c);

if (p_ble_thingy_tms_c->conn_handle == BLE_CONN_HANDLE_INVALID)
{
return NRF_ERROR_INVALID_STATE;
}

return cccd_configure(p_ble_thingy_tms_c->conn_handle,
p_ble_thingy_tms_c->peer_thingy_tms_db.rot_cccd_handle,
true);
}

this is my code in ble.c

there is code in main.c

static void thingy_tms_c_evt_handler(ble_thingy_tms_c_t * p_thingy_tms_c, ble_thingy_tms_c_evt_t * p_thingy_tms_c_evt)
{
ret_code_t err_code;
switch (p_thingy_tms_c_evt->evt_type)
{
case BLE_THINGY_TMS_C_EVT_DISCOVERY_COMPLETE:
{

// uint32_t ble_thingy_tms_c_raw_notif_enable(ble_thingy_tms_c_t * p_ble_thingy_tms_c)
err_code = ble_thingy_tms_c_heading_notif_enable(p_thingy_tms_c);
APP_ERROR_CHECK(err_code);
err_code = ble_thingy_tms_c_raw_notif_enable(p_thingy_tms_c);
APP_ERROR_CHECK(err_code);
err_code = ble_thingy_tms_c_quat_notif_enable(p_thingy_tms_c);
APP_ERROR_CHECK(err_code);
err_code = ble_thingy_tms_c_orientation_notif_enable(p_thingy_tms_c);
APP_ERROR_CHECK(err_code);
err_code = ble_thingy_tms_c_tap_notif_enable(p_thingy_tms_c);
APP_ERROR_CHECK(err_code);
err_code = ble_thingy_tms_c_pedo_notif_enable(p_thingy_tms_c);
APP_ERROR_CHECK(err_code);
err_code = ble_thingy_tms_c_euler_notif_enable(p_thingy_tms_c);
APP_ERROR_CHECK(err_code);
err_code = ble_thingy_tms_c_gravity_notif_enable(p_thingy_tms_c);
APP_ERROR_CHECK(err_code);
err_code = ble_thingy_tms_c_config_notif_enable(p_thingy_tms_c);
APP_ERROR_CHECK(err_code);
err_code = ble_thingy_tms_c_rot_notif_enable(p_thingy_tms_c);
APP_ERROR_CHECK(err_code);

ble_gap_conn_params_t conn_params;
conn_params.max_conn_interval = MAX_CONNECTION_INTERVAL;
conn_params.min_conn_interval = MIN_CONNECTION_INTERVAL;
conn_params.slave_latency = SLAVE_LATENCY;
conn_params.conn_sup_timeout = SUPERVISION_TIMEOUT;

sd_ble_gap_conn_param_update(p_thingy_tms_c_evt->conn_handle, &conn_params);

} break;
// case BLE_THINGY_TES_C_EVT_TEMPERATURE_NOTIFICATION:
//app_aggregator_temp_update(p_thingy_tes_c_evt->conn_handle, p_thingy_tes_c_evt->params.temperature.integer);
// break;
// case BLE_THINGY_TES_C_EVT_HUMIDITY_NOTIFICATION:
//app_aggregator_humid_update(p_thingy_tes_c_evt->conn_handle, p_thingy_tes_c_evt->params.humidity.humidity);
default :/*
app_aggregator_temp_update(p_thingy_tes_c_evt->conn_handle, p_thingy_tes_c_evt->params.temperature.integer);
app_aggregator_humid_update(p_thingy_tes_c_evt->conn_handle, p_thingy_tes_c_evt->params.humidity.humidity);*/
break;

}
}

it is correct?

RE: Only some of the connected thingy are output

ovrebekk — Wed, 05 Dec 2018 14:39:50 GMT

It seems you are setting up the handles correctly, but you also need functions to enable notifications.

As luck should have it I recently implemented the TMS client for a different project, and I am attaching the files here so you can use it for reference:
ble_thingy_tms_c.zip

Please note all the functions at the bottom of the .c file called something like ble_thingy_tms_c_xxx_notif_enable(..)

You need to call this function for each characteristic from which you want to receive notifications.

Best regards
Torbjørn

RE: Only some of the connected thingy are output

yumi — Wed, 05 Dec 2018 06:31:27 GMT

case THINGY_TMS_UUID_EULER_CHAR :
evt.params.peer_db.euler_cccd_handle = p_char->cccd_handle;
evt.params.peer_db.euler_handle = p_char->characteristic.handle_value;
NRF_LOG_INFO("evt euler handle = %d",p_ble_thingy_tms_c->conn_handle, evt.params.peer_db.euler_handle);
break;

case THINGY_TMS_UUID_CONFIG_CHAR :
evt.params.peer_db.config_cccd_handle = p_char->cccd_handle;
evt.params.peer_db.config_handle = p_char->characteristic.handle_value;
NRF_LOG_INFO("%d >> evt config handle = %d", p_ble_thingy_tms_c->conn_handle,evt.params.peer_db.config_handle);
break;

case THINGY_TMS_UUID_GRAVITY_CHAR :
evt.params.peer_db.gravity_cccd_handle = p_char->cccd_handle;
evt.params.peer_db.gravity_handle = p_char->characteristic.handle_value;
NRF_LOG_INFO("%d >> evt gravity handle = %d", p_ble_thingy_tms_c->conn_handle, evt.params.peer_db.gravity_handle);

case THINGY_TMS_UUID_ROT_MAT_CHAR :
evt.params.peer_db.rot_cccd_handle = p_char->cccd_handle;
evt.params.peer_db.rot_handle = p_char->characteristic.handle_value;
NRF_LOG_INFO("%d >> evt rot handle = %d", p_ble_thingy_tms_c->conn_handle,evt.params.peer_db.rot_handle);
break;

case THINGY_TMS_UUID_HEADING_CHAR :
evt.params.peer_db.heading_cccd_handle = p_char->cccd_handle;
evt.params.peer_db.heading_handle = p_char->characteristic.handle_value;
NRF_LOG_INFO("%d >> evt heading handle = %d", p_ble_thingy_tms_c->conn_handle, evt.params.peer_db.heading_handle);
break;

case THINGY_TMS_UUID_ORIENTATION_CHAR:
evt.params.peer_db.orientation_cccd_handle = p_char->cccd_handle;
evt.params.peer_db.orientation_handle = p_char->characteristic.handle_value;
NRF_LOG_INFO("%d >> evt orientation handle = %d",p_ble_thingy_tms_c->conn_handle, evt.params.peer_db.orientation_handle);
break;

case THINGY_TMS_UUID_QUATERNION_CHAR :
evt.params.peer_db.quat_cccd_handle = p_char->cccd_handle;
evt.params.peer_db.quat_handle = p_char->characteristic.handle_value;
NRF_LOG_INFO("%d >> evt quaternion handle = %d", p_ble_thingy_tms_c->conn_handle, evt.params.peer_db.quat_handle);
break;

*/

case THINGY_TMS_UUID_PEDOMETER_CHAR:
evt.params.peer_db.pedo_cccd_handle = p_char->cccd_handle;
evt.params.peer_db.pedo_handle = p_char->characteristic.handle_value;
NRF_LOG_INFO("%d >> evt pedo handle = %d", p_ble_thingy_tms_c->conn_handle,evt.params.peer_db.pedo_handle);
break;

case THINGY_TMS_UUID_RAW_CHAR :
evt.params.peer_db.raw_cccd_handle = p_char->cccd_handle;
evt.params.peer_db.raw_handle = p_char->characteristic.handle_value;
NRF_LOG_INFO("%d >> evt raw handle = %d",p_ble_thingy_tms_c->conn_handle, evt.params.peer_db.raw_handle);
break;

case THINGY_TMS_UUID_TAP_CHAR:
evt.params.peer_db.tap_cccd_handle = p_char->cccd_handle;
evt.params.peer_db.tap_handle = p_char->characteristic.handle_value;
NRF_LOG_INFO("%d >> evt tap handle = %d", p_ble_thingy_tms_c->conn_handle,evt.params.peer_db.tap_handle);
break;

this is my code

RE: Only some of the connected thingy are output

ovrebekk — Tue, 04 Dec 2018 17:05:27 GMT

The Thingy's should update values automatically as soon as you enable notifications for the corresponding characteristic (set the CCCD).

You can enable notifications on multiple characteristics in one thingy, and it should be possible to enable notifications on multiple characteristics in multiple thingy's as well.

If this is not working it might be down to the method you use for enabling notifications across the different thingy's.

Best regards
Torbjørn

RE: Only some of the connected thingy are output

yumi — Mon, 03 Dec 2018 06:29:46 GMT

I modified the code to access the environmental values and motion value

but if one thingy access one value such as temperature, that don't access other values.

It is possible for thingy to access a lot of values?
and It is possible for multi thingy to access a lot of values?

RE: Only some of the connected thingy are output

ovrebekk — Fri, 30 Nov 2018 13:18:48 GMT

If you pull the latest update I increased the max connections to 16 for the nRF52840 project.

Have you modified the code to access the environmental values?
Out of the box the multi link demo only accesses the UI service in the Thingys.

Best regards
Torbjørn