Re: [PATCH v13 10/10] bluetooth: policy: Treat bi-directional A2DP profiles as suitable for VOIP

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



That's Why I proposed to set the config switch to "legacy max negotiated  bitpool" (current code) by default, and let the user decide if he/she wants to activate the "XQ max negociated bitpool" (my patch). 

This way it's less likely to cause regressions than Pali's "forced bitpool" XQ modes : by default PA behaves like current version.

And it doesn't rely on future Bluez features.

Jp

14:29, 18 octobre 2019, "Tanu Kaskinen" <tanuk@xxxxxx>:

On Thu, 2019-10-17 at 15:34 +0200, Hyperion wrote:

 Regression would mean that some devices can't connect anymore : this
 won't happen if a workaround is provided, and this workaround won't
 be used often.

 Most (99% ?) of the devices will work correctly with my patch (many
 of them in XQ mode, and some in legacy mode because they will fall
 back to legacy bitpool during negociation)

 The remaining (1% ?) : will need a simple boolean swicth in one of
 the PA config files to restrict negociation to legacy bitpool (a
 module option ? or daemon.conf ?).

 I think it's really "simple", efficient, and not dependent of any
 upcoming Bluez feature.

 "The complex solution is always the best until one find a simpler one"


I don't know the number of users who use bluetooth headsets with
PulseAudio, but even just 1% regression rate can mean quite a few
unhappy users. When your headset suddenly stops working, it's not
trivial to figure out that you may need to pass a special argument to
module-bluetooth-discover in order to make it work again.

It would be better to have a module argument to enable the XQ settings.

--
Tanu


https://www.patreon.com/tanuk
https://liberapay.com/tanuk



--
Sent from Yandex.Mail for mobile
_______________________________________________
pulseaudio-discuss mailing list
pulseaudio-discuss@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux