linux-kernel - Re: [Patch-V3 0/6] HID: Support for the Logitech G920 Wheel

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20151120080527.GN19344@mail.corp.redhat.com>
Date:	Fri, 20 Nov 2015 09:05:28 +0100
From:	Benjamin Tissoires <benjamin.tissoires@...hat.com>
To:	Simon Wood <simon@...gewell.org>
Cc:	linux-input@...r.kernel.org, linux-kernel@...r.kernel.org,
	Jiri Kosina <jkosina@...e.cz>, Edwin <Edwin@...ds.nl>,
	Michal Malý <madcatxster@...oid-pointer.net>,
	elias vanderstuyft <elias.vds@...il.com>,
	Dmitry Torokhov <dmitry.torokhov@...il.com>
Subject: Re: [Patch-V3 0/6] HID: Support for the Logitech G920 Wheel

Hi Simon,

On Nov 19 2015 or thereabouts, Simon Wood wrote:
> Patch-V3 tweaked as per Benjamin's and Dmitry's requests.
> 

If you just amended with my comments and Dmitry's, you can keep the
previous Rev-by while resending the series (as a general rule).

The series is still rev-by me.

Cheers,
Benjamin

> This series of patches provide input support for the Logitech G920 gaming wheel.
> 
> This wheel is internally different from the other Logitech wheels; when first 
> connected it is in X-Box mode and can instructed to switch to HID with a 'magic 
> command' (1st patch). Once the wheel reconnects in HID mode it can communicate 
> with the HID++ protocol, but using a 'very long' packet size (2nd patch).
> 
> Basic input operation is possible with adustment of the 'range' (the amount that 
> the wheel turns) controlled via the '/sys' interface, same concept as the G25/G27/etc.
> 
> We also discovered that wheel uses some vendor specific pages, which confuse the 
> HID system resulting in lots of additional axis reported. This is prevented by 
> ignoring these pages (5th patch, thank you Elias).
> 
> The future... as the internals of the wheel are considerably more 'capable' we 
> are working on implementing Force Feedback using the forth-coming KLGD system.
> 
> Simon Wood (6):
>   INPUT: xpad: switch Logitech G920 Wheel into HID mode
>   HID: hid-logitech-hidpp: Add support for very long packets
>   HID: hid-logitech-hidpp: Add basic support for Logitech G920
>   HID: hid-logitech-hidpp: Add range sysfs for Logitech G920
>   HID: Add vendor specific usage pages for Logitech G920
>   HID: hid-logitech-hidpp: G920 remove deadzones
> 
>  drivers/hid/hid-core.c           |   1 +
>  drivers/hid/hid-ids.h            |   1 +
>  drivers/hid/hid-input.c          |   4 +
>  drivers/hid/hid-logitech-hidpp.c | 288 +++++++++++++++++++++++++++++++++++----
>  drivers/input/joystick/xpad.c    |  16 +++
>  include/linux/hid.h              |   2 +
>  6 files changed, 285 insertions(+), 27 deletions(-)
> 
> -- 
> 2.1.4
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/