All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arman Uguray <armansito@chromium.org>
To: BlueZ development <linux-bluetooth@vger.kernel.org>
Subject: Re: [PATCH BlueZ 00/12] core: Use shared/gatt-client.
Date: Tue, 18 Nov 2014 19:31:15 -0800	[thread overview]
Message-ID: <CAHrH25RC9HwEm-Td1cH8A7swJdSpkcO5P-y9qjB=m9OHzMMGcw@mail.gmail.com> (raw)
In-Reply-To: <1416252138-17477-1-git-send-email-armansito@chromium.org>

Hi all,

> On Mon, Nov 17, 2014 at 11:22 AM, Arman Uguray <armansito@chromium.org> wrote:
> This patch set integrates shared/gatt-client into bluetoothd. The following
> items have been tackled here:
>
>   1. src/device now uses shared/gatt-client to perform service discovery. State
>   updates such as disconnections, service changed events, etc. are done using
>   callback APIs of the new shared framework and attrib/gatt is no longer used to
>   perform GATT operations directly. A GAttrib instance is maintained for
>   backwards compatibility with profiles that haven't been converted yet.
>
>   2. A new gatt-callbacks API has been added which mirrors the attio APIs for
>   shared/gatt-client. Profiles/plugins are notified using these APIs when the
>   bearer disconnects, when services change, and when gatt-client is ready.
>
>   3. The profiles/gatt plugin has been rewritten. Since the GATT service is
>   handled by shared/gatt-client, this profile no longer deals with service
>   changed events performs out of place MTU exchanges, which used to get
>   performed in an arbitrary order. The profile has been renamed to profiles/gap
>   as it now only deals with the GAP service, and reads the "Appearance" and
>   "Device Name" characteristics using the new shared stack.
>
> Arman Uguray (12):
>   attrib/gattrib: Add g_attrib_get_att.
>   shared/att: Add bt_att_get_fd.
>   shared/att: Directly close fd on ATT violations.
>   shared/gatt-client: Call ready_handler only in init.
>   core: device: Use bt_att_register_disconnect.
>   core: device: Use shared/gatt-client for GATT.
>   core: Introduce gatt-callbacks
>   profiles/gatt: Don't handle GATT service.
>   profiles/gatt: Rename profile to gap.
>   profiles/gap: Rewrite using bt_gatt_client.
>   profiles/gap: Add Google copyright.
>   TODO: Update GAttrib related items.
>
>  Makefile.am              |   1 +
>  Makefile.plugins         |   4 +-
>  TODO                     |  25 +--
>  attrib/gattrib.c         |   8 +
>  attrib/gattrib.h         |   3 +
>  profiles/gap/gas.c       | 307 +++++++++++++++++++++++++++++++
>  profiles/gatt/gas.c      | 457 -----------------------------------------------
>  src/device.c             | 456 +++++++++++++++++++++++++++++++---------------
>  src/device.h             |   2 -
>  src/gatt-callbacks.h     |  42 +++++
>  src/shared/att-types.h   |   3 +-
>  src/shared/att.c         |  25 ++-
>  src/shared/att.h         |   1 +
>  src/shared/gatt-client.c |   3 +-
>  14 files changed, 711 insertions(+), 626 deletions(-)
>  create mode 100644 profiles/gap/gas.c
>  delete mode 100644 profiles/gatt/gas.c
>  create mode 100644 src/gatt-callbacks.h
>
> --
> 2.1.0.rc2.206.gedb03e5
>

ping

      parent reply	other threads:[~2014-11-19  3:31 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-17 19:22 [PATCH BlueZ 00/12] core: Use shared/gatt-client Arman Uguray
2014-11-17 19:22 ` [PATCH BlueZ 01/12] attrib/gattrib: Add g_attrib_get_att Arman Uguray
2014-11-19 15:05   ` Luiz Augusto von Dentz
2014-11-19 15:14     ` Arman Uguray
2014-11-19 15:15     ` Michael Janssen
2014-11-19 15:54       ` Luiz Augusto von Dentz
2014-11-17 19:22 ` [PATCH BlueZ 02/12] shared/att: Add bt_att_get_fd Arman Uguray
2014-11-19 15:07   ` Luiz Augusto von Dentz
2014-11-17 19:22 ` [PATCH BlueZ 03/12] shared/att: Directly close fd on ATT violations Arman Uguray
2014-11-17 19:22 ` [PATCH BlueZ 04/12] shared/gatt-client: Call ready_handler only in init Arman Uguray
2014-11-17 19:22 ` [PATCH BlueZ 05/12] core: device: Use bt_att_register_disconnect Arman Uguray
2014-11-17 19:22 ` [PATCH BlueZ 06/12] core: device: Use shared/gatt-client for GATT Arman Uguray
2014-11-19 15:43   ` Luiz Augusto von Dentz
2014-11-19 16:42     ` Arman Uguray
2014-11-19 17:09       ` Luiz Augusto von Dentz
2014-11-19 17:31         ` Arman Uguray
2014-11-17 19:22 ` [PATCH BlueZ 07/12] core: Introduce gatt-callbacks Arman Uguray
2014-11-17 19:22 ` [PATCH BlueZ 08/12] profiles/gatt: Don't handle GATT service Arman Uguray
2014-11-17 19:22 ` [PATCH BlueZ 09/12] profiles/gatt: Rename profile to gap Arman Uguray
2014-11-17 19:22 ` [PATCH BlueZ 10/12] profiles/gap: Rewrite using bt_gatt_client Arman Uguray
2014-11-17 19:22 ` [PATCH BlueZ 11/12] profiles/gap: Add Google copyright Arman Uguray
2014-11-17 19:22 ` [PATCH BlueZ 12/12] TODO: Update GAttrib related items Arman Uguray
2014-11-19  3:31 ` Arman Uguray [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAHrH25RC9HwEm-Td1cH8A7swJdSpkcO5P-y9qjB=m9OHzMMGcw@mail.gmail.com' \
    --to=armansito@chromium.org \
    --cc=linux-bluetooth@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.