Thread (6 messages) flat view 6 messages, 2 authors, 2021-03-04
STALE2013d REVIEWED: 2 (0M)

2 review trailers.

[Bluez PATCH 2/3] advertising: Create and use scannable adv param flag

From: Daniel Winkler <hidden>
Date: 2021-03-04 00:34:30
Subsystem: library code, the rest · Maintainers: Andrew Morton, Linus Torvalds

In order for the advertising parameters hci request to indicate that an
advertising set uses a scannable PDU, we pass a scannable flag along
with the initial parameters MGMT request.

Without this patch, a broadcast advertisement with a scan response will
either be rejected by the controller, or will ignore the requested scan
response. The patch is tested by performing the above and confirming
that the scan response is retrievable from a peer as expected.

Reviewed-by: Alain Michaud <redacted>
Reviewed-by: Sonny Sasaka <redacted>

---

 lib/mgmt.h        | 1 +
 src/advertising.c | 4 ++++
 2 files changed, 5 insertions(+)
diff --git a/lib/mgmt.h b/lib/mgmt.h
index 76a03c9c2..7b1b9ab54 100644
--- a/lib/mgmt.h
+++ b/lib/mgmt.h
@@ -507,6 +507,7 @@ struct mgmt_rp_add_advertising {
 #define MGMT_ADV_PARAM_TIMEOUT		(1 << 13)
 #define MGMT_ADV_PARAM_INTERVALS	(1 << 14)
 #define MGMT_ADV_PARAM_TX_POWER		(1 << 15)
+#define MGMT_ADV_PARAM_SCAN_RSP		(1 << 16)
 
 #define MGMT_OP_REMOVE_ADVERTISING	0x003F
 struct mgmt_cp_remove_advertising {
diff --git a/src/advertising.c b/src/advertising.c
index f3dc357a1..38cef565f 100644
--- a/src/advertising.c
+++ b/src/advertising.c
@@ -945,6 +945,10 @@ static int refresh_extended_adv(struct btd_adv_client *client,
 		return -EINVAL;
 	}
 
+	/* Indicate that this instance will be configured as scannable */
+	if (client->scan_rsp_len)
+		flags |= MGMT_ADV_PARAM_SCAN_RSP;
+
 	cp.flags = htobl(flags);
 
 	mgmt_ret = mgmt_send(client->manager->mgmt, MGMT_OP_ADD_EXT_ADV_PARAMS,
-- 
2.30.1.766.gb4fecdf3b7-goog
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help