Skip to content

Update to Linux 6.12#707

Merged
JoseExposito merged 2 commits intoDIGImend:masterfrom
Stroopwafe1:master
Jan 2, 2025
Merged

Update to Linux 6.12#707
JoseExposito merged 2 commits intoDIGImend:masterfrom
Stroopwafe1:master

Conversation

@Stroopwafe1
Copy link
Contributor

@Stroopwafe1 Stroopwafe1 commented Nov 25, 2024

This patch allows the digimend kernel drivers to build on Linux kernel 6.12.
The reason it failed before is because of a header rename from <asm/unaligned.h> to <linux/unaligned.h>. And the struct of the hid_driver changed requiring a const pointer for the report_fixup

@Stroopwafe1 Stroopwafe1 mentioned this pull request Nov 25, 2024
Closed
@kenhys
Copy link
Contributor

kenhys commented Dec 8, 2024

It would be nice if it also still supports 6.11 (#if LINUX_VERSION_CODE < KERNEL_VERSION(6, 12, 0))

@Stroopwafe1
Copy link
Contributor Author

Stroopwafe1 commented Dec 16, 2024 •
edited
Loading

I have updated the code with a check for the linux kernel, and I tested this in a VM with kernel version 6.9.3-76060903-generic and it built fine, as well as on my regular hardware using kernel 6.12.4-arch1-1. Let me know if there's anything else.

Oh and thank you @kenhys for the feedback, I didn't know you could check for the version like that

@SohamG
Copy link

SohamG commented Dec 30, 2024

Hey! I just updated to 6.12 and had to pull this driver from my system. I was wondering if this PR would land soon (given the primary maintainers resignation blog post)? I was considering adding this as a backport to my linux distro in the meanwhile (NixOS)

JoseExposito
JoseExposito approved these changes Jan 2, 2025
View reviewed changes
Copy link
Collaborator

@JoseExposito JoseExposito left a comment

Choose a reason for hiding this comment

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

I was fixing the same issue, but I realized you fixed it already. Thanks!

@JoseExposito JoseExposito merged commit f3c7c7f into DIGImend:master Jan 2, 2025
@JoseExposito JoseExposito mentioned this pull request Jan 2, 2025
Closed
This was referenced Jan 14, 2025
Closed
Closed
@danpashin danpashin mentioned this pull request Feb 7, 2025
Merged
@gleb0g gleb0g mentioned this pull request May 7, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JoseExposito JoseExposito JoseExposito approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Stroopwafe1 @kenhys @SohamG @JoseExposito