Bug 196624 - [patch] wsp.c toggle for one finger tap when integrated button is detected.
Summary: [patch] wsp.c toggle for one finger tap when integrated button is detected.
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: usb (show other bugs)
Version: 10.3-RELEASE
Hardware: Any Any
: --- Affects Some People
Assignee: Kyle Evans
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-12 17:52 UTC by David O'Rourke
Modified: 2018-01-27 06:11 UTC (History)
4 users (show)

See Also:
kevans: mfc-stable11+


Attachments
wsp.c one finger tap. (1.36 KB, patch)
2015-01-12 21:53 UTC, David O'Rourke
no flags Details | Diff
wsp(4) updated to include details of one finger tap. (420 bytes, patch)
2015-01-12 21:54 UTC, David O'Rourke
no flags Details | Diff
wsp.c one finger tap (1.41 KB, patch)
2017-02-08 18:14 UTC, David O'Rourke
no flags Details | Diff
wsp.4 man page update (420 bytes, patch)
2017-02-08 18:14 UTC, David O'Rourke
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David O'Rourke 2015-01-12 17:52:14 UTC
This patch adds hw.usb.wsp.one_finger_tap which allows toggling left clicking
by tapping on the trackpad with a single finger. Setting it to 0 disables,
setting it to >= 1 enables.

I have not updated the wsp(4) man page yet, but will be doing so shortly.

Environment:
FreeBSD 10.1-RELEASE
Comment 1 Gavin Atkinson freebsd_committer freebsd_triage 2015-01-12 18:48:59 UTC
Hi,

Due to an issue with the backend FreeBSD Bugzilla database, your original PR and any updates to it since have been lost.  I've recreated the original PR as best as I can, however any attachments and updates you submitted to the PR have been lost.  Please could you resubmit them?

Thanks, and apologies.
Comment 2 David O'Rourke 2015-01-12 21:53:44 UTC
Created attachment 151509 [details]
wsp.c one finger tap.

One finger tap left-click for wsp.c. Resubmitting after Bugzilla maintenance issues.
Comment 3 David O'Rourke 2015-01-12 21:54:36 UTC
Created attachment 151510 [details]
wsp(4) updated to include details of one finger tap.

wsp(4) man page with one finger tap details.
Comment 4 Thomas Brierley 2016-07-01 10:40:15 UTC
I did exactly the same thing under a different OID before seeing this :D

Tested on 10.3-RELEASE
Comment 5 Hans Petter Selasky freebsd_committer freebsd_triage 2016-07-01 10:47:50 UTC
Hi,

Is this issue fixed in 10-stable? Or is this patch still needed?

--HPS
Comment 6 David O'Rourke 2016-07-01 11:26:01 UTC
A quick look at the source for wsp.c on github suggests that the patch is still needed, however work on wsp.c since I submitted this patch means it won't apply cleanly anymore.

If there is still interest in this feature, I should be able to clean up the .diff later on so that it once again apples to HEAD.
Comment 7 Thomas Brierley 2016-07-01 11:28:12 UTC
The patch still needs to be applied. None of the branches have this in wsp.c (or anything equivalent).

wsp.c in STABLE is at the same commit as RELEASE. CURRENT has a couple of small changes that aren't for touch click.
Comment 8 Thomas Brierley 2016-07-01 11:32:24 UTC
Hi David

I think your patch will still work, there's only one additional line and one line change on master, it should be able to figure out the displacement.
Comment 9 Thomas Brierley 2016-07-01 13:14:15 UTC
No I'm wrong... it fails, sorry.

I can submit a new one based on the same changes later today if you like, I need it anyway.
Comment 10 Dave Cottlehuber freebsd_committer freebsd_triage 2017-01-25 19:42:19 UTC
I would be interested in this too if it can be bumped for 11 or 12.
Comment 11 David O'Rourke 2017-02-08 18:14:02 UTC
Created attachment 179770 [details]
wsp.c one finger tap

One finger tap patch updated for 10.3
Comment 12 David O'Rourke 2017-02-08 18:14:49 UTC
Created attachment 179771 [details]
wsp.4 man page update

Updated man page for 10.3.
Comment 13 David O'Rourke 2017-02-08 18:15:47 UTC
This feature has now been updated for 10.3. I'll be looking at FreeBSD 11 shortly and providing a new patch if needed.
Comment 14 Kyle Evans freebsd_committer freebsd_triage 2018-01-15 21:30:50 UTC
This feature had been committed (in a different form) as r314467. Hijacking this PR for MFC tracking, and wsp(4) should probably be updated in a similar fashion to the patch attached here to indicate the existence of this new sysctl.
Comment 15 commit-hook freebsd_committer freebsd_triage 2018-01-15 21:53:45 UTC
A commit references this bug:

Author: kevans
Date: Mon Jan 15 21:53:33 UTC 2018
New revision: 328027
URL: https://svnweb.freebsd.org/changeset/base/328027

Log:
  wsp(4): Update to reflect new sysctl from r314467

  r314467 introduced hw.usb.wsp.enable_single_tap_clicks to enable/disable
  single-tap left click behavior. Update the man page to reflect the new
  sysctl.

  PR:		196624
  MFC after:	3 days
  X-MFC-With:	r314467

Changes:
  head/share/man/man4/wsp.4
Comment 16 commit-hook freebsd_committer freebsd_triage 2018-01-27 06:10:29 UTC
A commit references this bug:

Author: kevans
Date: Sat Jan 27 06:10:21 UTC 2018
New revision: 328460
URL: https://svnweb.freebsd.org/changeset/base/328460

Log:
  MFC r314467,r328027: Add hw.usb.wsp.enable_single_tap_clicks sysctl

  MFC r314467 (imp): Make wsp process a single touchpad tap and interpret it
  as a left-click event.  It can be disabled by setting the new
  hw.usb.wsp.enable_single_tap_clicks sysctl to 0.

  MFC r328027: wsp(4): Update to reflect new sysctl from r314467

  r314467 introduced hw.usb.wsp.enable_single_tap_clicks to enable/disable
  single-tap left click behavior. Update the man page to reflect the new
  sysctl.

  PR:		196624

Changes:
_U  stable/11/
  stable/11/share/man/man4/wsp.4
  stable/11/sys/dev/usb/input/wsp.c