From: Benjamin Tissoires <benjamin.tissoires@redhat.com>
To: Simon Wood <simon@mungewell.org>
Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org,
"Jiri Kosina" <jkosina@suse.cz>, Edwin <Edwin@Velds.nl>,
"Michal Malý" <madcatxster@devoid-pointer.net>,
"elias vanderstuyft" <elias.vds@gmail.com>,
"Dmitry Torokhov" <dmitry.torokhov@gmail.com>
Subject: Re: [Patch-V3 0/6] HID: Support for the Logitech G920 Wheel
Date: Fri, 20 Nov 2015 09:05:28 +0100 [thread overview]
Message-ID: <20151120080527.GN19344@mail.corp.redhat.com> (raw)
In-Reply-To: <1447976535-2281-1-git-send-email-simon@mungewell.org>
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
>
next prev parent reply other threads:[~2015-11-20 8:05 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-19 23:42 [Patch-V3 0/6] HID: Support for the Logitech G920 Wheel Simon Wood
2015-11-19 23:42 ` [Patch-V3 1/6] INPUT: xpad: switch Logitech G920 Wheel into HID mode Simon Wood
2015-11-19 23:42 ` [Patch-V3 2/6] HID: hid-logitech-hidpp: Add support for very long packets Simon Wood
2015-11-19 23:42 ` [Patch-V3 3/6] HID: hid-logitech-hidpp: Add basic support for Logitech G920 Simon Wood
2015-11-19 23:42 ` [Patch-V3 4/6] HID: hid-logitech-hidpp: Add range sysfs " Simon Wood
2015-11-19 23:42 ` [Patch-V3 5/6] HID: Add vendor specific usage pages " Simon Wood
2015-11-19 23:42 ` [Patch-V3 6/6] HID: hid-logitech-hidpp: G920 remove deadzones Simon Wood
2015-11-20 8:05 ` Benjamin Tissoires [this message]
2015-11-20 9:32 ` [Patch-V3 0/6] HID: Support for the Logitech G920 Wheel Jiri Kosina
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151120080527.GN19344@mail.corp.redhat.com \
--to=benjamin.tissoires@redhat.com \
--cc=Edwin@Velds.nl \
--cc=dmitry.torokhov@gmail.com \
--cc=elias.vds@gmail.com \
--cc=jkosina@suse.cz \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=madcatxster@devoid-pointer.net \
--cc=simon@mungewell.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).