Skip to content

Commit 6aa16e0

Browse files
wopu-otnashif
authored andcommitted
Bluetooth: controller: hci: Fix naming for ISO path setup parameters
Fix field names of bt_hci_cp_le_setup_iso_path in le_setup_iso_path to make CIS central/periphal compile and adjust parameter names for ll_setup_iso_path accordingly. Signed-off-by: Wolfgang Puffitsch <wopu@demant.com>
1 parent bc33ef4 commit 6aa16e0

File tree

3 files changed

+13
-11
lines changed

3 files changed

+13
-11
lines changed

subsys/bluetooth/controller/hci/hci.c

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1781,21 +1781,23 @@ static void le_setup_iso_path(struct net_buf *buf, struct net_buf **evt)
17811781
{
17821782
struct bt_hci_cp_le_setup_iso_path *cmd = (void *)buf->data;
17831783
struct bt_hci_rp_le_setup_iso_path *rp;
1784-
uint8_t status;
1785-
uint16_t handle;
1786-
uint16_t company_id;
1787-
uint16_t vendor_id;
17881784
uint32_t controller_delay;
17891785
uint8_t *codec_config;
1786+
uint8_t coding_format;
1787+
uint16_t vs_codec_id;
1788+
uint16_t company_id;
1789+
uint16_t handle;
1790+
uint8_t status;
17901791

1791-
handle = sys_le16_to_cpu(cmd->handle);
1792-
company_id = sys_le16_to_cpu(cmd->company_id);
1793-
vendor_id = sys_le16_to_cpu(cmd->vendor_id);
1792+
handle = sys_le16_to_cpu(cmd->handle);
1793+
coding_format = cmd->codec_id.coding_format;
1794+
company_id = sys_le16_to_cpu(cmd->codec_id.company_id);
1795+
vs_codec_id = sys_le16_to_cpu(cmd->codec_id.vs_codec_id);
17941796
controller_delay = sys_get_le24(cmd->controller_delay);
17951797
codec_config = &cmd->codec_config[0];
17961798

17971799
status = ll_setup_iso_path(handle, cmd->path_dir, cmd->path_id,
1798-
cmd->coding_format, company_id, vendor_id,
1800+
coding_format, company_id, vs_codec_id,
17991801
controller_delay, cmd->codec_config_len,
18001802
codec_config);
18011803

subsys/bluetooth/controller/include/ll.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ uint8_t ll_configure_data_path(uint8_t data_path_dir,
159159
uint8_t *vs_config);
160160
uint8_t ll_setup_iso_path(uint16_t handle, uint8_t path_dir, uint8_t path_id,
161161
uint8_t coding_format, uint16_t company_id,
162-
uint16_t vendor_id, uint32_t controller_delay,
162+
uint16_t vs_codec_id, uint32_t controller_delay,
163163
uint8_t codec_config_len, uint8_t *codec_config);
164164
uint8_t ll_remove_iso_path(uint16_t handle, uint8_t path_dir);
165165
uint8_t ll_iso_receive_test(uint16_t handle, uint8_t payload_type);

subsys/bluetooth/controller/ll_sw/ull_iso.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,15 +27,15 @@ __weak uint8_t ll_configure_data_path(uint8_t data_path_dir,
2727

2828
uint8_t ll_setup_iso_path(uint16_t handle, uint8_t path_dir, uint8_t path_id,
2929
uint8_t coding_format, uint16_t company_id,
30-
uint16_t vendor_id, uint32_t controller_delay,
30+
uint16_t vs_codec_id, uint32_t controller_delay,
3131
uint8_t codec_config_len, uint8_t *codec_config)
3232
{
3333
ARG_UNUSED(handle);
3434
ARG_UNUSED(path_dir);
3535
ARG_UNUSED(path_id);
3636
ARG_UNUSED(coding_format);
3737
ARG_UNUSED(company_id);
38-
ARG_UNUSED(vendor_id);
38+
ARG_UNUSED(vs_codec_id);
3939
ARG_UNUSED(controller_delay);
4040
ARG_UNUSED(codec_config_len);
4141
ARG_UNUSED(codec_config);

0 commit comments

Comments
 (0)