linux-kernel - Re: [PATCH] Input: xpad - Add Mad Catz FightStick TE 2 VID/PID

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: <CAKvUva95HCU3qpsnHhpG5cQyim1xKrKhuVTcbJSzRo1XMNXhKg@mail.gmail.com>
Date:	Sun, 3 Apr 2016 19:44:23 +0200
From:	Silvan Jegen <s.jegen@...il.com>
To:	Dmitry Torokhov <dmitry.torokhov@...il.com>,
	Pavel Rojtberg <rojtberg@...il.com>,
	linux-input@...r.kernel.org,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	"Pierre-Loup A. Griffais" <githubpublic@...gman.net>
Cc:	Silvan Jegen <s.jegen@...il.com>,
	lkml <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] Input: xpad - Add Mad Catz FightStick TE 2 VID/PID

Hi

Ping!

Please let me know if you want me to change anything to get this change in.


Cheers,

Silvan

On Mon, Mar 14, 2016 at 8:07 PM, Silvan Jegen <s.jegen@...il.com> wrote:
> This adds the VID/PID combination for the Xbox One version of the Mad
> Catz FightStick TE 2.
>
> The functionality that this provides is about on par with what the
> Windows drivers for the stick manage to deliver.
>
> What works:
> - Digital stick
> - 6 main buttons
> - Xbox button
> - The two buttons on the back
> - The locking buttons (preventing accidental Xbox button press)
>
> What doesn't work:
> - Two of the main buttons (don't work on Windows either)
> - The "Haptic" button setting does not have an effect (not sure if it
>   works on Windows)
>
> I added the MAP_TRIGGERS_TO_BUTTONS option but in my (limited) testing
> there was no practical difference with or without. The FightStick does
> not have triggers though so adding it makes sense.
>
> Signed-off-by: Silvan Jegen <s.jegen@...il.com>
> ---
>  drivers/input/joystick/xpad.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/input/joystick/xpad.c b/drivers/input/joystick/xpad.c
> index e8a84d1..d3d7f7c 100644
> --- a/drivers/input/joystick/xpad.c
> +++ b/drivers/input/joystick/xpad.c
> @@ -149,6 +149,7 @@ static const struct xpad_device {
>         { 0x0738, 0x4556, "Mad Catz Lynx Wireless Controller", 0, XTYPE_XBOX },
>         { 0x0738, 0x4716, "Mad Catz Wired Xbox 360 Controller", 0, XTYPE_XBOX360 },
>         { 0x0738, 0x4718, "Mad Catz Street Fighter IV FightStick SE", 0, XTYPE_XBOX360 },
> +       { 0x0738, 0x4a01, "Mad Catz FightStick TE 2", MAP_TRIGGERS_TO_BUTTONS, XTYPE_XBOXONE },
>         { 0x0738, 0x4726, "Mad Catz Xbox 360 Controller", 0, XTYPE_XBOX360 },
>         { 0x0738, 0x4728, "Mad Catz Street Fighter IV FightPad", MAP_TRIGGERS_TO_BUTTONS, XTYPE_XBOX360 },
>         { 0x0738, 0x4738, "Mad Catz Wired Xbox 360 Controller (SFIV)", MAP_TRIGGERS_TO_BUTTONS, XTYPE_XBOX360 },
> @@ -304,6 +305,7 @@ static struct usb_device_id xpad_table[] = {
>         XPAD_XBOX360_VENDOR(0x046d),            /* Logitech X-Box 360 style controllers */
>         XPAD_XBOX360_VENDOR(0x0738),            /* Mad Catz X-Box 360 controllers */
>         { USB_DEVICE(0x0738, 0x4540) },         /* Mad Catz Beat Pad */
> +       XPAD_XBOXONE_VENDOR(0x0738),            /* Mad Catz FightStick TE 2 */
>         XPAD_XBOX360_VENDOR(0x0e6f),            /* 0x0e6f X-Box 360 controllers */
>         XPAD_XBOX360_VENDOR(0x12ab),            /* X-Box 360 dance pads */
>         XPAD_XBOX360_VENDOR(0x1430),            /* RedOctane X-Box 360 controllers */
> --
> 2.7.2
>