Skip to content

hwdb: add axis range corrections for Dell Latitude E7470#6156

Merged
whot merged 1 commit intosystemd:masterfrom
pakdel:Dell_Latitude_E7470_TouchPad
Jun 21, 2017
Merged

hwdb: add axis range corrections for Dell Latitude E7470#6156
whot merged 1 commit intosystemd:masterfrom
pakdel:Dell_Latitude_E7470_TouchPad

Conversation

@pakdel
Copy link
Contributor

@pakdel pakdel commented Jun 20, 2017

@poettering
Copy link
Member

poettering commented Jun 20, 2017

/cc @whot

@whot whot merged commit 9d3c1ef into systemd:master Jun 21, 2017
@macrothian
Copy link
Contributor

macrothian commented Nov 3, 2017

These ranges broke the edge scrolling on my E7470 as the edge was suddenly further right than possible.
Please see #7252 that has ranges touchpad-edge-detector suggested (and worked).

@whot
Copy link
Contributor

whot commented Nov 6, 2017

@pakdel please re-measure your touchpad, the range we merged in commit 9f14144 looks a bit unusual. thanks

@pakdel
Copy link
Contributor Author

pakdel commented Nov 6, 2017 •
edited
Loading

@whot Here is what I got:

[root@RP-YUL-L1319 ~]# touchpad-edge-detector 98x52 /dev/input/event20
Touchpad AlpsPS/2 ALPS DualPoint TouchPad on /dev/input/event20
Move one finger around the touchpad to detect the actual edges
Kernel says:    x [39..5856], y [10..1532]
Touchpad sends: x [21..2937], y [45..1535] -|-^C

Touchpad size as listed by the kernel: 98x52mm
User-specified touchpad size: 98x52mm
Calculated ranges: 2916/1490

Suggested udev rule:
# <Laptop model description goes here>
evdev:name:AlpsPS/2 ALPS DualPoint TouchPad:dmi:bvnDellInc.:bvr1.17.5:bd07/11/2017:svnDellInc.:pnLatitudeE7470:pvr:rvnDellInc.:rn0T6HHJ:rvrA00:cvnDellInc.:ct9:cvr:*
 EVDEV_ABS_00=21:2937:30
 EVDEV_ABS_01=45:1535:29
 EVDEV_ABS_35=21:2937:30
 EVDEV_ABS_36=45:1535:29

@whot
Copy link
Contributor

whot commented Nov 6, 2017

ok, looks like the one we merged was a typo 5->2 would make the range more sensible. We'll fix this as part of #7252 then, thanks.

@pakdel
Copy link
Contributor Author

pakdel commented Nov 6, 2017

Sorry for the typo.
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@whot whot whot approved these changes

Assignees

No one assigned

Labels

hwdb

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@pakdel @poettering @macrothian @whot