[BlueZ,v5,4/6] shared/bap: Do stream configuration in bap_bcast_config

Commit Message

Andrei Istodorescu March 1, 2024, 3 p.m. UTC

Stream configuration for BAP broadcast streams should be done inside
bap_bcast_config and not when setting the user data.
---
 src/shared/bap.c | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

Patch

diff --git a/src/shared/bap.c b/src/shared/bap.c
index 638c6f8a4947..50d03456e6cd 100644
--- a/src/shared/bap.c
+++ b/src/shared/bap.c
@@ -1693,11 +1693,8 @@  static unsigned int bap_bcast_config(struct bt_bap_stream *stream,
 				     bt_bap_stream_func_t func, void *user_data)
 {
 	stream->qos = *qos;
-	if (stream->lpac->type == BT_BAP_BCAST_SINK) {
-		if (data)
-			stream_config(stream, data, NULL);
-		stream_set_state(stream, BT_BAP_STREAM_STATE_CONFIG);
-	}
+	stream->lpac->ops->config(stream, stream->cc, &stream->qos,
+			ep_config_cb, stream->lpac->user_data);
 
 	return 1;
 }
@@ -5369,10 +5366,6 @@  bool bt_bap_stream_set_user_data(struct bt_bap_stream *stream, void *user_data)
 
 	stream->user_data = user_data;
 
-	if (bt_bap_stream_get_type(stream) == BT_BAP_STREAM_TYPE_BCAST)
-		stream->lpac->ops->config(stream, stream->cc, &stream->qos,
-					ep_config_cb, stream->lpac->user_data);
-
 	return true;
 }