diff mbox series

[RFC,BlueZ,v2,01/11] org.bluez.MediaEndpoint: removing BAP streams with ClearConfiguration

Message ID 3c9eb72cf7b16ea0b710d62221436e2777f7360b.1746374514.git.pav@iki.fi
State Superseded
Headers show
Series [RFC,BlueZ,v2,01/11] org.bluez.MediaEndpoint: removing BAP streams with ClearConfiguration | expand

Commit Message

Pauli Virtanen May 4, 2025, 4:01 p.m. UTC
Allow user to remove specific streams by calling
ClearConfiguration(transport_path) on the endpoint.  If the path is the
endpoint path instead, clear all streams associated with the endpoint.
---

Notes:
    Whole patch series:
    
    Pipewire side: https://gitlab.freedesktop.org/pvir/pipewire/-/commits/bap-codec-switch-select
    
    Tested with
    - Sony Linkbuds S: switching between AC 6(ii), 7(ii), 8(ii), 11(ii)
    - Earfun Air Pro 3: switching between AC 6(i), 11(i)
    
    TODO:
    - Enabling/Disabling while Reconfigure/ClearConfiguration is in progress
      should be handled better.
      Stream QoS should be delayed until CIG is no longer busy.

 doc/org.bluez.MediaEndpoint.rst | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

bluez.test.bot@gmail.com May 4, 2025, 5:36 p.m. UTC | #1
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=959452

---Test result---

Test Summary:
CheckPatch                    PENDING   0.26 seconds
GitLint                       PENDING   0.25 seconds
BuildEll                      PASS      20.62 seconds
BluezMake                     PASS      2621.64 seconds
MakeCheck                     PASS      20.06 seconds
MakeDistcheck                 PASS      197.84 seconds
CheckValgrind                 PASS      274.37 seconds
CheckSmatch                   WARNING   303.13 seconds
bluezmakeextell               PASS      127.66 seconds
IncrementalBuild              PENDING   0.30 seconds
ScanBuild                     PASS      898.75 seconds

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

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

##############################
Test: CheckSmatch - WARNING
Desc: Run smatch tool with source
Output:
src/shared/bap.c:317:25: warning: array of flexible structuressrc/shared/bap.c: note: in included file:./src/shared/ascs.h:88:25: warning: array of flexible structuressrc/shared/bap.c:317:25: warning: array of flexible structuressrc/shared/bap.c: note: in included file:./src/shared/ascs.h:88:25: warning: array of flexible structuressrc/shared/bap.c:317:25: warning: array of flexible structuressrc/shared/bap.c: note: in included file:./src/shared/ascs.h:88:25: warning: array of flexible structuressrc/shared/bap.c:317:25: warning: array of flexible structuressrc/shared/bap.c: note: in included file:./src/shared/ascs.h:88:25: warning: array of flexible structuressrc/shared/bap.c:317:25: warning: array of flexible structuressrc/shared/bap.c: note: in included file:./src/shared/ascs.h:88:25: warning: array of flexible structuressrc/shared/bap.c:317:25: warning: array of flexible structuressrc/shared/bap.c: note: in included file:./src/shared/ascs.h:88:25: warning: array of flexible structures
##############################
Test: IncrementalBuild - PENDING
Desc: Incremental build with the patches in the series
Output:



---
Regards,
Linux Bluetooth
patchwork-bot+bluetooth@kernel.org May 6, 2025, 5:20 p.m. UTC | #2
Hello:

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

On Sun,  4 May 2025 19:01:53 +0300 you wrote:
> Allow user to remove specific streams by calling
> ClearConfiguration(transport_path) on the endpoint.  If the path is the
> endpoint path instead, clear all streams associated with the endpoint.
> ---
> 
> Notes:
>     Whole patch series:
> 
> [...]

Here is the summary with links:
  - [RFC,BlueZ,v2,01/11] org.bluez.MediaEndpoint: removing BAP streams with ClearConfiguration
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=63b07d1cb639
  - [RFC,BlueZ,v2,02/11] org.bluez.MediaEndpoint: add client role Reconfigure
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=8d0f178974f8
  - [RFC,BlueZ,v2,03/11] shared/bap: bap_abort_stream_req() should cancel also current req
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=62f0b9b539bc
  - [RFC,BlueZ,v2,04/11] shared/bap: fix ucast client ASE usage
    (no matching commit)
  - [RFC,BlueZ,v2,05/11] bap: discard stream when no longer in use
    (no matching commit)
  - [RFC,BlueZ,v2,06/11] bap: do not try QoS before links are updated & io created
    (no matching commit)
  - [RFC,BlueZ,v2,07/11] bap: add ready callback for setup configuration
    (no matching commit)
  - [RFC,BlueZ,v2,08/11] bap: support removing streams with ClearConfiguration()
    (no matching commit)
  - [RFC,BlueZ,v2,09/11] bap: add callback at the end of ucast client select/config
    (no matching commit)
  - [RFC,BlueZ,v2,10/11] bap: implement Reconfigure()
    (no matching commit)
  - [RFC,BlueZ,v2,11/11] bap: delay recreating IO after ongoing configuration
    (no matching commit)

You are awesome, thank you!
diff mbox series

Patch

diff --git a/doc/org.bluez.MediaEndpoint.rst b/doc/org.bluez.MediaEndpoint.rst
index f2b830ab0..b81106f0b 100644
--- a/doc/org.bluez.MediaEndpoint.rst
+++ b/doc/org.bluez.MediaEndpoint.rst
@@ -109,6 +109,12 @@  void ClearConfiguration(object transport)
 
 	Clear transport configuration.
 
+	**Server role:** [ISO only]
+
+	Close the stream associated with the given transport. If the
+	path given is the path of this endpoint, all its streams are
+	closed.
+
 void Release()
 ``````````````