LPM various cleanup and fixes#381
Merged
Merged
Conversation
kwillis01
commented
Jul 15, 2025
kwillis01
commented
Contributor
- add schedutil governor to the documentation for CPUFreq governor
- update references from SDK 11.0 to SDK 11.1
- remove unimplemented ways to enter MCU Only and Partial I/O
- remove false assertion that PMIC_LPM_EN signal is 0V when device is in DeepSleep
kwillis01
requested review from
StaticRocket,
VeeruPrudhvi,
aniket-l,
cshilwant,
praneethbajjuri and
uditkumarti
as code owners
github-actions
Bot
assigned
devarsht,
DhruvaG2000,
praneethbajjuri,
r-vignesh,
uditkumarti and
vishalmti
github-actions
Bot
requested review from
DhruvaG2000,
devarsht,
r-vignesh and
vishalmti
|
New warnings found with rstcheck: |
Member
|
Please address the vale comments. It did catch at least one valid spelling error. |
akashdeep-ti
suggested changes
Jul 15, 2025
Member
|
There are still pending issues with passive voice |
kwillis01
force-pushed
the
cleanup
branch
2 times, most recently
from
f2618d5 to
02bc6e2
Compare
|
New warnings found with rstcheck: |
1 similar comment
|
New warnings found with rstcheck: |
Documenting ways to enter low power modes that are not implemented is unnecessary and can be confusing to users. Remove the unimplemented features and add them back when they are implemented. Signed-off-by: Kendall Willis <k-willis@ti.com>
The PMIC_LPM_EN signal stays high when the device is in DeepSleep or MCU Only for AM62P and AM62A. It can only go low in DeepSleep when programmed on AM62X. Reword and move the extra confirmation to only appear in AM62X documentation. Signed-off-by: Kendall Willis <k-willis@ti.com>
schedutil is a CPUFreq governor in Linux. schedutil was left out of the documentation originally. Signed-off-by: Kendall Willis <k-willis@ti.com>
References to the files in the Linux SDK were pointing to SDK 11.0. Fix them to point to 11.1. Signed-off-by: Kendall Willis <k-willis@ti.com>
This file did not have the three space indentation as specified by the contributing guide [1]. Fix all cases of this. [1] https://github.com/TexasInstruments/processor-sdk-doc/blob/master/CONTRIBUTING.md#indentation-and-whitespace Signed-off-by: Kendall Willis <k-willis@ti.com>
Referenced files were put into the documentation as normal text. Add the :file: styling to the referenced files so that it aligns with rest of documentation. Signed-off-by: Kendall Willis <k-willis@ti.com>
Add in a code block for a Linux command that is referenced. Signed-off-by: Kendall Willis <k-willis@ti.com>
StaticRocket
approved these changes
Jul 16, 2025
praneethbajjuri
approved these changes
Jul 17, 2025
praneethbajjuri
merged commit aef493d
into
TexasInstruments:master
5 of 6 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.