Re: [PATCH BlueZ 2/2] android/A2DP: Fix sending notification on bt_a2dp_unregister

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Mon, Jan 20, 2014 at 2:25 PM, Luiz Augusto von Dentz
<luiz.dentz@xxxxxxxxx> wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>
>
> At this point IPC might have been closed already.
> ---
>  android/a2dp.c | 13 ++++---------
>  1 file changed, 4 insertions(+), 9 deletions(-)
>
> diff --git a/android/a2dp.c b/android/a2dp.c
> index 8ec03c8..5569691 100644
> --- a/android/a2dp.c
> +++ b/android/a2dp.c
> @@ -119,8 +119,10 @@ static void unregister_endpoint(void *data)
>         g_free(endpoint);
>  }
>
> -static void a2dp_device_free(struct a2dp_device *dev)
> +static void a2dp_device_free(void *data)
>  {
> +       struct a2dp_device *dev = data;
> +
>         if (dev->idle_id > 0)
>                 g_source_remove(dev->idle_id);
>
> @@ -1471,13 +1473,6 @@ fail:
>         return false;
>  }
>
> -static void a2dp_device_disconnected(gpointer data, gpointer user_data)
> -{
> -       struct a2dp_device *dev = data;
> -
> -       bt_a2dp_notify_state(dev, HAL_A2DP_STATE_DISCONNECTED);
> -}
> -
>  void bt_a2dp_unregister(void)
>  {
>         DBG("");
> @@ -1488,7 +1483,7 @@ void bt_a2dp_unregister(void)
>         g_slist_free_full(endpoints, unregister_endpoint);
>         endpoints = NULL;
>
> -       g_slist_foreach(devices, a2dp_device_disconnected, NULL);
> +       g_slist_free_full(devices, a2dp_device_free);
>         devices = NULL;
>
>         ipc_unregister(HAL_SERVICE_ID_A2DP);
> --
> 1.8.4.2

Applied, patch 1/2 will need some changes so it is not applied yet.


-- 
Luiz Augusto von Dentz
--
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




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux