From c966eac722e351f4bb6acb5084b827c751b44235 Mon Sep 17 00:00:00 2001 From: Lars Knudsen Date: Fri, 25 Oct 2024 13:42:17 +0200 Subject: [PATCH] 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 --- subsys/bluetooth/host/shell/iso.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/subsys/bluetooth/host/shell/iso.c b/subsys/bluetooth/host/shell/iso.c index b42340452a2..7c30b09bce9 100644 --- a/subsys/bluetooth/host/shell/iso.c +++ b/subsys/bluetooth/host/shell/iso.c @@ -821,6 +821,7 @@ static int cmd_big_sync(const struct shell *sh, size_t argc, char *argv[]) } bis_iso_qos.tx = NULL; + bis_iso_qos.rx = &iso_rx_qos; param.bis_channels = bis_channels; 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[]) return -ENOEXEC; } - if (!IN_RANGE(mse, - BT_ISO_SYNC_MSE_MIN, - BT_ISO_SYNC_MSE_MAX)) { + if (mse != BT_ISO_SYNC_MSE_ANY && + !IN_RANGE(mse, BT_ISO_SYNC_MSE_MIN, BT_ISO_SYNC_MSE_MAX)) { shell_error(sh, "Invalid mse %lu", mse); return -ENOEXEC;