-
Notifications
You must be signed in to change notification settings - Fork 7.5k
RFC: Renaming PM APIs #11972
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
RFC: Renaming PM APIs #11972
Conversation
Remove leading underscore from PM APIs. _ was used for internal APIs. Signed-off-by: Anas Nashif <[email protected]>
sys_soc is just redundant, just call APIs with sys_*. Signed-off-by: Anas Nashif <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #11972 +/- ##
==========================================
+ Coverage 48.07% 48.47% +0.39%
==========================================
Files 281 293 +12
Lines 43349 50722 +7373
Branches 10392 13038 +2646
==========================================
+ Hits 20841 24586 +3745
- Misses 18350 21351 +3001
- Partials 4158 4785 +627
Continue to review full report at Codecov.
|
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.
doc changes LGTM
No objections to this change. |
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.
This seems a safe change to me.
@nashif why is this tagged as DNM? |
recheck |
Fixes #11047