Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-2686-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: Silviu Florian Barbulescu <silviu.barbulescu@nxp.com>
To: linux-bluetooth@vger.kernel.org
Cc: mihai-octavian.urzica@nxp.com,
	silviu.barbulescu@nxp.com,
	vlad.pruteanu@nxp.com,
	andrei.istodorescu@nxp.com,
	luiz.dentz@gmail.com,
	iulia.tanasescu@nxp.com
Subject: [PATCH BlueZ 1/1] player: Adjust SDU size based on the number of locations
Date: Thu, 21 Mar 2024 18:12:55 +0200
Message-Id: <20240321161255.24892-2-silviu.barbulescu@nxp.com>
In-Reply-To: <20240321161255.24892-1-silviu.barbulescu@nxp.com>
References: <20240321161255.24892-1-silviu.barbulescu@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?zxREh6WJOwmjlJYYHdJovfL8zHh/NI8V0W9tFIlbQ/j8MpvYqT+tJadTyZBp?=
 =?us-ascii?Q?aYmwmVj7DFwLlRHNC3gVx+TXmQzPsiLAApnW+ifRzWmhFpLCSs0B8PNaA9Jk?=
 =?us-ascii?Q?TVUJ31+YGW5WEbQ0y4ZONc81Uo9CsW8CuDfm34muDCy8SNH82TwQV3J49hOr?=
 =?us-ascii?Q?W3k2/onvTOMLQZLdWK9g0Pmr0SG5gD0mRLJVWtJN5lvMQwxIXuALlpyL4KS8?=
 =?us-ascii?Q?x8eeHpvBDddcI5UPJLb+y3br78Apu7wxqbEf+OYUmDJGWwDnW6sSUtK4l6pa?=
 =?us-ascii?Q?P8BkkU+a1i94ClcASoD46lkPMQaJCF1blYAyb2eAZkpaCsILNHOTfxQ6yo9X?=
 =?us-ascii?Q?f6mIzYQGRIlpPhk+TMeUWng2HoOZSSDlrBGpBJQiyT8gwoKLm3T6DZdXJrnB?=
 =?us-ascii?Q?ySNxw3sSSlCmhotdvv37h2PkaT5wVMmH4nXFlYARuEv+Taqr09aXSVtB6vND?=
 =?us-ascii?Q?dZaWl9UMZS+citP7Ai1aq2j0FlJlDW0Am8j9kTJbNkSbcepL64ZVsvs+vzRi?=
 =?us-ascii?Q?avUlwJVmtgjnL3YwalLUGK5SbDfDhw7RtyRzo6OLWPwV59nLnMBpQJowbWzR?=
 =?us-ascii?Q?YLMYURrw39UJCMtTtiQMcKGCCT0J+ofVUGzrlKK4u2Xz/y096p0HJR2A4VDE?=
 =?us-ascii?Q?0vW4WJRTOIIFD3xGWQnshYldJ6/2Zbh6mgAsVUd2x5QrbJubSKVkkQngYe64?=
 =?us-ascii?Q?XwK/+EIb4PBapcWwU5bW0tYABPc9CV5ylZ4kViS+bhmWNivj8n91Iaw5M+6u?=
 =?us-ascii?Q?15vqvIZ0dGQ2mf3J25KauHDM+N79AbVFb27v1A1uwzN85LHSMoHH0uo073zN?=
 =?us-ascii?Q?YrnHp8SJQ2dMr3zYqDGzI/cTFx54k9RZLOJXtVcX7ZqqVyiXO9hpYbsfHG4R?=
 =?us-ascii?Q?lmvfew4f89Un6RR/OpUay+xA6R9kbV0fvygHCR1VMQkrFNlAGS/yIhT1dBE5?=
 =?us-ascii?Q?sMr/hMA+mRb8T6FIGh2+UwgNnyzeS7l/k99PcZLpW27aAZdDsticRxdEXfQr?=
 =?us-ascii?Q?wQE/psE16FnvafrGpdRZmxlCpFr5bOWeTCUutlawmb6P7U/i36awCz4H33ju?=
 =?us-ascii?Q?UHq++Bn1Sx9LcxMRdOssKgnlRS3eP9pDuI06w3PefB00O6HDHJZdT8qOAHur?=
 =?us-ascii?Q?6wQZi4pjP90tac4eAQjwRbCf5PkaJHtaeFphxt+a8xAb4MyX2k+TTQOzeTPq?=
 =?us-ascii?Q?Al5z21xgMMeQY+t+0bE88PWLDGR1T6SlKJ3bbtFYDewRYMea1CSIVOArnetJ?=
 =?us-ascii?Q?gwfWQF8RVygzzVU8Yb3vSZyDAUDJC/07OAjY/ZX/lOfOyAwsVbyZGdJudcP/?=
 =?us-ascii?Q?FbZ1Fq2sKBSEnSQl7JmZrPwmxsLLWUmYu/bF5j5YrHW1qFYjMjUrtyHN/SSV?=
 =?us-ascii?Q?/8qskmN0J88vxbQJgu3iQnOENArf7WUCgdjdkNJDJNp7HjIYJIBkAyc8dHyC?=
 =?us-ascii?Q?O722LnnoEJ+AEM3YRUtGxBPFxaBimMThp4gXCyS9Rtewsr0QtVQJ5TC7nTP5?=
 =?us-ascii?Q?x/N5v0SmSqynfdDG6VzD5QKACiP9YBOqPgi98YtEyH9EpbaX2nUc9oP4GIDJ?=
 =?us-ascii?Q?BBSsU0tw9HrpVJkyrz6xI4P7ie3XB0GxLMgMI65sFQSPRVG06zrBl+bgj7ji?=
 =?us-ascii?Q?OA=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3992bb57-64d3-4e1a-b26a-08dc49c1c82f
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB10073.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Mar 2024 16:13:01.7052
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 4Iycvi7+D49Q96UjqweCmww0xZjZORR3j2zOGtfofLWuY3cFlPkd73hrmHcXmiFrfPZ0ATnkAEFVnueOEdMUmLZngbn5b59qz6AFKfK8imc=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB9956

Adjust the SDU size based on the number of locations/channels
 that is being requested.

---
 client/player.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/client/player.c b/client/player.c
index ab33bfc46..7856be3b5 100644
--- a/client/player.c
+++ b/client/player.c
@@ -3574,6 +3574,7 @@ static void config_endpoint_channel_location(const char *input, void *user_data)
 	struct endpoint_config *cfg = user_data;
 	char *endptr = NULL;
 	uint32_t location;
+	uint8_t channels = 1;
 
 	if (!strcasecmp(input, "n") || !strcasecmp(input, "no"))
 		goto add_meta;
@@ -3590,6 +3591,13 @@ static void config_endpoint_channel_location(const char *input, void *user_data)
 	iov_append_ltv(&cfg->caps, LC3_CONFIG_CHAN_ALLOC_LEN,
 			LC3_CONFIG_CHAN_ALLOC, &location);
 
+	/* Adjust the SDU size based on the number of
+	 * locations/channels that is being requested.
+	 */
+	channels = __builtin_popcount(location);
+	if (channels > 1)
+		cfg->qos.bcast.io_qos.sdu *= channels;
+
 add_meta:
 	/* Add metadata */
 	bt_shell_prompt_input(cfg->ep->path, "Enter Metadata (value/no):",
-- 
2.39.2