Re: [PATCH BlueZ] core/gatt: Fix not exporting new services

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

 



Hi,

On Fri, Sep 4, 2015 at 11:46 AM, Luiz Augusto von Dentz
<luiz.dentz@xxxxxxxxx> wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>
>
> Ever since cache validation was introduced the services are no longer
> cleared on disconnect so checking for queue empty is not valid anymore.
> ---
>  src/gatt-client.c | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/src/gatt-client.c b/src/gatt-client.c
> index 3356ee4..d8a3429 100644
> --- a/src/gatt-client.c
> +++ b/src/gatt-client.c
> @@ -1538,6 +1538,9 @@ static struct service *service_create(struct gatt_db_attribute *attr,
>         /* Set service active so we can skip discovering next time */
>         gatt_db_service_set_active(attr, true);
>
> +       /* Mark the service as claimed since it going to be exported */
> +       gatt_db_service_set_claimed(attr, true);
> +
>         return service;
>  }
>
> @@ -1840,11 +1843,7 @@ void btd_gatt_client_ready(struct btd_gatt_client *client)
>
>         DBG("GATT client ready");
>
> -       if (queue_isempty(client->services)) {
> -               DBG("Exporting services");
> -               create_services(client);
> -               return;
> -       }
> +       create_services(client);
>
>         /*
>          * Services have already been created before. Re-enable notifications
> --
> 2.4.3

Applied.



-- 
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