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

add device hwdb for IdeaPad Miix510-12ISK #11108

Merged
merged 1 commit into from Dec 14, 2018
Merged

add device hwdb for IdeaPad Miix510-12ISK #11108

merged 1 commit into from Dec 14, 2018

Conversation

BruceZhang1993
Copy link
Contributor

Add hwdb ACCEL_MOUNT_MATRIX for IdeaPad Miix510-12ISK

@poettering
Copy link
Member

/cc @hadess

@poettering
Copy link
Member

also, tests don't pass, hwdb file is not properly formatted.

@poettering poettering added the ci-fails/needs-rework 🔥 Please rework this, the CI noticed an issue with the PR label Dec 10, 2018
@poettering
Copy link
Member

looks ok, but please extend the comment there, do mention your device model

@poettering poettering added squash-on-merge and removed ci-fails/needs-rework 🔥 Please rework this, the CI noticed an issue with the PR labels Dec 10, 2018
@hadess
Copy link
Contributor

hadess commented Dec 11, 2018

Could you please squash the changes squash on your branch? git rebase -i origin/master and use the fixup or squash option to merge all 3 changes into one, and use git push -f to push your changes to your branch.

add ACCEL_MOUNT_MATRIX for IdeaPad Miix510-12ISK

multiple match expressions for Miix510 series

extend comments for miix510 series

Signed-off-by: Bruce Zhang <zttt183525594@gmail.com>

add device hwdb for IdeaPad Miix510-12ISK

Signed-off-by: Bruce Zhang <zttt183525594@gmail.com>
@keszybz
Copy link
Member

keszybz commented Dec 14, 2018

@hadess it's not really necessary to ask contributors to squash. Github can do that for us when merging, which works just as well as long as we are fine with having one patch in the end. For simple PRs like this this is good enough. We use 'please-squash-on-merge' label to mark such cases.

@keszybz keszybz merged commit 0b10930 into systemd:master Dec 14, 2018
@BruceZhang1993 BruceZhang1993 deleted the patch-1 branch December 14, 2018 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

4 participants