Re: [PATCH BlueZ 1/2] shared/gatt-client: Don't send Exchange MTU for default value

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

 



Hi,

On Fri, Mar 4, 2016 at 2:24 PM, Luiz Augusto von Dentz
<luiz.dentz@xxxxxxxxx> wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>
>
> If the MTU to be exchange is the default there is no pointing in sending
> it since the remote already assumes it anyway.
> ---
>  src/shared/gatt-client.c | 19 +++++++++++++++++++
>  1 file changed, 19 insertions(+)
>
> diff --git a/src/shared/gatt-client.c b/src/shared/gatt-client.c
> index b255175..45acf7b 100644
> --- a/src/shared/gatt-client.c
> +++ b/src/shared/gatt-client.c
> @@ -1544,6 +1544,11 @@ static bool gatt_client_init(struct bt_gatt_client *client, uint16_t mtu)
>         if (!op)
>                 return false;
>
> +       /* Check if MTU needs to be send */
> +       mtu = MAX(BT_ATT_DEFAULT_LE_MTU, mtu);
> +       if (mtu == BT_ATT_DEFAULT_LE_MTU)
> +               goto discover;
> +
>         /* Configure the MTU */
>         client->mtu_req_id = bt_gatt_exchange_mtu(client->att,
>                                                 MAX(BT_ATT_DEFAULT_LE_MTU, mtu),
> @@ -1558,6 +1563,20 @@ static bool gatt_client_init(struct bt_gatt_client *client, uint16_t mtu)
>         client->in_init = true;
>
>         return true;
> +
> +discover:
> +       client->discovery_req = bt_gatt_discover_all_primary_services(
> +                                                       client->att, NULL,
> +                                                       discover_primary_cb,
> +                                                       discovery_op_ref(op),
> +                                                       discovery_op_unref);
> +       if (!client->discovery_req) {
> +               discovery_op_free(op);
> +               return false;
> +       }
> +
> +       client->in_init = true;
> +       return true;
>  }
>
>  struct pdu_data {
> --
> 2.5.0

Applied (1/2 only).


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