feat(pm_cpuidle): Enabling and Utilizing AM62 Standby Mode#363
Merged
VeeruPrudhvi merged 2 commits intoJul 14, 2025
Conversation
DhruvaG2000
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
praneethbajjuri
requested changes
Jul 11, 2025
praneethbajjuri
previously approved these changes
Jul 11, 2025
cshilwant
reviewed
Jul 11, 2025
kwillis01
reviewed
Jul 11, 2025
kwillis01
left a comment
kwillis01
left a comment
Contributor
There was a problem hiding this comment.
Documentation overall looks great!! Good job!!
It is optional to fix the things in the comments I left because I will also be sending a PR for this page and I can fix it then.
Add comprehensive documentation for CPUIdle on AM62X, AM62AX, and AM62PX platforms explaining: - Standby mode implementation through CPUIdle WFI state - Integration between Linux CPUIdle framework and TF-A - PSCI interaction via cpu_standby and validate_power_state hooks - Source file references for both Linux and TF-A components - Kernel configuration options and usage information This documentation helps users understand that the TI "standby" mode is automatically handled by the CPUIdle framework without requiring special configuration. Signed-off-by: Dhruva Gole <d-gole@ti.com>
DhruvaG2000
force-pushed
the
standby_cpuidle
branch
from
c23ed60 to
2737a91
Compare
|
New warnings found with rstcheck: |
DhruvaG2000
force-pushed
the
standby_cpuidle
branch
from
2737a91 to
d8ea568
Compare
Mention the standby power mode in the release summary pages for all relevant sitara devices. While at it, also replace the IO+DDR release specific note because it no longer applies as release specific for this particular release Signed-off-by: Dhruva Gole <d-gole@ti.com>
DhruvaG2000
force-pushed
the
standby_cpuidle
branch
from
d8ea568 to
a968ac4
Compare
VeeruPrudhvi
changed the title
akashdeep-ti
approved these changes
Jul 14, 2025
ti-sebin
approved these changes
Jul 14, 2025
cshilwant
approved these changes
Jul 14, 2025
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
11 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.
Add comprehensive documentation for CPUIdle on AM62X, AM62AX, and AM62PX platforms explaining:
This documentation helps users understand that the TI "standby" mode is automatically handled by the CPUIdle framework without requiring special configuration.