Hi Pauli, On Mon, May 15, 2023 at 1:30 PM Pauli Virtanen <pav@xxxxxx> wrote: > > Hi Luiz, > > ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti: > > Hi Pauli, > > > > On Sat, May 13, 2023 at 10:41 AM <bluez.test.bot@xxxxxxxxx> wrote: > > > > > > 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=747273 > > > > > > ---Test result--- > > > > > > Test Summary: > > > CheckPatch FAIL 1.14 seconds > > > GitLint PASS 0.52 seconds > > > BuildEll PASS 26.21 seconds > > > BluezMake PASS 745.32 seconds > > > MakeCheck PASS 11.04 seconds > > > MakeDistcheck PASS 151.18 seconds > > > CheckValgrind PASS 243.86 seconds > > > CheckSmatch PASS 326.99 seconds > > > bluezmakeextell PASS 98.83 seconds > > > IncrementalBuild PASS 1262.88 seconds > > > ScanBuild WARNING 964.73 seconds > > > > > > Details > > > ############################## > > > Test: CheckPatch - FAIL > > > Desc: Run checkpatch.pl script > > > Output: > > > [BlueZ,v3,1/2] test-runner: revert udevd and audio support > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled? > > > #49: > > > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88 > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled? > > > #50: > > > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f > > > > > > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked > > > > > > NOTE: For some of the reported defects, checkpatch may be able to > > > mechanically convert to the typical style using --fix or --fix-inplace. > > > > > > /github/workspace/src/src/13240258.patch has style problems, please review. > > > > > > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO > > > > > > NOTE: If any of the errors are false positives, please report > > > them to the maintainer, see CHECKPATCH in MAINTAINERS. > > > > > > > > > ############################## > > > Test: ScanBuild - WARNING > > > Desc: Run Scan Build > > > Output: > > > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value > > > printf("Running command %s\n", cmdname ? cmdname : argv[0]); > > > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > > 1 warning generated. > > > > > > > > > > > > --- > > > Regards, > > > Linux Bluetooth > > > > I tried running on fedora but it looks like something is missing in my setup: > > > > [E][00003.767959] spa.dbus | [ dbus.c: 329 > > impl_connection_get()] Failed to connect to session bus: Unable to > > autolaun1 > > [E][00003.769967] mod.portal | [ module-portal.c: 326 > > pipewire__module_init()] Failed to connect to session bus: > > Input/output error > > I have those two "errors" too, they should be harmless as the xdg- > desktop-portal module is optional. With `test-runner -d -l -A` you > should get the Bluetooth endpoints to appear. > > Removing the mod.portal from the configuration can't be done with drop- > in config files currently, and I didn't want to include a full config > file here as it has some tens of lines of mandatory boilerplate. Ok, looks like it configuring the endpoints: https://gist.github.com/Vudentz/0d8851e719affd9f2dc1f4081ce32fb7 That said I probably need to compile pipewire from source in order to have the LE Audio endpoints registered, btw do you have any instructions on how to build it, do I need to build wireplumber as well? > > -- > Pauli Virtanen -- Luiz Augusto von Dentz