Thanks to this patch hog instances can be now stored in queue.
This will help to clean the code from glib dependencies.
---
android/hog.c | 34 +++++++++++++++-------------------
1 file changed, 15 insertions(+), 19 deletions(-)
diff --git a/android/hog.c b/android/hog.c
index 9b76357..6bec2aa 100644
--- a/android/hog.c
+++ b/android/hog.c
@@ -101,7 +101,7 @@ struct bt_hog {
struct bt_scpp *scpp;
struct bt_dis *dis;
struct queue *bas;
- GSList *instances;
+ struct queue *instances;
struct bt_gatt_client *client;
struct gatt_db *db;
};
@@ -875,7 +875,7 @@ static void hog_free(void *data)
bt_hog_detach(hog);
queue_destroy(hog->bas, (void *) bt_bas_unref);
- g_slist_free_full(hog->instances, hog_free);
+ queue_destroy(hog->instances, hog_free);
bt_scpp_unref(hog->scpp);
bt_dis_unref(hog->dis);
@@ -904,6 +904,7 @@ struct bt_hog *bt_hog_new(int fd, const char *name, uint16_t vendor,
hog->bas = queue_new();
hog->reports = queue_new();
+ hog->instances = queue_new();
if (fd < 0)
hog->uhid = bt_uhid_new_default();
@@ -912,7 +913,7 @@ struct bt_hog *bt_hog_new(int fd, const char *name, uint16_t vendor,
hog->uhid_fd = fd;
- if (!hog->bas || !hog->uhid || !hog->reports) {
+ if (!hog->bas || !hog->uhid || !hog->reports || !hog->instances) {
hog_free(hog);
return NULL;
}
@@ -1026,7 +1027,7 @@ static void hog_attach_hog(struct bt_hog *hog, uint16_t service_handle)
return;
bt_hog_attach(instance, hog->attrib, hog->client);
- hog->instances = g_slist_append(hog->instances, instance);
+ queue_push_tail(hog->instances, instance);
}
static void primary_cb(uint8_t status, GSList *services, void *user_data)
@@ -1095,7 +1096,7 @@ static void service_cb(struct gatt_db_attribute *attrib, void *user_data)
bool bt_hog_attach(struct bt_hog *hog, void *gatt, void *client)
{
- GSList *l;
+ const struct queue_entry *hog_entry;
if (hog->attrib || hog->client)
return false;
@@ -1118,8 +1119,9 @@ bool bt_hog_attach(struct bt_hog *hog, void *gatt, void *client)
queue_foreach(hog->bas, (void *) bt_bas_attach, gatt);
- for (l = hog->instances; l; l = l->next) {
- struct bt_hog *instance = l->data;
+ hog_entry = queue_get_entries(hog->instances);
+ while (hog_entry) {
+ struct bt_hog *instance = hog_entry->data;
bt_hog_attach(instance, gatt, client);
}
@@ -1140,19 +1142,11 @@ bool bt_hog_attach(struct bt_hog *hog, void *gatt, void *client)
void bt_hog_detach(struct bt_hog *hog)
{
- GSList *l;
-
if (!hog->attrib || !hog->client)
return;
queue_foreach(hog->bas, (void *) bt_bas_detach, NULL);
-
- for (l = hog->instances; l; l = l->next) {
- struct bt_hog *instance = l->data;
-
- bt_hog_detach(instance);
- }
-
+ queue_foreach(hog->instances, (void *) bt_hog_detach, NULL);
queue_foreach(hog->reports, report_disable_notif, NULL);
if (hog->scpp)
@@ -1187,7 +1181,7 @@ int bt_hog_set_control_point(struct bt_hog *hog, bool suspend)
int bt_hog_send_report(struct bt_hog *hog, void *data, size_t size, int type)
{
struct report *report;
- GSList *l;
+ const struct queue_entry *hog_entry;
if (!hog)
return -EINVAL;
@@ -1212,10 +1206,12 @@ int bt_hog_send_report(struct bt_hog *hog, void *data, size_t size, int type)
report->decl->value_handle,
false, data, size);
- for (l = hog->instances; l; l = l->next) {
- struct bt_hog *instance = l->data;
+ hog_entry = queue_get_entries(hog->instances);
+ while (hog_entry) {
+ struct bt_hog *instance = hog_entry->data;
bt_hog_send_report(instance, data, size, type);
+ hog_entry = hog_entry->next;
}
return 0;
--
1.9.1
--
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
