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

Micmute Key support of ThinkCentre M820z and M920z #9619

Merged
merged 2 commits into from
Jul 19, 2018

Conversation

shengyao
Copy link
Contributor

There was a ThinkCentre M800z Micmute Key support entry.
But on new models M820z and M920z, the lines must change a bit,
the keyboard:name... must change to evdev:name.
Already tested on the M820z and M920z hardwares.

There was a ThinkCentre M800z Micmute Key support entry.
But on new models M820z and M920z, the lines must change a bit,
the keyboard:name... must change to evdev:name.
Already tested on the M820z and M920z hardwares.
@poettering
Copy link
Member

poettering commented Jul 18, 2018

Quite frankly the existing line simply looks wrong... It's the only entry using the keyboard: prefix, and we generally don't do that.

This looks like something where we should just fix the entry by replacing the "keyboard:" prefix by "evdev:", and extend the comment. Can you change things loike that and force push a new version?

@poettering poettering added hwdb reviewed/needs-rework 🔨 PR has been reviewed and needs another round of reworks labels Jul 18, 2018
Per Poettering's comment in systemd#9619,
We better combine M800z and M820/M920z entries and use the "evdev" prefix.
@poettering poettering merged commit 3230e79 into systemd:master Jul 19, 2018
@shengyao shengyao deleted the shengyao branch July 19, 2018 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hwdb reviewed/needs-rework 🔨 PR has been reviewed and needs another round of reworks
Development

Successfully merging this pull request may close these issues.

None yet

2 participants