From: "Yoann Congal" <yoann.congal@smile.fr>
To: <shuai.zhang@oss.qualcomm.com>,
<openembedded-core@lists.openembedded.org>
Cc: <cheng.jiang@oss.qualcomm.com>, <wei.deng@oss.qualcomm.com>,
<mengshi.wu@oss.qualcomm.com>, <jinwang.li@oss.qualcomm.com>,
<xiuzhuo.shang@oss.qualcomm.com>,
<chenhong.zhou@oss.qualcomm.com>
Subject: Re: [OE-core] [PATCH v2] [wrynose][backport] bluez5: Abort continuing response on fragmented CT replies
Date: Mon, 09 Nov 2026 11:19:11 +0100 [thread overview]
Message-ID: <DMQLQYYMKVUP.2W0R64KZ4VG6I@smile.fr> (raw)
In-Reply-To: <20261021090017.683390-1-shuai.zhang@oss.qualcomm.com>
On Wed Oct 21, 2026 at 11:00 AM CEST, Shuai Zhang via lists.openembedded.org wrote:
> Send AbortContinuingResponse when a Get Element Attributes reply
> arrives fragmented, as the CT side does not reassemble fragments.
>
> Backport upstream bluez commit 249e2bedf ("avrcp: Abort continuing
> response on fragmented CT replies") to bluez 5.86.
>
> Fixes PTS test AVRCP/CT/RCR/BV-03-C.
>
> Signed-off-by: Shuai Zhang <shuai.zhang@oss.qualcomm.com>
> ---
> Changes for v2:
> - Fix subject prefix to [wrynose][backport]
> - Drop Upstream-Status tag from the commit message
> - Keep upstream Upstream-Status and both Signed-off-by lines in the recipe patch
>
> Link to v1: https://lists.openembedded.org/g/openembedded-core/topic/patchtest_results_for/121863269
> ---
> meta/recipes-connectivity/bluez5/bluez5.inc | 1 +
> ...inuing-response-on-fragmented-CT-rep.patch | 63 +++++++++++++++++++
> 2 files changed, 64 insertions(+)
> create mode 100644 meta/recipes-connectivity/bluez5/bluez5/0001-avrcp-Abort-continuing-response-on-fragmented-CT-rep.patch
>
> diff --git a/meta/recipes-connectivity/bluez5/bluez5.inc b/meta/recipes-connectivity/bluez5/bluez5.inc
> index d857c23..2d0ee56 100644
> --- a/meta/recipes-connectivity/bluez5/bluez5.inc
> +++ b/meta/recipes-connectivity/bluez5/bluez5.inc
> @@ -76,6 +76,7 @@ SRC_URI = "${KERNELORG_MIRROR}/linux/bluetooth/bluez-${PV}.tar.xz \
> file://0001-src-device-Fix-stored-gatt-cache-DB-Hash-value-no.patch \
> file://0001-profile-Set-L2CAP-IMTU-for-OBEX-profile-listeners.patch \
> file://0001-adapter-restrict-delta-0-RSSI-to-proximity-filters.patch \
> + file://0001-avrcp-Abort-continuing-response-on-fragmented-CT-rep.patch \
> "
> S = "${UNPACKDIR}/bluez-${PV}"
>
> diff --git a/meta/recipes-connectivity/bluez5/bluez5/0001-avrcp-Abort-continuing-response-on-fragmented-CT-rep.patch b/meta/recipes-connectivity/bluez5/bluez5/0001-avrcp-Abort-continuing-response-on-fragmented-CT-rep.patch
> new file mode 100644
> index 0000000..9b29ffa
> --- /dev/null
> +++ b/meta/recipes-connectivity/bluez5/bluez5/0001-avrcp-Abort-continuing-response-on-fragmented-CT-rep.patch
> @@ -0,0 +1,63 @@
> +From 41a00c93baa48cbd63b182b853e93a946962cada Mon Sep 17 00:00:00 2001
> +From: Simon Mikuda <simon.mikuda@streamunlimited.com>
> +Date: Tue, 9 Jun 2026 23:26:56 +0200
> +Subject: [PATCH] avrcp: Abort continuing response on fragmented CT replies
> +
> +Send AbortContinuingResponse when a Get Element Attributes reply
> +arrives fragmented, as the CT side does not reassemble fragments.
> +
> +Fixes PTS test AVRCP/CT/RCR/BV-03-C
> +
> +Upstream-Status: Backport [https://github.com/bluez/bluez/commit/249e2bedfacf51fc6c89f43a0adb9a12d761c2b7]
> +Signed-off-by: Simon Mikuda <simon.mikuda@streamunlimited.com>
Simon did not Signed-off the upstream commit. Please do not add someone
else's SoB => Please remove this for v3.
> +Signed-off-by: Shuai Zhang <shuai.zhang@oss.qualcomm.com>
This is fine: you're supposed to SoB the patches you backport.
> +---
> + profiles/audio/avrcp.c | 25 +++++++++++++++++++++++++
> + 1 file changed, 25 insertions(+)
Regards,
--
Yoann Congal
Smile ECS
prev parent reply other threads:[~2026-11-09 10:19 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-21 9:00 [PATCH v2] [wrynose][backport] bluez5: Abort continuing response on fragmented CT replies Shuai Zhang
2026-10-27 7:22 ` Shuai Zhang
2026-10-27 8:44 ` [OE-core] " Yoann Congal
2026-11-09 10:19 ` Yoann Congal [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DMQLQYYMKVUP.2W0R64KZ4VG6I@smile.fr \
--to=yoann.congal@smile.fr \
--cc=cheng.jiang@oss.qualcomm.com \
--cc=chenhong.zhou@oss.qualcomm.com \
--cc=jinwang.li@oss.qualcomm.com \
--cc=mengshi.wu@oss.qualcomm.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=shuai.zhang@oss.qualcomm.com \
--cc=wei.deng@oss.qualcomm.com \
--cc=xiuzhuo.shang@oss.qualcomm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox