Skip to content

Conversation

@flying-elephant
Copy link
Contributor

@flying-elephant flying-elephant commented Nov 6, 2025

This PR imports the patch applied as 41cb08555c41 and modifies the configure.ac file to
apply it to the appropriate Linux versions.

[[email protected]: Imported into input-wacom (41cb08555c41)]
Signed-off-by: Tatsunosuke Tobita [email protected]
Link: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/hid/wacom_wac.c?h=v6.18-rc4&id=41cb08555c4164996d67c78b3bf1c658075b75f1

Move this API to the canonical timer_*() namespace.

[ tglx: Redone against pre rc1 ]

Signed-off-by: Ingo Molnar [email protected]
Signed-off-by: Thomas Gleixner [email protected]
Link: https://lore.kernel.org/all/[email protected]
From the kernel version 6.16, timer_container_of is used instead
of from_timer. To check whether or not it is available,
another AC_MSG_CHECKING block has been added.

Signed-off-by: Tatsunosuke Tobita [email protected]
@Pinglinux
Copy link
Member

The commit comments for the 3rd patch had extra spaces. Can we drop the 3rd patch? When "timer_container_of is not defined", it is not an error. It just means that we are on an older kernel (< 6.16) that doesn't support timer_container_of. No error messages need to be reported.

While you are on the page, can you make sure the email addresses in the first two patches follow the correct format
Signed-off-by: Tatsunosuke Tobita [email protected]? You missed <> for them.

@flying-elephant
Copy link
Contributor Author

@Pinglinux
timer_container_of is a preprocessor macro, so we cannot use as we did. (timer_container_of cannot be detected as a symbol of a function)
Instead, we need "error" when the target macro, which is timer_container_of, is not defined as a define macro.(used as #define, which is not a function)

configure.ac: modify timer_container_of check

    Since timer_container_of is defined as a macro, the checking
    process has been modified.

    Signed-off-by: Tatsunosuke Tobita <[email protected]>
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