Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753785Ab3IFWv3 (ORCPT ); Fri, 6 Sep 2013 18:51:29 -0400 Received: from mail-ie0-f182.google.com ([209.85.223.182]:62043 "EHLO mail-ie0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751788Ab3IFWv2 (ORCPT ); Fri, 6 Sep 2013 18:51:28 -0400 MIME-Version: 1.0 In-Reply-To: <20130906215910.GA356@x4> References: <20130906202022.GA353@x4> <20130906215910.GA356@x4> Date: Sat, 7 Sep 2013 00:51:27 +0200 Message-ID: Subject: Re: [GIT] HID for 3.12 merge window From: David Herrmann To: Markus Trippelsdorf Cc: Jiri Kosina , Linus Torvalds , linux-kernel , "open list:HID CORE LAYER" , Dmitry Torokhov Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1898 Lines: 52 Hi On Fri, Sep 6, 2013 at 11:59 PM, Markus Trippelsdorf wrote: > On 2013.09.06 at 23:50 +0200, David Herrmann wrote: >> Hi >> >> On Fri, Sep 6, 2013 at 10:20 PM, Markus Trippelsdorf >> wrote: >> > On 2013.09.06 at 14:00 +0200, Jiri Kosina wrote: >> >> >> >> David Herrmann (12): >> > ... >> >> HID: wiimote: add support for Guitar-Hero drums >> > >> > commit 61e00655e9cb82e034eb72b95a51072e718d14a7 >> > Author: David Herrmann >> > Date: Mon Aug 26 19:14:46 2013 +0200 >> > >> > Input: introduce BTN/ABS bits for drums and guitars >> > >> > The commit above breaks my Logitech mouse. The mouse cursor just sits in >> > the middle of the screen and doesn't react to movements. dmesg is >> > normal, but Xorg.0.log says: >> >> Ok, the issue is the kernel assumes ABS_MAX to be a power-of-2 minus 1 >> (used as mask). That wasn't really obvious to me. Attached is a patch >> which should fix that. Could you apply it on top of linus/master and >> give it a try? > > Your patch fixes the issue. Thanks. Thanks a lot for reporting+testing! I am still not sure how to solve the EVIOCSABS thingy. Problem is, it's defined as: #define EVIOCSABS(_abs) ...0xc0 + (_abs)... But if (_abs > 0x3f) this will be bigger than 0xff. Unfortunately, the upper part of the ioctl is defined as 'E' which is 0x45 in hex and thus sets the LSB. That means we cannot extend the _IOC_TYPE field to the upper region (which would cause endian-issues, anyway). I guess we're screwed here and need to revert that... Dmitry, any comment on this? Or am I missing something? Regards David -- 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/