Re: [PATCH] hid: microsoft: Add rumble support for Xbox One S controller

On Mon, 2018-08-13 at 07:37 -0700, Andrey Smirnov wrote:

On Fri, Aug 10, 2018 at 4:38 AM Bastien Nocera [off-list ref]
wrote:

quoted

On Thu, 2018-08-09 at 17:17 -0700, Andrey Smirnov wrote:

quoted

Add HID quirk driver for Xbox One S controller over bluetooth.

This driver only adds support for rumble. Standard controller
functionality is exposed by default HID driver.

Did you manage to make the joypad work without hacks in the
Bluetooth
stack[1]?

I was not aware this hack actually existed, but now that I see it I
think it explains why doing

echo 1 > /sys/module/bluetooth/parameters/disable_ertm

was necessary to make the controller connect over Bluetooth on my
machine. It looks like disabling ERTM just happens to have the same
effect as the hack that you mention. I'd like/plan to look into
finding a proper solution to replace that hack, but that'd probably
be
a separate patch.

I gave it a try a couple of months back, but without much success.
L2CAP is unfortunately a bit too low-level to know anything about the
device which requested this. I think that some level of "STREAMING"
support is needed for the Bluetooth audio portion of the device.

Marcel was also interested in looking into the problem, but didn't get
to it. Let me know if you manage to get anything working without the
big hammer of disabling ERTM support for everything.

Cheers