Skip to content

Conversation

kylebonnici
Copy link
Contributor

The ok state is deprecated and very few files are using this. The DTS spec also does not have this value.

This PR removes this value once and for all.

pillo79
pillo79 previously approved these changes Oct 7, 2025
Copy link
Contributor

@mbolivar mbolivar left a comment

Choose a reason for hiding this comment

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

Please mention this in the release notes file for the record. No objections to the idea.

The `ok` state is deprecated and very few files are using this.
The DTS spec also does not have this value.

This PR removes this value once and for all.

Signed-off-by: Kyle Micallef Bonnici <[email protected]>
@kylebonnici
Copy link
Contributor Author

Please mention this in the release notes file for the record. No objections to the idea.

Done

@zephyrbot zephyrbot requested a review from teburd October 7, 2025 17:20
Copy link

sonarqubecloud bot commented Oct 7, 2025

Copy link
Contributor

@DBS06 DBS06 left a comment

Choose a reason for hiding this comment

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

Nice! LGTM 👍

@cfriedt cfriedt merged commit 1476fcb into zephyrproject-rtos:main Oct 9, 2025
44 checks passed
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.

7 participants