[RESEND PATCH] transport: always show A2DP Delay property as sink if DelayReporting

[RESEND PATCH] transport: always show A2DP Delay property as sink if DelayReporting

[Pauli Virtanen]

The Delay property is currently not shown to sound server unless remote
side has acked delay reporting.  However, when we are A2DP Sink, we
reply to delay report requests with the current value, so the value
logically exists and is active.  This behavior was apparently changed in
commits 05f8bd489fd1 and 5d4efe960fd.

Show Delay value for local A2DP Sink endpoint if it has DelayReporting.

Fixes: https://github.com/bluez/bluez/issues/1541
---
 profiles/audio/media.c     | 5 +++++
 profiles/audio/media.h     | 1 +
 profiles/audio/transport.c | 5 +++++
 3 files changed, 11 insertions(+)

diff --git a/profiles/audio/media.c b/profiles/audio/media.c
index deb321e6c..a7a8575f9 100644
--- a/profiles/audio/media.c
+++ b/profiles/audio/media.c
@@ -3488,6 +3488,11 @@ const char *media_endpoint_get_uuid(struct media_endpoint *endpoint)
 	return endpoint->uuid;
 }
 
+bool media_endpoint_get_delay_reporting(struct media_endpoint *endpoint)
+{
+	return endpoint->delay_reporting;
+}
+
 uint8_t media_endpoint_get_codec(struct media_endpoint *endpoint)
 {
 	return endpoint->codec;
diff --git a/profiles/audio/media.h b/profiles/audio/media.h
index d3954abd6..380951f28 100644
--- a/profiles/audio/media.h
+++ b/profiles/audio/media.h
@@ -19,6 +19,7 @@ void media_unregister(struct btd_adapter *btd_adapter);
 
 struct a2dp_sep *media_endpoint_get_sep(struct media_endpoint *endpoint);
 const char *media_endpoint_get_uuid(struct media_endpoint *endpoint);
+bool media_endpoint_get_delay_reporting(struct media_endpoint *endpoint);
 uint8_t media_endpoint_get_codec(struct media_endpoint *endpoint);
 struct btd_adapter *media_endpoint_get_btd_adapter(
 					struct media_endpoint *endpoint);
diff --git a/profiles/audio/transport.c b/profiles/audio/transport.c
index 08ed699b2..08ca360f4 100644
--- a/profiles/audio/transport.c
+++ b/profiles/audio/transport.c
@@ -1011,8 +1011,13 @@ static gboolean delay_reporting_exists(const GDBusPropertyTable *property,
 							void *data)
 {
 	struct media_transport *transport = data;
+	struct media_endpoint *endpoint = transport->endpoint;
 	struct avdtp_stream *stream;
 
+	/* Local A2DP sink decides itself if it has delay reporting */
+	if (!strcmp(media_endpoint_get_uuid(endpoint), A2DP_SINK_UUID))
+		return media_endpoint_get_delay_reporting(endpoint);
+
 	stream = media_transport_get_stream(transport);
 	if (stream == NULL)
 		return FALSE;
-- 
2.51.1

RE: [RESEND] transport: always show A2DP Delay property as sink if DelayReporting

[bluez.test.bot]

[-- Attachment #1: Type: text/plain, Size: 1262 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=1020034

---Test result---

Test Summary:
CheckPatch                    PENDING   0.29 seconds
GitLint                       PENDING   0.33 seconds
BuildEll                      PASS      20.12 seconds
BluezMake                     PASS      2598.27 seconds
MakeCheck                     PASS      20.38 seconds
MakeDistcheck                 PASS      187.08 seconds
CheckValgrind                 PASS      239.37 seconds
CheckSmatch                   PASS      312.39 seconds
bluezmakeextell               PASS      129.47 seconds
IncrementalBuild              PENDING   0.38 seconds
ScanBuild                     PASS      936.17 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: [RESEND PATCH] transport: always show A2DP Delay property as sink if DelayReporting

[patchwork-bot+bluetooth]

Hello:

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

On Wed,  5 Nov 2025 20:26:52 +0200 you wrote:
> The Delay property is currently not shown to sound server unless remote
> side has acked delay reporting.  However, when we are A2DP Sink, we
> reply to delay report requests with the current value, so the value
> logically exists and is active.  This behavior was apparently changed in
> commits 05f8bd489fd1 and 5d4efe960fd.
> 
> Show Delay value for local A2DP Sink endpoint if it has DelayReporting.
> 
> [...]

Here is the summary with links:
  - [RESEND] transport: always show A2DP Delay property as sink if DelayReporting
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=14f868fd141a

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