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

Pixelbook fixes #3

Merged
merged 3 commits into from
Sep 11, 2019
Merged

Pixelbook fixes #3

merged 3 commits into from
Sep 11, 2019

Conversation

valpackett
Copy link
Contributor

Fixes for the Google Pixelbook touchpad :) See commit messages.

Gestures work, tap works, tap and drag works, only physical click doesn't work, but the physical click on this device sucks anyway.

(the no-two-writes commit is not very good code, feel free to do whatever you feel is better code style - append the buf into cmd, etc.)

This write does not seem correct, Linux and OpenBSD do not do this.

The Google Pixelbook (eve)'s touchpad (rose) was going off the rails
(i.e. screwing up the whole i2c bus) because of this write.
The Google Pixelbook (eve)'s touchpad (rose) did not understand the
two writes and did not switch into multitouch mode.
Copy link
Owner

@wulf7 wulf7 left a comment

Choose a reason for hiding this comment

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

Good catch! Thanks! MS specs does not contain this write too

@wulf7 wulf7 merged commit 1325946 into wulf7:master Sep 11, 2019
@wulf7
Copy link
Owner

wulf7 commented Sep 11, 2019

(the no-two-writes commit is not very good code, feel free to do whatever you feel is better code style - append the buf into cmd, etc.)
I combined 2 writes in iic driver level with IIC_NOSTART flag set. I hope it works

@valpackett
Copy link
Contributor Author

Yes! Works fine, thanks.

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