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
linux_4_18: HID: core: fix grouping by application #46061
Conversation
@GrahamcOfBorg build linux_4_18 |
Success on aarch64-linux (full log) Attempted: linux_4_18 Partial log (click to expand)
|
Timed out, unknown build status on x86_64-linux (full log) Attempted: linux_4_18 Partial log (click to expand)
|
563ab47
to
9a9d396
Compare
This patch fixes NixOS#45165 commit f07b3c1da92d ("HID: generic: create one input report per application type") was effectively the same as MULTI_INPUT: hidinput->report was never set, so hidinput_match_application() always returned null. Fix that by testing against the real application. Note that this breaks some old eGalax touchscreens that expect MULTI_INPUT instead of HID_QUIRK_INPUT_PER_APP. Enable this quirk for backward compatibility on all non-Win8 touchscreens. link: https://bugzilla.kernel.org/show_bug.cgi?id=200847 link: https://bugzilla.kernel.org/show_bug.cgi?id=200849 link: https://bugs.archlinux.org/task/59699 link: NixOS#45165 Cc: stable@vger.kernel.org # v4.18+ Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
9a9d396
to
b2423a3
Compare
patch = fetchpatch { | ||
name = name + ".patch"; | ||
# https://patchwork.kernel.org/patch/10587369/ | ||
url = https://git.kernel.org/pub/scm/linux/kernel/git/jikos/hid.git/patch/?id=0d6c3011409135ea84e2a231b013a22017ff999a; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This patch can be fetched through either https://git.kernel.org/pub/scm/linux/kernel/git/jikos/hid.git/patch/?id=0d6c3011409135ea84e2a231b013a22017ff999a or https://patchwork.kernel.org/patch/10587369/raw/ I'm not sure which one is better to use here as I'm afraid the patchwork link might change over time, and I'm not sure this direct commit link will remain available. The checksum of the URLs differs due to the change in the comments in the header of the patch. @NeQuissimus thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you're unsure, just bring the patch into the tree, so we have a local versions. I would imagine the git.kernel.org version to be the preferable solution, though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NeQuissimus How about we push it as is for now? It should land in an upstream release soon anyway.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Success on aarch64-linux (full log) Attempted: linux_4_18 Partial log (click to expand)
|
Timed out, unknown build status on x86_64-linux (full log) Attempted: linux_4_18 Partial log (click to expand)
|
@grahamc Is the 1800s timeout a hard limit? Needs a good amount of steam to build a kernel in 30 mins :D |
@NeQuissimus it's controlled by the package's meta, See Chromium. We can probably increase it to a few hours in a separate PR. EDIT: It should probably be done here. |
Ah, good find, did not know about this! |
@NeQuissimus I filed #46073 for increasing the timeout. |
I saw :D |
nice! Thanks. |
Motivation for this change
This patch fixes #45165
commit f07b3c1da92d ("HID: generic: create one input report per
application type") was effectively the same as MULTI_INPUT:
hidinput->report was never set, so hidinput_match_application()
always returned null.
Fix that by testing against the real application.
Note that this breaks some old eGalax touchscreens that expect MULTI_INPUT
instead of HID_QUIRK_INPUT_PER_APP. Enable this quirk for backward
compatibility on all non-Win8 touchscreens.
link: https://bugzilla.kernel.org/show_bug.cgi?id=200847
link: https://bugzilla.kernel.org/show_bug.cgi?id=200849
link: https://bugs.archlinux.org/task/59699
link: #45165
Cc: stable@vger.kernel.org # v4.18+
Signed-off-by: Benjamin Tissoires benjamin.tissoires@redhat.com
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)cc @NeQuissimus