[PATCH BlueZ] transport: fix A2DP Delay values missing from DBus

[PATCH BlueZ] transport: fix A2DP Delay values missing from DBus

[Pauli Virtanen]

With headsets, a2dp_transport::session == NULL usually until stream
resumes. During this time, delay_reporting_exists() incorrectly returns
FALSE, because streams corresponding to NULL session cannot be found.
As no further "Delay" property updates will usually arrive, the property
remains missing even though delay reports have been received.

Fix by setting a2dp->session when processing a delay report, if missing.

Log (bluetoothctl):
[NEW] Transport /org/bluez/hci1/dev_XX_XX_XX_XX_XX_XX/sep2/fd1
[CHG] Transport /org/bluez/hci1/dev_XX_XX_XX_XX_XX_XX/sep2/fd1 Delay is nil
---
 profiles/audio/transport.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/profiles/audio/transport.c b/profiles/audio/transport.c
index 2b9832a76..ab149bcd7 100644
--- a/profiles/audio/transport.c
+++ b/profiles/audio/transport.c
@@ -2649,6 +2649,9 @@ void media_transport_update_delay(struct media_transport *transport,
 	if (a2dp->delay == delay)
 		return;
 
+	if (a2dp->session == NULL)
+		a2dp->session = a2dp_avdtp_get(transport->device);
+
 	a2dp->delay = delay;
 
 	g_dbus_emit_property_changed(btd_get_dbus_connection(),
-- 
2.50.1

RE: [BlueZ] transport: fix A2DP Delay values missing from DBus

[bluez.test.bot]

[-- Attachment #1: Type: text/plain, Size: 1261 bytes --]

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=992140

---Test result---

Test Summary:
CheckPatch                    PENDING   0.28 seconds
GitLint                       PENDING   0.22 seconds
BuildEll                      PASS      20.23 seconds
BluezMake                     PASS      2654.69 seconds
MakeCheck                     PASS      20.07 seconds
MakeDistcheck                 PASS      183.29 seconds
CheckValgrind                 PASS      233.49 seconds
CheckSmatch                   PASS      303.45 seconds
bluezmakeextell               PASS      126.95 seconds
IncrementalBuild              PENDING   0.29 seconds
ScanBuild                     PASS      902.09 seconds

Details
##############################
Test: CheckPatch - PENDING
Desc: Run checkpatch.pl script
Output:

##############################
Test: GitLint - PENDING
Desc: Run gitlint
Output:

##############################
Test: IncrementalBuild - PENDING
Desc: Incremental build with the patches in the series
Output:



---
Regards,
Linux Bluetooth

Re: [PATCH BlueZ] transport: fix A2DP Delay values missing from DBus

[patchwork-bot+bluetooth]

Hello:

This patch was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>:

On Sat, 16 Aug 2025 11:56:53 +0300 you wrote:
> With headsets, a2dp_transport::session == NULL usually until stream
> resumes. During this time, delay_reporting_exists() incorrectly returns
> FALSE, because streams corresponding to NULL session cannot be found.
> As no further "Delay" property updates will usually arrive, the property
> remains missing even though delay reports have been received.
> 
> Fix by setting a2dp->session when processing a delay report, if missing.
> 
> [...]

Here is the summary with links:
  - [BlueZ] transport: fix A2DP Delay values missing from DBus
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=0bb66d3d1abd

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html