From: johan.hedberg@gmail.com
To: linux-bluetooth@vger.kernel.org
Subject: [PATCH 4/7] Bluetooth: Fix updating advertising data needlessly
Date: Mon, 14 Oct 2013 13:52:42 +0300 [thread overview]
Message-ID: <1381747965-24599-5-git-send-email-johan.hedberg@gmail.com> (raw)
In-Reply-To: <1381747965-24599-1-git-send-email-johan.hedberg@gmail.com>
From: Johan Hedberg <johan.hedberg@intel.com>
We need to ensure that the advertising data is up-to-date whenever
advertising is enabled, but when disabling advertising we do not need to
worry about it (since it will eventually get fixed as soon as
advertising is enabled again). This patch fixes this in the command
complete callback for set_adv_enable.
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
---
net/bluetooth/hci_event.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c
index aa6fed3..da2bc3d 100644
--- a/net/bluetooth/hci_event.c
+++ b/net/bluetooth/hci_event.c
@@ -924,7 +924,7 @@ static void hci_cc_le_set_adv_enable(struct hci_dev *hdev, struct sk_buff *skb)
clear_bit(HCI_ADVERTISING, &hdev->dev_flags);
}
- if (!test_bit(HCI_INIT, &hdev->flags)) {
+ if (*sent && !test_bit(HCI_INIT, &hdev->flags)) {
struct hci_request req;
hci_req_init(&req, hdev);
--
1.8.3.1
next prev parent reply other threads:[~2013-10-14 10:52 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-14 10:52 [PATCH 0/7] Bluetooth: Hook up connectable setting for LE johan.hedberg
2013-10-14 10:52 ` [PATCH 1/7] Bluetooth: Add connectable argument to enable_advertising() johan.hedberg
2013-10-14 11:29 ` Marcel Holtmann
2013-10-14 11:36 ` Johan Hedberg
2013-10-14 11:43 ` Marcel Holtmann
2013-10-14 10:52 ` [PATCH 2/7] Bluetooth: Reorganize set_connectable HCI command sending johan.hedberg
2013-10-14 10:52 ` [PATCH 3/7] Bluetooth: Move static advertising functions to avoid forward declarations johan.hedberg
2013-10-14 10:52 ` johan.hedberg [this message]
2013-10-14 10:52 ` [PATCH 5/7] Bluetooth: Update Set Connectable to also impact the advertising type johan.hedberg
2013-10-14 10:52 ` [PATCH 6/7] Bluetooth: Move set_bredr_scan() to avoid forward declaration johan.hedberg
2013-10-14 10:52 ` [PATCH 7/7] Bluetooth: Fix updating scan mode in set_bredr() johan.hedberg
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=1381747965-24599-5-git-send-email-johan.hedberg@gmail.com \
--to=johan.hedberg@gmail.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