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. > > Johan > -- > 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 \Lukasz -- 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
