Hi Lukasz,
On 03/30/2014 04:00 AM, Lukasz Rymanowski wrote:
HI Andre,
On Fri, Mar 28, 2014 at 5:15 PM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote:
Hi Andre,
On Mon, Mar 24, 2014, Andre Guedes wrote:
This patch introduces commands to add and remove connection parameters.
---
doc/mgmt-api.txt | 84 +++++++++++++++++++++++++++++++++++++++++++++++++++++++-
1 file changed, 83 insertions(+), 1 deletion(-)
All in all this looks pretty good to me. Just one thing I'd fix:
+Add Connection Parameters Command
+====================================
+
+ Command Code: 0x0032
+ Controller Index: <controller id>
+ Command Parameters: Address (6 Octets)
+ Address_Type (1 Octet)
+ Auto_Connect_Option (1 Octet)
+ Min_Connection_Interval (2 Octets)
+ Max_Connection_Interval (2 Octes)
+ Return Parameters:
+
This should have the Address and Address_Type also in the return
parameters for consistency and ability to distinguish the completion of
two simultaneous commands for the different devices.
In addition to what Johan wrote, maybe you should send Command
Complete event on both success and failure.
So you always get the address information in userspace.
Yes, sure.
Thanks,
Andre
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
