javaos74 / btstack

Automatically exported from code.google.com/p/btstack
0 stars 0 forks source link

Patch for /trunk/src/hci_cmds.c #448

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I observed , HCI Opcode Commands hci_cmd_t 
hci_enhanced_setup_synchronous_connection and hci_cmd_t 
hci_enhanced_accept_synchronous_connection had the same opcode as 0x3d.
For that reason,
Modified the HCI Command, Opcode field from
const hci_cmd_t hci_enhanced_setup_synchronous_connection = {
OPCODE(OGF_LINK_CONTROL, 0x3d), "H4412212222441221222211111111221"
};
as
const hci_cmd_t hci_enhanced_setup_synchronous_connection = {
OPCODE(OGF_LINK_CONTROL, 0x35), "H4412212222441221222211111111221"
};
to get rid of from the duplication of the const hci_cmd_t 
hci_enhanced_accept_synchronous_connection = {
OPCODE(OGF_LINK_CONTROL, 0x3d), "B4412212222441221222211111111221"
};

Original issue reported on code.google.com by svrajas...@gmail.com on 20 Feb 2015 at 6:20

Attachments:

GoogleCodeExporter commented 9 years ago
Hi.. Thanks, correct observation. I did stumble upon that yesterday, too, and 
fixed it in r3207
https://code.google.com/p/btstack/source/detail?r=3207

best,
matthias

Original comment by matthias.ringwald@gmail.com on 20 Feb 2015 at 10:39

GoogleCodeExporter commented 9 years ago
oh. that was another one. I've also corrected the 
hci_enhanced_setup_synchronous_connection in r3209

Original comment by matthias.ringwald@gmail.com on 20 Feb 2015 at 10:42