2011-06-17 09:44:13

by Antti Julku

[permalink] [raw]
Subject: [PATCH 1/2] Implement fast connectable mode for mgmt interface

Management interface implementation for fast connectable mode.
---
doc/mgmt-api.txt | 8 ++++++++
lib/mgmt.h | 5 +++++
plugins/mgmtops.c | 24 +++++++++++++++++++++---
3 files changed, 34 insertions(+), 3 deletions(-)

diff --git a/doc/mgmt-api.txt b/doc/mgmt-api.txt
index d89467c..8bf5e52 100644
--- a/doc/mgmt-api.txt
+++ b/doc/mgmt-api.txt
@@ -335,6 +335,14 @@ Unblock Device Command
Command Parameters: Address (6 Octets)
Return Parameters: Status (1 octet)

+Set Fast Connectable Command
+============================
+
+ Command Code: 0x0001F
+ Controller Index: <controller id>
+ Command Parameters: Enable (1 Octet)
+ Return Parameters: Status (1 octet)
+
Read Tracing Buffer Size Command
================================

diff --git a/lib/mgmt.h b/lib/mgmt.h
index f22434e..95003a2 100644
--- a/lib/mgmt.h
+++ b/lib/mgmt.h
@@ -215,6 +215,11 @@ struct mgmt_cp_unblock_device {
bdaddr_t bdaddr;
} __packed;

+#define MGMT_OP_SET_FAST_CONNECTABLE 0x001F
+struct mgmt_cp_set_fast_connectable {
+ uint8_t enable;
+} __packed;
+
#define MGMT_EV_CMD_COMPLETE 0x0001
struct mgmt_ev_cmd_complete {
uint16_t opcode;
diff --git a/plugins/mgmtops.c b/plugins/mgmtops.c
index d6226c4..b98c995 100644
--- a/plugins/mgmtops.c
+++ b/plugins/mgmtops.c
@@ -1207,6 +1207,9 @@ static void mgmt_cmd_complete(int sk, uint16_t index, void *buf, size_t len)
case MGMT_OP_UNBLOCK_DEVICE:
DBG("unblock_device complete");
break;
+ case MGMT_OP_SET_FAST_CONNECTABLE:
+ DBG("set_fast_connectable complete");
+ break;
default:
error("Unknown command complete for opcode %u", opcode);
break;
@@ -1661,10 +1664,25 @@ static int mgmt_cancel_resolve_name(int index, bdaddr_t *bdaddr)
return -ENOSYS;
}

-static int mgmt_fast_connectable(int index, gboolean enable)
+static int mgmt_set_fast_connectable(int index, gboolean enable)
{
+ char buf[MGMT_HDR_SIZE + sizeof(struct mgmt_cp_set_fast_connectable)];
+ struct mgmt_hdr *hdr = (void *) buf;
+ struct mgmt_cp_set_fast_connectable *cp = (void *) &buf[sizeof(*hdr)];
+
DBG("index %d enable %d", index, enable);
- return -ENOSYS;
+
+ memset(buf, 0, sizeof(buf));
+ hdr->opcode = htobs(MGMT_OP_SET_FAST_CONNECTABLE);
+ hdr->len = htobs(sizeof(*cp));
+ hdr->index = htobs(index);
+
+ cp->enable = enable;
+
+ if (write(mgmt_sock, buf, sizeof(buf)) < 0)
+ return -errno;
+
+ return 0;
}

static int mgmt_read_clock(int index, bdaddr_t *bdaddr, int which, int timeout,
@@ -2056,7 +2074,7 @@ static struct btd_adapter_ops mgmt_ops = {
.cancel_resolve_name = mgmt_cancel_resolve_name,
.set_name = mgmt_set_name,
.set_dev_class = mgmt_set_dev_class,
- .set_fast_connectable = mgmt_fast_connectable,
+ .set_fast_connectable = mgmt_set_fast_connectable,
.read_clock = mgmt_read_clock,
.read_bdaddr = mgmt_read_bdaddr,
.block_device = mgmt_block_device,
--
1.7.2.5



2011-06-17 09:44:14

by Antti Julku

[permalink] [raw]
Subject: [PATCH 2/2] Rename fast_connectable function in hciops

Rename fast_connectable function in hciops to set_fast_connectable
to make it consistent with rest of the file and with mgmt interface
counterpart.
---
plugins/hciops.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/plugins/hciops.c b/plugins/hciops.c
index 6ce0e27..db4b9a4 100644
--- a/plugins/hciops.c
+++ b/plugins/hciops.c
@@ -3107,7 +3107,7 @@ static int hciops_stop_discovery(int index)
}
}

-static int hciops_fast_connectable(int index, gboolean enable)
+static int hciops_set_fast_connectable(int index, gboolean enable)
{
struct dev_info *dev = &devs[index];
write_page_activity_cp cp;
@@ -3662,7 +3662,7 @@ static struct btd_adapter_ops hci_ops = {
.cancel_resolve_name = hciops_cancel_resolve_name,
.set_name = hciops_set_name,
.set_dev_class = hciops_set_dev_class,
- .set_fast_connectable = hciops_fast_connectable,
+ .set_fast_connectable = hciops_set_fast_connectable,
.read_clock = hciops_read_clock,
.read_bdaddr = hciops_read_bdaddr,
.block_device = hciops_block_device,
--
1.7.2.5


2011-06-16 18:57:40

by Johan Hedberg

[permalink] [raw]
Subject: Re: [PATCH 1/2] Implement fast connectable mode for mgmt interface

Hi Antti,

On Thu, Jun 16, 2011, Antti Julku wrote:
> Management interface implementation for fast connectable mode.
> ---
> doc/mgmt-api.txt | 8 ++++++++
> lib/mgmt.h | 5 +++++
> plugins/mgmtops.c | 24 +++++++++++++++++++++---
> 3 files changed, 34 insertions(+), 3 deletions(-)

Now that your blacklist patch has been applied this one doesn't apply
cleanly anymore, so you'll have to rebase against the upstream tree.

Johan

2011-09-27 09:32:47

by Johan Hedberg

[permalink] [raw]
Subject: Re: [PATCH 1/2] Implement fast connectable mode for mgmt interface

Hi Antti,

On Fri, Jun 17, 2011, Antti Julku wrote:
> Management interface implementation for fast connectable mode.
> ---
> doc/mgmt-api.txt | 8 ++++++++
> lib/mgmt.h | 5 +++++
> plugins/mgmtops.c | 24 +++++++++++++++++++++---
> 3 files changed, 34 insertions(+), 3 deletions(-)

Both patches applied. Thanks.

Johan