Path: csiph.com!eternal-september.org!feeder.eternal-september.org!border1.nntp.ams1.giganews.com!border2.nntp.ams1.giganews.com!nntp.giganews.com!news.panservice.it!bofh.it!news.nic.it!robomod From: Benjamin Tissoires Newsgroups: linux.kernel Subject: Re: [Patch-V3 0/6] HID: Support for the Logitech G920 Wheel Date: Fri, 20 Nov 2015 09:10:02 +0100 Message-ID: References: X-Original-To: Simon Wood MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline X-Scanned-By: MIMEDefang 2.68 on 10.5.11.27 Sender: robomod@news.nic.it List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Approved: robomod@news.nic.it Lines: 55 Organization: linux.* mail to news gateway X-Original-Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, Jiri Kosina , Edwin , Michal =?utf-8?B?TWFsw70=?= , elias vanderstuyft , Dmitry Torokhov X-Original-Date: Fri, 20 Nov 2015 09:05:28 +0100 X-Original-Message-ID: <20151120080527.GN19344@mail.corp.redhat.com> X-Original-References: <1447976535-2281-1-git-send-email-simon@mungewell.org> X-Original-Sender: linux-kernel-owner@vger.kernel.org Xref: csiph.com linux.kernel:1273776 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/