From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752681AbbCKRZr (ORCPT ); Wed, 11 Mar 2015 13:25:47 -0400 Received: from mail-ob0-f174.google.com ([209.85.214.174]:34659 "EHLO mail-ob0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751478AbbCKRZo (ORCPT ); Wed, 11 Mar 2015 13:25:44 -0400 Date: Wed, 11 Mar 2015 12:25:25 -0500 From: Seth Forshee To: Jiri Kosina Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, Benjamin Tissoires Subject: Re: [PATCH v2] HID: multitouch: Add support for button type usage Message-ID: <20150311172525.GB136254@ubuntu-hedt> References: <1425070801-73595-1-git-send-email-seth.forshee@canonical.com> <1425180871-5232-1-git-send-email-seth.forshee@canonical.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Mar 11, 2015 at 12:19:13PM -0400, Jiri Kosina wrote: > On Sat, 28 Feb 2015, Seth Forshee wrote: > > > According to [1], Windows Precision Touchpad devices must supply > > a button type usage in the device capabilities feature report. A > > value of 0 indicates that the device contains a depressible > > button (i.e. it's a click-pad) whereas a value of 1 indicates > > a non-depressible button. Add support for this usage and set > > INPUT_PROP_BUTTONPAD on the touchpad input device whenever a > > depressible button is present. > > > > v2: Add string for button type usage in debugfs. > > > > [1] https://msdn.microsoft.com/en-us/library/windows/hardware/dn467314(v=vs.85).aspx > > > > Signed-off-by: Seth Forshee > > I'd like to have Benjamin's Ack on this. Adding him to CC. Please also see > 015fdaa9f8ed ("HID: multitouch: add support of clickpads"). That commit does also gets INPUT_PROP_BUTTONPAD set for my touchpad. I'll leave it up to the two of you to decide whether or not there's benefit in also using the usage to set the property bit. I'll also follow up with a v3 patch which resolves the conflicts with Benjamin's patch. Also, I should probably mention that I don't have any non-depressible touchpads which set the button type usage that I can test with. If anyone has such hardware it would be great to get this tested against it. Thanks, Seth