Re: bluetooth issues since kernel 6.4 - not discovering other bt devices - /linux/drivers/bluetooth/btusb.c

From: Bagas Sanjaya <bagasdotme@gmail.com>
To: "Erik Dobák" <erik.dobak@gmail.com>,
	"Linux Stable" <stable@vger.kernel.org>,
	"Linux Bluetooth" <linux-bluetooth@vger.kernel.org>,
	"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
	"Linux ARM" <linux-arm-kernel@lists.infradead.org>,
	"Linux Mediatek" <linux-mediatek@lists.infradead.org>,
	"Linux Regressions" <regressions@lists.linux.dev>
Cc: "Marcel Holtmann" <marcel@holtmann.org>,
	"Johan Hedberg" <johan.hedberg@gmail.com>,
	"Luiz Augusto von Dentz" <luiz.dentz@gmail.com>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	"AngeloGioacchino Del Regno"
	<angelogioacchino.delregno@collabora.com>,
	"Tomasz Moń" <tomasz.mon@nordicsemi.no>
Subject: Re: bluetooth issues since kernel 6.4 - not discovering other bt devices - /linux/drivers/bluetooth/btusb.c
Date: Tue, 3 Oct 2023 06:46:58 +0700	[thread overview]
Message-ID: <ZRtWcgiH5JhD5NU2@debian.me> (raw)
In-Reply-To: <CAH7-e5sb+kT_LRb1_y-c5JaFN0=KrrRT97otUPKzTCgzGsVdrQ@mail.gmail.com>

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

On Sun, Oct 01, 2023 at 07:47:54AM +0200, Erik Dobák wrote:
> Hello!
> 
> I bought a new laptop fujitsu life book and everything is going fine
> on artix just the bt makes trouble:
> 
> /var/log/error.log
> Sep 30 18:43:48 nexus bluetoothd[2266]:
> src/adapter.c:reset_adv_monitors_complete() Failed to reset Adv
> Monitors: Failed (0x03)
> Sep 30 18:43:48 nexus bluetoothd[2266]: Failed to clear UUIDs: Failed (0x03)
> Sep 30 18:43:48 nexus bluetoothd[2266]: Failed to add UUID: Failed (0x03)
> Sep 30 18:43:48 nexus bluetoothd[2266]: Failed to add UUID: Failed (0x03)
> 
> i searched a bit the webs and found a new commit at kernel org that
> does do the trouble:
> https://bugs.archlinux.org/task/78980
> 
> follow the linkeys inside the commits there or read this one:
> 
> ---------------before------------------------------------
> /* interface numbers are hardcoded in the spec */
>         if (intf->cur_altsetting->desc.bInterfaceNumber != 0) {
>                 if (!(id->driver_info & BTUSB_IFNUM_2))
>                         return -ENODEV;
>                 if (intf->cur_altsetting->desc.bInterfaceNumber != 2)
>                         return -ENODEV;
>         }
> -----------after----------------------------------------------------
> if ((id->driver_info & BTUSB_IFNUM_2) &&
>     (intf->cur_altsetting->desc.bInterfaceNumber != 0) &&
>     (intf->cur_altsetting->desc.bInterfaceNumber != 2))
> return -ENODEV;
> --------------------------------------------------------
> 
> the dude just hooked up 3 conditions in a row with && where before it
> was 2 conditions in 1 condition. + the comment was removed.
> 
> 

Try latest mainline first (currently v6.6-rc4). Because you have to
compile your own kernel, please refer to
Documentation/admin-guide/quickly-build-trimmed-linux.rst if you don't know how to do the compilation.

Also, don't top-post when replying; reply inline with appropriate context
instead.

Thanks.

-- 
An old man doll... just what I always wanted! - Clara

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]