There is no need to check if device has specific services, it was checked in session_init_control. --- profiles/audio/avrcp.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/profiles/audio/avrcp.c b/profiles/audio/avrcp.c index b1552b1..a57756e 100644 --- a/profiles/audio/avrcp.c +++ b/profiles/audio/avrcp.c @@ -3394,8 +3394,7 @@ static void target_init(struct avrcp *session) DBG("%p version 0x%04x", target, target->version); service = btd_device_get_service(session->dev, AVRCP_REMOTE_UUID); - if (service != NULL) - btd_service_connecting_complete(service, 0); + btd_service_connecting_complete(service, 0); player = g_slist_nth_data(server->players, 0); if (player != NULL) { @@ -3440,8 +3439,7 @@ static void controller_init(struct avrcp *session) session->supported_events |= (1 << AVRCP_EVENT_VOLUME_CHANGED); service = btd_device_get_service(session->dev, AVRCP_TARGET_UUID); - if (service != NULL) - btd_service_connecting_complete(service, 0); + btd_service_connecting_complete(service, 0); /* Only create player if category 1 is supported */ if (!(controller->features & AVRCP_FEATURE_CATEGORY_1)) -- 2.1.0 -- 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