Return-Path: From: Bharat Bhusan Panda To: linux-bluetooth@vger.kernel.org Cc: cpgs@samsung.com References: <1445346988-12782-1-git-send-email-bharat.panda@samsung.com> <1445346988-12782-2-git-send-email-bharat.panda@samsung.com> In-reply-to: <1445346988-12782-2-git-send-email-bharat.panda@samsung.com> Subject: RE: [PATCH 2/3] audio/avrcp: Handle PlaylistChanged signal Date: Fri, 27 Nov 2015 16:51:30 +0530 Message-id: <001c01d12905$cd41ec80$67c5c580$@samsung.com> MIME-version: 1.0 Content-type: text/plain; charset=us-ascii Sender: linux-bluetooth-owner@vger.kernel.org List-ID: ping > -----Original Message----- > From: linux-bluetooth-owner@vger.kernel.org [mailto:linux-bluetooth- > owner@vger.kernel.org] On Behalf Of Bharat Panda > Sent: Tuesday, October 20, 2015 6:46 PM > To: linux-bluetooth@vger.kernel.org > Cc: cpgs@samsung.com; Bharat Panda > Subject: [PATCH 2/3] audio/avrcp: Handle PlaylistChanged signal > > Added signal handler for "PlaylistChanged" on MediaPlayer2.Playlists > interface. > --- > profiles/audio/media.c | 79 > ++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 79 insertions(+) > > diff --git a/profiles/audio/media.c b/profiles/audio/media.c index > 1b5246d..94fce79 100644 > --- a/profiles/audio/media.c > +++ b/profiles/audio/media.c > @@ -113,6 +113,7 @@ struct media_player { > guint watch; > guint properties_watch; > guint seek_watch; > + guint playlist_watch; > char *status; > uint32_t position; > uint32_t duration; > @@ -973,6 +974,7 @@ static void media_player_free(gpointer data) > g_dbus_remove_watch(conn, mp->watch); > g_dbus_remove_watch(conn, mp->properties_watch); > g_dbus_remove_watch(conn, mp->seek_watch); > + g_dbus_remove_watch(conn, mp->playlist_watch); > > if (mp->track) > g_hash_table_unref(mp->track); > @@ -1913,6 +1915,76 @@ static gboolean > position_changed(DBusConnection *connection, DBusMessage *msg, > return TRUE; > } > > +static struct media_playlist *find_playlist_by_id(struct media_player *mp, > + char *playlist_id) > +{ > + GSList *l; > + > + for (l = mp->playlists; l; l = l->next) { > + struct media_playlist *playlist = l->data; > + > + if (g_strcmp0(playlist->id, playlist_id) == 0) { > + return playlist; > + } > + } > + > + return NULL; > +} > + > +static gboolean playlist_changed(DBusConnection *connection, > + DBusMessage *msg, > + void *user_data) > +{ > + struct media_player *mp = user_data; > + struct media_playlist *playlist; > + DBusMessageIter iter; > + DBusMessageIter entry; > + char *playlist_id; > + char *name; > + char *icon; > + > + dbus_message_iter_init(msg, &iter); > + if (dbus_message_iter_get_arg_type(&iter) != DBUS_TYPE_STRUCT) > + return FALSE; > + > + dbus_message_iter_recurse(&iter, &entry); > + > + if (dbus_message_iter_get_arg_type(&entry) != > + DBUS_TYPE_OBJECT_PATH) > + return FALSE; > + > + dbus_message_iter_get_basic(&entry, &playlist_id); > + > + dbus_message_iter_next(&entry); > + if (dbus_message_iter_get_arg_type(&entry) != > + DBUS_TYPE_STRING) > + return FALSE; > + > + dbus_message_iter_get_basic(&entry, &name); > + > + dbus_message_iter_next(&entry); > + if (dbus_message_iter_get_arg_type(&entry) != > + DBUS_TYPE_STRING) > + return FALSE; > + > + dbus_message_iter_get_basic(&entry, &icon); > + > + playlist = find_playlist_by_id(mp, playlist_id); > + > + if (playlist) { > + playlist->name = g_strdup(name); > + playlist->icon = g_strdup(icon); > + } else { > + playlist = g_new0(struct media_playlist, 1); > + > + playlist->id = playlist_id; > + playlist->name = g_strdup(name); > + playlist->icon = g_strdup(icon); > + } > + > + return TRUE; > +} > + > static struct media_player *media_player_create(struct media_adapter > *adapter, > const char *sender, > const char *path, > @@ -1995,6 +2067,13 @@ static DBusMessage > *register_player(DBusConnection *conn, DBusMessage *msg, > if (!get_playlists(mp)) > DBG("Error fetching playlists"); > > + mp->playlist_watch = g_dbus_add_signal_watch(conn, > + sender, path, > + > MEDIA_PLAYER_PLAYLIST_INTERFACE, > + "PlaylistChanged", > + playlist_changed, > + mp, NULL); > + > return g_dbus_create_reply(msg, DBUS_TYPE_INVALID); } > > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in > the body of a message to majordomo@vger.kernel.org More majordomo > info at http://vger.kernel.org/majordomo-info.html