2024-06-15 13:14:16

by Pauli Virtanen

Subject: [PATCH BlueZ] shared/bap: make BT_BAP_* direction defines valid bitmasks

The directions appear to be intended as bitmasks, as
bt_bap_stream_io_dir() will bitwise or linked stream directions.

Fix the defines to be separate bits.

Fixes confusion due to BT_BAP_BCAST_SOURCE == BT_BAP_SINK|BT_BAP_SOURCE,
which causes e.g. unicast transports to be in PENDING state after QoS
although this does not make sense for BAP unicast Client.
---
src/shared/bap-defs.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/shared/bap-defs.h b/src/shared/bap-defs.h
index 2467e816f..75e6950ea 100644
--- a/src/shared/bap-defs.h
+++ b/src/shared/bap-defs.h
@@ -17,8 +17,8 @@

#define BT_BAP_SINK 0x01
#define BT_BAP_SOURCE 0x02
-#define BT_BAP_BCAST_SOURCE 0x03
-#define BT_BAP_BCAST_SINK 0x04
+#define BT_BAP_BCAST_SOURCE 0x04
+#define BT_BAP_BCAST_SINK 0x08

#define BT_BAP_STREAM_TYPE_UCAST 0x01
#define BT_BAP_STREAM_TYPE_BCAST 0x02
--
2.45.2

2024-06-15 15:04:32

by bluez.test.bot

Subject: RE: [BlueZ] shared/bap: make BT_BAP_* direction defines valid bitmasks

This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=862280

---Test result---

Test Summary:
CheckPatch PASS 0.43 seconds
GitLint PASS 0.31 seconds
BuildEll PASS 24.74 seconds
BluezMake PASS 1719.34 seconds
MakeCheck PASS 13.74 seconds
MakeDistcheck PASS 180.21 seconds
CheckValgrind PASS 253.79 seconds
CheckSmatch PASS 357.19 seconds
bluezmakeextell PASS 121.15 seconds
IncrementalBuild PASS 1470.74 seconds
ScanBuild PASS 1034.61 seconds



---
Regards,
Linux Bluetooth