From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?Q2zDqW1lbnQ=?= Calmels Subject: Re: [PATCH] Input: xpad - remove spurious events of wireless xpad 360 controller Date: Sun, 22 Nov 2015 17:26:14 +0100 Message-ID: <20151122172614.10d1fc46@gromit> References: <1446409571-15146-1-git-send-email-clement.calmels@free.fr> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from smtpfb2-g21.free.fr ([212.27.42.10]:57583 "EHLO smtpfb2-g21.free.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752138AbbKVQ0f (ORCPT ); Sun, 22 Nov 2015 11:26:35 -0500 In-Reply-To: <1446409571-15146-1-git-send-email-clement.calmels@free.fr> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Dmitry Torokhov , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org On Sun, 1 Nov 2015 21:26:11 +0100 clement.calmels@free.fr wrote: > From: Clement Calmels > > When powering up a wireless xbox 360 controller, some wrong joystick > events are generated. It is annoying because, for example, it makes > unwanted moves in Steam big picture mode's menu. > > When my controller connects, this packet is received by the driver: > 00000000: 00 0f 00 f0 00 cc ff cf 8b e0 86 6a 68 f0 00 > 20 ...........jh.. 00000010: 13 e3 20 1d 30 03 40 01 50 01 ff > ff .. .0.@.P... According to xboxdrv: > https://github.com/xboxdrv/xboxdrv/blob/stable/src/xbox360_wireless_controller.cpp#L118 > this packet is only dumping a serial id and should not be interpreted > as joystick events. > > This issue can be easily seen with: $ jstest --event /dev/input/js0 > This patch only adds a way to filter out this "serial" packet and as a > result it removes the spurous events. > > Signed-off-by: Clement Calmels > --- > drivers/input/joystick/xpad.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/input/joystick/xpad.c > b/drivers/input/joystick/xpad.c index f8850f9..68e5bd3 100644 > --- a/drivers/input/joystick/xpad.c > +++ b/drivers/input/joystick/xpad.c > @@ -523,7 +523,7 @@ static void xpad360w_process_packet(struct > usb_xpad *xpad, u16 cmd, unsigned cha } > > /* Valid pad data */ > - if (!(data[1] & 0x1)) > + if (!(data[1] & 0x1) || data[1] == 0xf) > return; > > xpad360_process_packet(xpad, cmd, &data[4]); After more tests, the 'if' statement can be simplified. I will send a second version of this patch. c.