Re: [PATCH v1 1/2] dt-bindings: net: bluetooth: ti: Add property for enhanced SCO setup erratum

[Krzysztof Kozlowski]

On Mon, Feb 23, 2026 at 01:15:59PM +0100, Stefano Radaelli wrote:
> On Mon, Feb 23, 2026 at 01:03:27PM +0100, Krzysztof Kozlowski wrote:
> > On 23/02/2026 12:52, Stefano Radaelli wrote:
> > > From: Stefano Radaelli <stefano.r@xxxxxxxxxxxxx>
> > > 
> > > Some Bluetooth controller/firmware combinations advertise support for the
> > 
> > Which ones?
> > 
> > 
> > This looks a lot like controller issue, thus deducible from the
> > compatible. And even if particular firmware has problem, how do you see
> > changing static DTS on linux-firmware update?
> > 
> > Best regards,
> > Krzysztof
> 
> Hi Krzysztof,
> 
> thank you for your review!
> 
> We have observed this behaviour specifically on TI WL1831 modules.
> 
> When using the HCI Enhanced Setup Synchronous Connection command,
> SCO setup fails even though the controller reports the command as
> supported. The only working path for this controller is to fall back
> to the legacy HCI Setup Synchronous Connection (0x0028).
> The behaviour matches the scenario described in commit 05abad857277,
> which introduced HCI_QUIRK_BROKEN_ENHANCED_SETUP_SYNC_CONN for that
> reason.
> 
> We are using the latest firmware available from TI for the wl18xx
> family (from the official TI repository), so this is not related to
> an outdated firmware version.
> 
> The reason for proposing a DT property was to keep the workaround
> flexible and avoid forcing the quirk on all TI controllers handled
> by hci_ll without confirmation.

I did not propose to enable it on ALL TI controllers. I asked to deduce
it from given compatible, so to enable it on affected devices.

> 
> However, since this may be considered deducible from the compatible,
> would you prefer enabling
> HCI_QUIRK_BROKEN_ENHANCED_SETUP_SYNC_CONN automatically for
> "ti,wl1831-st" instead of using a DT property?

Yes

Best regards,
Krzysztof