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

Remove module * parameter from class_create() #222

Merged
merged 2 commits into from
Sep 2, 2023

Conversation

mechanicalamit
Copy link
Collaborator

@mechanicalamit mechanicalamit commented Sep 1, 2023

  • From v6.4, class_create() does not have module *
    (See commit 11ba11 below)
  • Added #ifdev for KERNEL_VERSION >= 6.4

From linux commit aaba11da9aa7d7d6b52a74d45b31cac118295a1

driver core: class: remove module * from class_create()

The module pointer in class_create() never actually did anything,
and it shouldn't have been requred to be set as a parameter even
if it did something. So just remove it and fix up all callers of the
function in the kernel tree at the same time.

@jserv jserv requested a review from linD026 September 2, 2023 02:05
- From v6.4, class_create() does not have module *
  (See commit 11ba11 below)
- Added #ifdev for KERNEL_VERSION >= 6.4

From linux commit aaba11da9aa7d7d6b52a74d45b31cac118295a1
---------------------------------------------
driver core: class: remove module * from class_create()

The module pointer in class_create() never actually did anything,
and it shouldn't have been requred to be set as a parameter even
if it did something.  So just remove it and fix up all callers of the
function in the kernel tree at the same time.
examples/chardev.c Outdated Show resolved Hide resolved
@mechanicalamit mechanicalamit marked this pull request as ready for review September 2, 2023 11:21
Copy link
Contributor

@jserv jserv left a comment

Choose a reason for hiding this comment

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

Please run clang-format -i to ensure the consistent style.

@jserv jserv changed the title remove module * parameter from class_create() Remove module * parameter from class_create() Sep 2, 2023
@jserv jserv merged commit 0addb86 into sysprog21:master Sep 2, 2023
1 check passed
@jserv
Copy link
Contributor

jserv commented Sep 2, 2023

Thank @mechanicalamit for contributing!

@mechanicalamit
Copy link
Collaborator Author

Thank @mechanicalamit for contributing!

Thank you @jserv, for guiding me through.

@mechanicalamit mechanicalamit deleted the chardev.c-v6.4 branch September 3, 2023 05:52
mechanicalamit added a commit to mechanicalamit/lkmpg that referenced this pull request Sep 17, 2023
From v6.4, class_create() does not have module *
Same fix as (sysprog21#222), commit 0addb86 to chardev.c
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.

3 participants