Skip to content

Commit c966eac

Browse files
larsgkmmahadevan108
authored andcommitted
Bluetooth: Shell: Fix missing RX QoS param and MSE check
The bis_iso_qos.rx was missing in param sent in bt_iso_big_sync Also, MSE = 0 is valid (fixed error check) Signed-off-by: Lars Knudsen <[email protected]>
1 parent 20e3134 commit c966eac

File tree

1 file changed

+3
-3
lines changed
  • subsys/bluetooth/host/shell

1 file changed

+3
-3
lines changed

subsys/bluetooth/host/shell/iso.c

+3-3
Original file line numberDiff line numberDiff line change
@@ -821,6 +821,7 @@ static int cmd_big_sync(const struct shell *sh, size_t argc, char *argv[])
821821
}
822822

823823
bis_iso_qos.tx = NULL;
824+
bis_iso_qos.rx = &iso_rx_qos;
824825

825826
param.bis_channels = bis_channels;
826827
param.num_bis = BIS_ISO_CHAN_COUNT;
@@ -846,9 +847,8 @@ static int cmd_big_sync(const struct shell *sh, size_t argc, char *argv[])
846847
return -ENOEXEC;
847848
}
848849

849-
if (!IN_RANGE(mse,
850-
BT_ISO_SYNC_MSE_MIN,
851-
BT_ISO_SYNC_MSE_MAX)) {
850+
if (mse != BT_ISO_SYNC_MSE_ANY &&
851+
!IN_RANGE(mse, BT_ISO_SYNC_MSE_MIN, BT_ISO_SYNC_MSE_MAX)) {
852852
shell_error(sh, "Invalid mse %lu", mse);
853853

854854
return -ENOEXEC;

0 commit comments

Comments
 (0)