Skip to content

kconfig: series of fix typo #92330

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

Merged

Conversation

ndrs-pst
Copy link
Contributor

Utilize a code spell-checking tool to scan for and correct spelling errors, focusing on Kconfig files.

@ndrs-pst ndrs-pst force-pushed the pr_kconfig_typo_2025-06-28 branch from f5dd289 to 9535efd Compare June 28, 2025 07:58
nashif
nashif previously approved these changes Jun 28, 2025
@nashif nashif assigned kartben and unassigned ruuddw and evgeniy-paltsev Jun 28, 2025
@@ -13,7 +13,7 @@ choice NULL_POINTER_EXCEPTION_DETECTION
endchoice

# BT relies on PSA Crypto API to perform crypto operations and, on this platform,
# these APIs are provided thougth Mbed TLS. Unfortunately this platform is not
# these APIs are provided through Mbed-TLS. Unfortunately, this platform is not
Copy link
Contributor

Choose a reason for hiding this comment

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

It's "Mbed TLS" afaict

Copy link
Contributor Author

@ndrs-pst ndrs-pst Jun 30, 2025

Choose a reason for hiding this comment

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

Yes, after some checking, it appears that "Mbed TLS" is the correct form.
Should I omit this or address it? Will address later.

Copy link
Contributor

@kartben kartben Jun 30, 2025

Choose a reason for hiding this comment

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

please don't change it since it's the correct spelling. But feel free to fix any wrong usage via this PR though, but this one line isn't one of them

Copy link
Contributor

Choose a reason for hiding this comment

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

@ndrs-pst ping?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had just missed it, but it's been addressed now.

aaronyegx
aaronyegx previously approved these changes Jun 30, 2025
erwango
erwango previously approved these changes Jun 30, 2025
ruuddw
ruuddw previously approved these changes Jun 30, 2025
@ndrs-pst ndrs-pst dismissed stale reviews from ruuddw, erwango, aaronyegx, and nashif via 216fe95 June 30, 2025 16:00
@ndrs-pst ndrs-pst force-pushed the pr_kconfig_typo_2025-06-28 branch from 9535efd to 216fe95 Compare June 30, 2025 16:00
sylvioalves
sylvioalves previously approved these changes Jun 30, 2025
@ndrs-pst
Copy link
Contributor Author

Thank you for the review. I hope I have covered all of the suggestions.

etienne-lms
etienne-lms previously approved these changes Jun 30, 2025
Copy link
Contributor

@mathieuchopstm mathieuchopstm left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM.

Utilize a code spell-checking tool to scan for and correct spelling errors
in `Kconfig` files within the `drivers` directory.
Additionally, incorporates a fix recommended by the reviewer.

Signed-off-by: Pisit Sawangvonganan <pisit@ndrsolution.com>
@ndrs-pst ndrs-pst dismissed stale reviews from etienne-lms and sylvioalves via 7d4ec76 July 1, 2025 15:31
@ndrs-pst ndrs-pst force-pushed the pr_kconfig_typo_2025-06-28 branch from 216fe95 to 7d4ec76 Compare July 1, 2025 15:31
sylvioalves
sylvioalves previously approved these changes Jul 1, 2025
ndrs-pst added 2 commits July 1, 2025 23:17
Utilize a code spell-checking tool to scan for and correct spelling errors
in `Kconfig` files within the `arch`, `boards`, `kernel`, `modules`,
`samples`, and `share` directory.
Additionally, incorporates a fix recommended by the reviewer.

Signed-off-by: Pisit Sawangvonganan <pisit@ndrsolution.com>
Utilize a code spell-checking tool to scan for and correct spelling errors
in `Kconfig` files within the `soc` and `subsys` directory.
Additionally, incorporates a fix recommended by the reviewer.

Signed-off-by: Pisit Sawangvonganan <pisit@ndrsolution.com>
Copy link
Contributor

@kartben kartben left a comment

Choose a reason for hiding this comment

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

Thank you very much for this, @ndrs-pst!

Copy link

sonarqubecloud bot commented Jul 1, 2025

Copy link
Contributor

@dkalowsk dkalowsk left a comment

Choose a reason for hiding this comment

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

Thanks for the cleanup!

@dkalowsk dkalowsk merged commit 0ec49fa into zephyrproject-rtos:main Jul 1, 2025
37 checks passed
@ndrs-pst ndrs-pst deleted the pr_kconfig_typo_2025-06-28 branch July 2, 2025 05:52
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.