Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <CACk7upyDgd8kCY+8j+RMJz35GXrF=bwRzDNH76Oh8q4nB8i0KQ@mail.gmail.com>
 <126c5bce-6496-4744-9101-6a61f0fff106@app.fastmail.com>
In-Reply-To: <126c5bce-6496-4744-9101-6a61f0fff106@app.fastmail.com>
From:   Raul Cheleguini <raul.cheleguini@gmail.com>
Date:   Thu, 13 Apr 2023 10:54:47 -0300
Message-ID: <CACk7upyYLmiS9GMJVatjDz52Z4MEPRVAoHi_tSchHWjQfqo2sA@mail.gmail.com>
Subject: Re: unexpected SMP command errors with controller ATS2851
To:     Christopher Snowhill <chris@kode54.net>
Cc:     linux-bluetooth@vger.kernel.org,
        Marcel Holtmann <marcel@holtmann.org>, johan.hedberg@gmail.com,
        luiz.dentz@gmail.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Fri, Mar 31, 2023 at 2:50=E2=80=AFAM Christopher Snowhill <chris@kode54.=
net> wrote:
>
> Would some pcapng logs be useful? It appears this device flags some comma=
nds as supported, while indicating their entire feature sets are not suppor=
ted in the local supported features bitfields. Or that's as near as I can g=
uess.
>
> First is captured from a generic driver that works, the latter is capture=
d from Linux 6.2.9. Included in the working capture is the beginning of an =
exchange with my Sony headphones. The latter stops logging after the first =
failed command.
>
> https://f.losno.co/bluetooth_ats2851.pcapng.zst
> https://f.losno.co/bluetooth_ats2851_broken.pcapng.zst

Hi Christopher,

Thank you for sharing the pcap files. Based on your comments from another
thread regarding this adapter, I suggest that you apply the following patch=
es:

[PATCH] Bluetooth: Improve support for Actions Semi ATS2851 based devices
https://marc.info/?l=3Dlinux-bluetooth&m=3D167846197021176&w=3D2

[PATCH] Bluetooth: Add new quirk for broken set random RPA timeout for ATS.=
..
https://marc.info/?l=3Dlinux-bluetooth&m=3D167957918920723&w=3D2

[PATCH] Bluetooth: Add new quirk for broken extended create connection for.=
..
https://marc.info/?l=3Dlinux-bluetooth&m=3D167957998621276&w=3D2

You would end up in the situation mentioned in this thread, and you could
resume your tests from there. I haven't had the time to experiment with SMP=
 yet,
so any assistance would be greatly appreciated.