From: Rafal Garbat <rafal.garbat@xxxxxxxxx> This patch adds support to discover Heart Rate Service characteristics when connected to remote device. --- profiles/heartrate/heartrate.c | 49 +++++++++++++++++++++++++++++++++++++++++- profiles/heartrate/heartrate.h | 3 ++- profiles/heartrate/manager.c | 22 ++++++++++++++++++- 3 files changed, 71 insertions(+), 3 deletions(-) diff --git a/profiles/heartrate/heartrate.c b/profiles/heartrate/heartrate.c index 918546e..e4147cf 100644 --- a/profiles/heartrate/heartrate.c +++ b/profiles/heartrate/heartrate.c @@ -48,6 +48,13 @@ struct heartrate { struct heartrate_adapter *hra; GAttrib *attrib; guint attioid; + struct att_range *svc_range; + GSList *chars; +}; + +struct characteristic { + struct gatt_char attr; + struct heartrate *hr; }; static GSList *heartrate_adapters = NULL; @@ -95,7 +102,11 @@ static void destroy_heartrate_device(gpointer user_data) if (hr->attrib != NULL) g_attrib_unref(hr->attrib); + if (hr->chars != NULL) + g_slist_free_full(hr->chars, g_free); + btd_device_unref(hr->dev); + g_free(hr->svc_range); g_free(hr); } @@ -109,6 +120,33 @@ static void destroy_heartrate_adapter(gpointer user_data) g_free(hra); } +static void configure_heartrate_cb(GSList *characteristics, guint8 status, + gpointer user_data) +{ + struct heartrate *hr = user_data; + GSList *l; + + if (status != 0) { + error("Discover Heart Rate characteristics: %s", + att_ecode2str(status)); + return; + } + + for (l = characteristics; l; l = l->next) { + struct gatt_char *c = l->data; + struct characteristic *ch; + + ch = g_new0(struct characteristic, 1); + ch->attr.handle = c->handle; + ch->attr.properties = c->properties; + ch->attr.value_handle = c->value_handle; + memcpy(ch->attr.uuid, c->uuid, MAX_LEN_UUID_STR + 1); + ch->hr = hr; + + hr->chars = g_slist_append(hr->chars, ch); + } +} + static void attio_connected_cb(GAttrib *attrib, gpointer user_data) { struct heartrate *hr = user_data; @@ -116,6 +154,10 @@ static void attio_connected_cb(GAttrib *attrib, gpointer user_data) DBG("GATT Connected"); hr->attrib = g_attrib_ref(attrib); + + gatt_discover_char(hr->attrib, hr->svc_range->start, + hr->svc_range->end, NULL, + configure_heartrate_cb, hr); } static void attio_disconnected_cb(gpointer user_data) @@ -153,7 +195,8 @@ void heartrate_adapter_unregister(struct btd_adapter *adapter) destroy_heartrate_adapter(hra); } -int heartrate_device_register(struct btd_device *device) +int heartrate_device_register(struct btd_device *device, + struct gatt_primary *pattr) { struct heartrate_adapter *hra; struct btd_adapter *adapter; @@ -172,6 +215,10 @@ int heartrate_device_register(struct btd_device *device) hra->devices = g_slist_prepend(hra->devices, hr); + hr->svc_range = g_new0(struct att_range, 1); + hr->svc_range->start = pattr->range.start; + hr->svc_range->end = pattr->range.end; + hr->attioid = btd_device_add_attio_callback(device, attio_connected_cb, attio_disconnected_cb, diff --git a/profiles/heartrate/heartrate.h b/profiles/heartrate/heartrate.h index 486f5b3..ec3ed7a 100644 --- a/profiles/heartrate/heartrate.h +++ b/profiles/heartrate/heartrate.h @@ -22,5 +22,6 @@ int heartrate_adapter_register(struct btd_adapter *adapter); void heartrate_adapter_unregister(struct btd_adapter *adapter); -int heartrate_device_register(struct btd_device *device); +int heartrate_device_register(struct btd_device *device, + struct gatt_primary *pattr); void heartrate_device_unregister(struct btd_device *device); diff --git a/profiles/heartrate/manager.c b/profiles/heartrate/manager.c index 2a9326d..d7d7ae4 100644 --- a/profiles/heartrate/manager.c +++ b/profiles/heartrate/manager.c @@ -34,6 +34,14 @@ #include "heartrate.h" #include "manager.h" +static gint primary_uuid_cmp(gconstpointer a, gconstpointer b) +{ + const struct gatt_primary *prim = a; + const char *uuid = b; + + return g_strcmp0(prim->uuid, uuid); +} + static int heartrate_adapter_probe(struct btd_adapter *adapter) { return heartrate_adapter_register(adapter); @@ -46,7 +54,19 @@ static void heartrate_adapter_remove(struct btd_adapter *adapter) static int heartrate_device_probe(struct btd_device *device, GSList *uuids) { - return heartrate_device_register(device); + struct gatt_primary *pattr; + GSList *primaries, *l; + + primaries = btd_device_get_primaries(device); + + l = g_slist_find_custom(primaries, HEART_RATE_UUID, + primary_uuid_cmp); + if (l == NULL) + return -EINVAL; + + pattr = l->data; + + return heartrate_device_register(device, pattr); } static void heartrate_device_remove(struct btd_device *device) -- 1.7.11.3 -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html