-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
kconfig: series of fix typo #92330
Conversation
f5dd289
to
9535efd
Compare
@@ -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 |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ndrs-pst ping?
There was a problem hiding this comment.
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.
9535efd
to
216fe95
Compare
Thank you for the review. I hope I have covered all of the suggestions. |
There was a problem hiding this 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>
216fe95
to
7d4ec76
Compare
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>
7d4ec76
to
1f96d5a
Compare
There was a problem hiding this 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!
|
There was a problem hiding this 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!
Utilize a code spell-checking tool to scan for and correct spelling errors, focusing on
Kconfig
files.