Re: [Patch-V2 1/6] INPUT: xpad: Add minimal support for Logitech G920 Wheel

Hi Simon,

On Mon, Jan 04, 2016 at 01:05:35PM -0700, Simon Wood wrote:

On Mon, 4 Jan 2016 02:55:24 -0700, Benjamin Tissoires wrote:
your next pull request for

quoted

Linus. We might want to drop it before it hits Linus' tree.

We can still keep the HID work in place even if the device is not
switched into the HID protocol at plug.

Simon, do you mind looking into Dmitry's suggestion of having a clean,
small usb device which loads itself when the G920 is plugged in and
switches it immediately into the HID mode?

Hi,

As noted 'xpad.c' sends the magic bytes to switch the G920 wheel into
HID mode, the wheel then detaches and reconnects as a HID+ device
handled by 'logitech-hidpp.c'.

I'd like to point that up to this point the wheel _is_ a Xbox One
control device, that speaks some undocumented protocol from Microsoft.
To my mind xpad is the correct place to send the magic bytes.

OTOH one can argue if it does not speak protocol that xpad understands
then it does not belong in xpad.

Currently xpad.ko "weighs" at 44K (in my configuration), this is steep
price for sending 1 packet to the device. I am certain that adding a
skeleton usb driver into hid module will be much cheaper.

It is possible that at some future time that the Xpad devs will
understand the protocol enough sufficiently to do useful stuff with
the wheel. In an earlier version of the patch I had a param to disable
the 'switch to HID', and that might be resurrected.

This is way down the road. We may even end up with a brand new driver
for it, not xpad. And if/when that happens we'll be able to drop hid
potion.

Thanks.

-- 
Dmitry