Skip to content

documentation:kernel_modules: fix typos and grammar #417

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Quin-Darcy
Copy link

This pull request focuses on improving the clarity and accuracy of the Documentation/teaching/labs/kernel_modules.rst file. The changes primarily involve fixing typographical errors, improving grammar, and ensuring consistency in formatting and terminology.

Typographical and grammatical corrections:

  • Corrected "writting" to "writing" in the hint about setting the console log level.
  • Replaced "an user program" with "a user program" for grammatical correctness.
  • Changed "providing the possibility" to "providing the ability" for better readability.
  • Reworded "Until the release of those references" to "Until those references are released" for improved clarity.

Terminology and formatting improvements:

  • Standardized the use of := in Kbuild file examples for consistency. [1] [2]
  • Updated "loadable kernel mode" to "loadable kernel module" for accuracy.
  • Replaced "simple and quick way" with "simple and quick method" to improve phrasing.
  • Adjusted phrasing to use "ability to perform live debugging" instead of "possibility to perform live debugging."

Punctuation adjustments:

  • Replaced a semicolon with an em dash for better sentence flow in an explanation about storing values in memory.
  • Added a comma for clarity in a sentence about system freezes and logs.

@Quin-Darcy Quin-Darcy changed the title docs(kernel_modules): fix typos and grammar documentation:kernel_modules: fix typos and grammar May 1, 2025
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.

1 participant