From: Johan Hedberg <johan.hedberg@gmail.com>
To: chen.ganir@ti.com
Cc: linux-bluetooth@vger.kernel.org
Subject: Re: [PATCH v2] gatt: Delay D-Bus reply on char discovery
Date: Fri, 21 Sep 2012 16:19:27 +0300 [thread overview]
Message-ID: <20120921131927.GA15946@x220> (raw)
In-Reply-To: <1347353240-1942-1-git-send-email-chen.ganir@ti.com>
Hi Chen,
On Tue, Sep 11, 2012, chen.ganir@ti.com wrote:
> Delay sending the D-Bus reply for the discover_characteristics
> command. The D-Bus reply for characteristics is sent before all
> the relevant characteristic information is gathered. This can
> cause problems, when trying to get characteristic information too
> soon. This patch moves the D-Bus reply to the end of the char
> discovery process. Only after all descriptors are discovered and
> read, the D-Bus reply is sent.
> ---
> attrib/client.c | 138 +++++++++++++++++++++++++++++++++++++------------------
> 1 file changed, 93 insertions(+), 45 deletions(-)
This one needs to be rebased against latest git (I think the
btd_get_dbus_connection patch broke it).
Johan
prev parent reply other threads:[~2012-09-21 13:19 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-11 8:47 [PATCH v2] gatt: Delay D-Bus reply on char discovery chen.ganir
2012-09-11 11:44 ` Anderson Lizardo
2012-09-11 12:22 ` Chen Ganir
2012-09-11 12:27 ` Anderson Lizardo
2012-09-11 12:36 ` Chen Ganir
2012-09-21 13:19 ` Johan Hedberg [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=20120921131927.GA15946@x220 \
--to=johan.hedberg@gmail.com \
--cc=chen.ganir@ti.com \
--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 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).