Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore i2c mouse devices #74

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Ignore i2c mouse devices #74

wants to merge 3 commits into from

Conversation

c84c
Copy link

@c84c c84c commented Feb 13, 2022

Description

Ignore i2c mouse devices. Already known issue.

Works like a charm on my Dell XPS 15 9560.

WARNING

To be deeply tested with other PC vendors/products !

I haven't tested it with other PC than mine, but I can suppose that the devices interaction with i2c kernel modules is the same.

Related Issues

Maybe related issues are: #40, #61, #64 and #67.

For #67 I think that i2c devices are one problem. The other is multiple mouse connected/discovered.

System info

image

Devices:

image

Extension devices:

image

@askmrsinh
Copy link
Owner

askmrsinh commented Feb 14, 2022

Thanks again! I think this should work in majority of the cases unless the device manufacturer does something weird.
The original idea was to implement an UI field that would allow such devices to be ignored manually.

@c84c
Copy link
Author

c84c commented Feb 14, 2022

The original idea was to implement an UI field that would allow such devices to be ignored manually.

By the user? Can be a good idea, but debugging extension with user ignored devices can be complicated. In event of implementation of this feature, I suggest you to add a mandatory field in issue templates where insert user ignored devices.

…se "Wacom HID 488F" devices are reported as Finger or Pen in /proc/bus/input/devices
Copy link
Author

@c84c c84c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I miss the review.
Implements user excluded devices can be another PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants