From: Yunhan Wang <yunhanw@nestlabs.com>
To: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Cc: "linux-bluetooth@vger.kernel.org"
<linux-bluetooth@vger.kernel.org>, Jonah Petri <jonah@sense.com>
Subject: Re: [PATCH v2 00/10] gatt: Add server support for AcquireWrite and AcquireNotify
Date: Thu, 21 Sep 2017 11:15:02 -0700 [thread overview]
Message-ID: <CALvjcs9cM-2u6423kg3ybxVkrzCyJfNqFfEUM5KJRPo0MhCLPg@mail.gmail.com> (raw)
In-Reply-To: <CABBYNZJTYWfm6BWOTBt4i2XDbKmg_4TW936gUpFCMSvugTE58Q@mail.gmail.com>
Hi, Luiz
It is pretty good.
Thanks
Best wishes
Yunhan
On Thu, Sep 21, 2017 at 12:49 AM, Luiz Augusto von Dentz
<luiz.dentz@gmail.com> wrote:
> Hi,
>
> On Wed, Sep 20, 2017 at 10:44 AM, Luiz Augusto von Dentz
> <luiz.dentz@gmail.com> wrote:
>> From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
>>
>> This implements similar mechanist to use dedicated file descriptors
>> for IO bypassing D-Bus.
>>
>> Note: The current implementation opens just one fd per characteristic,
>> though it could be possible to have one fd per device.
>>
>> v2: Allow any type of write procedure to work when WriteAcquired exists.
>>
>> Luiz Augusto von Dentz (10):
>> gatt: Remove useless debug
>> client: Rework variables for AcquireWrite/AcquireNotify
>> doc/gatt-api: Add server support for AcquireWrite and AcquireNotify
>> shared/gatt-server: Add bt_gatt_server_get_mtu
>> shared/gatt-db: Add gatt_db_attribute_get_user_data
>> gatt: Implement AcquireWrite for server
>> client: Implement AcquireWrite for server
>> gatt: Implement AcquireNotify for server
>> client: Implement AcquireNotify for server
>> gatt: Update signature of AcquireWrite and AcquireNotify
>>
>> client/gatt.c | 319 ++++++++++++++++++++++++++++++++++++++++-------
>> doc/gatt-api.txt | 36 ++++--
>> mesh/gatt.c | 23 +++-
>> src/gatt-client.c | 8 +-
>> src/gatt-database.c | 258 +++++++++++++++++++++++++++++++++++++-
>> src/shared/gatt-db.c | 8 ++
>> src/shared/gatt-db.h | 2 +
>> src/shared/gatt-server.c | 8 ++
>> src/shared/gatt-server.h | 1 +
>> 9 files changed, 598 insertions(+), 65 deletions(-)
>>
>> --
>> 2.13.5
>
> Any other feedback on this? Does it address all the use cases you guys
> have been struggling?
>
>
> --
> Luiz Augusto von Dentz
next prev parent reply other threads:[~2017-09-21 18:15 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-20 7:44 [PATCH v2 00/10] gatt: Add server support for AcquireWrite and AcquireNotify Luiz Augusto von Dentz
2017-09-20 7:44 ` [PATCH v2 01/10] gatt: Remove useless debug Luiz Augusto von Dentz
2017-09-20 7:44 ` [PATCH v2 02/10] client: Rework variables for AcquireWrite/AcquireNotify Luiz Augusto von Dentz
2017-09-20 7:44 ` [PATCH v2 03/10] doc/gatt-api: Add server support for AcquireWrite and AcquireNotify Luiz Augusto von Dentz
2017-09-20 7:44 ` [PATCH v2 04/10] shared/gatt-server: Add bt_gatt_server_get_mtu Luiz Augusto von Dentz
2017-09-20 7:44 ` [PATCH v2 05/10] shared/gatt-db: Add gatt_db_attribute_get_user_data Luiz Augusto von Dentz
2017-09-20 7:44 ` [PATCH v2 06/10] gatt: Implement AcquireWrite for server Luiz Augusto von Dentz
2017-09-29 5:04 ` Yunhan Wang
2017-09-29 6:54 ` Luiz Augusto von Dentz
2017-09-29 7:14 ` Yunhan Wang
2017-09-29 7:22 ` Luiz Augusto von Dentz
2017-09-29 7:50 ` Yunhan Wang
2017-09-29 8:02 ` Luiz Augusto von Dentz
2017-09-30 3:39 ` Yunhan Wang
2017-10-02 11:57 ` Luiz Augusto von Dentz
2017-10-03 0:24 ` Yunhan Wang
2017-09-20 7:44 ` [PATCH v2 07/10] client: " Luiz Augusto von Dentz
2017-09-20 7:44 ` [PATCH v2 08/10] gatt: Implement AcquireNotify " Luiz Augusto von Dentz
2017-09-29 4:47 ` Yunhan Wang
2017-09-29 7:03 ` Luiz Augusto von Dentz
2017-10-03 6:13 ` Yunhan Wang
2017-10-03 7:36 ` Luiz Augusto von Dentz
2017-10-03 18:48 ` Yunhan Wang
2017-09-20 7:44 ` [PATCH v2 09/10] client: " Luiz Augusto von Dentz
2017-09-20 7:44 ` [PATCH v2 10/10] gatt: Update signature of AcquireWrite and AcquireNotify Luiz Augusto von Dentz
2017-09-21 7:49 ` [PATCH v2 00/10] gatt: Add server support for " Luiz Augusto von Dentz
2017-09-21 18:15 ` Yunhan Wang [this message]
2017-09-22 10:58 ` Luiz Augusto von Dentz
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=CALvjcs9cM-2u6423kg3ybxVkrzCyJfNqFfEUM5KJRPo0MhCLPg@mail.gmail.com \
--to=yunhanw@nestlabs.com \
--cc=jonah@sense.com \
--cc=linux-bluetooth@vger.kernel.org \
--cc=luiz.dentz@gmail.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).