linux-kernel - Re: [PATCH v3] HID: multitouch: Add support for button type usage

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: <20150311215523.GC136254@ubuntu-hedt>
Date:	Wed, 11 Mar 2015 16:55:23 -0500
From:	Seth Forshee <seth.forshee@...onical.com>
To:	Benjamin Tissoires <benjamin.tissoires@...hat.com>
Cc:	Jiri Kosina <jkosina@...e.cz>, linux-input@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3] HID: multitouch: Add support for button type usage

On Wed, Mar 11, 2015 at 04:55:59PM -0400, Benjamin Tissoires wrote:
> Hi Seth,
> 
> On Mar 11 2015 or thereabouts, 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.
> > v3: Fix conflicts with 015fdaa9f8ed ("HID: multitouch: add
> >     support of clickpads").
> > 
> > [1] https://msdn.microsoft.com/en-us/library/windows/hardware/dn467314(v=vs.85).aspx
> > 
> > Signed-off-by: Seth Forshee <seth.forshee@...onical.com>
> > ---
> 
> [Replying to the previous thread here for convenience]
> 
> Sorry I might have overlooked your patch and forgot about it when I sent
> mine. I should definitively have looked at this in detail because that's
> a better approach IMO.
> However, the patch I sent allows to handle clickpads which are not
> following MS spec, so I think your v3 on top of mine makes sense.
> 
> I have a few bike-shedding if you don't mind:

Sure, I opted to maintain the terminology from the MS spec but I have no
problem doing it that way. I'll follow up with another patch once I get
it tested.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/