Skip to content

PM: Introduce docs for s2idle #562

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.

Sign up for GitHub

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

Jump to bottom
Draft
DhruvaG2000 wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

PM: Introduce docs for s2idle #562

DhruvaG2000 wants to merge 1 commit into from
+308 −0

Conversation

@DhruvaG2000
Copy link
Collaborator

@DhruvaG2000 DhruvaG2000 commented Dec 8, 2025

Document all about s2idle and PSCI and how the whole stack helps us in selecting between low power modes on the TI AM62L

@DhruvaG2000 DhruvaG2000 changed the title feat(linux): Add s2idle docs [WIP][DNM] Add s2idle docs Dec 8, 2025
@github-actions
Copy link

github-actions bot commented Dec 8, 2025

New warnings found with rstcheck:

source/linux/Foundational_Components/Power_Management/pm_psci_s2idle.rst:25: (ERROR/3) Unexpected indentation.

cshilwant
cshilwant reviewed Dec 8, 2025
View reviewed changes
source/linux/Foundational_Components/Power_Management/pm_psci_s2idle.rst Outdated
Suspend-to-Idle (s2idle), also known as "freeze," is a generic, pure software, light-weight variant of system suspend.
In this state, the Linux kernel freezes user space tasks, suspends devices, and then puts all CPUs into their deepest available idle state.

**********************
Copy link
Member

@cshilwant cshilwant Dec 8, 2025

Choose a reason for hiding this comment

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

The section identifier line should be of equal length to the section name.

@DhruvaG2000
Copy link
Collaborator Author

DhruvaG2000 commented Dec 9, 2025

@cshilwant , and rest I'll mark the PR ready for review once it has undergone LPM team internal review. Till then requesting you to hold off reviews.
This is going to be a major documentation update for AM62L PM story hence might take some time to get reviews and complete.

Thanks!

kwillis01
kwillis01 suggested changes Dec 9, 2025
View reviewed changes
@github-actions
Copy link

github-actions bot commented Dec 15, 2025

New warnings found with rstcheck:

source/linux/Foundational_Components/Power_Management/pm_psci_s2idle.rst:48: (WARNING/2) Bullet list ends without a blank line; unexpected unindent.
source/linux/Foundational_Components/Power_Management/pm_psci_s2idle.rst:29: (ERROR/3) Error parsing content block for the "list-table" directive: two-level bullet list expected, but row 4 does not contain a second-level bullet list.

@DhruvaG2000 DhruvaG2000 force-pushed the s2idle_doc branch 2 times, most recently from c7f9288 to 4a0b798 Compare December 23, 2025 13:14
@github-actions
Copy link

github-actions bot commented Dec 23, 2025

New warnings found with rstcheck:

source/linux/Foundational_Components/Power_Management/pm_psci_s2idle.rst:302: (WARNING/2) Bullet list ends without a blank line; unexpected unindent.
source/linux/Foundational_Components/Power_Management/pm_psci_s2idle.rst:283: (ERROR/3) Error parsing content block for the "list-table" directive: two-level bullet list expected, but row 4 does not contain a second-level bullet list.

kwillis01
kwillis01 suggested changes Dec 29, 2025
View reviewed changes
Copy link
Contributor

@kwillis01 kwillis01 left a comment

Choose a reason for hiding this comment

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

Besides the execution level things, everything else looks good to me! Good work on adding all of this, it's very infomative

@DhruvaG2000
feat(linux): Add s2idle docs
09ca039 
Introduce the concept of s2idle and how we use it for mode
selection

Signed-off-by: Dhruva Gole <d-gole@ti.com>
@DhruvaG2000 DhruvaG2000 changed the title [WIP][DNM] Add s2idle docs PM: Introduce docs for s2idle Dec 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cshilwant cshilwant cshilwant left review comments

@devarsht devarsht Awaiting requested review from devarsht

@praneethbajjuri praneethbajjuri Awaiting requested review from praneethbajjuri praneethbajjuri is a code owner

@r-vignesh r-vignesh Awaiting requested review from r-vignesh

@uditkumarti uditkumarti Awaiting requested review from uditkumarti uditkumarti is a code owner

@vishalmti vishalmti Awaiting requested review from vishalmti

@gehariprasath gehariprasath Awaiting requested review from gehariprasath gehariprasath is a code owner

@StaticRocket StaticRocket Awaiting requested review from StaticRocket StaticRocket will be requested when the pull request is marked ready for review StaticRocket is a code owner

@VeeruPrudhvi VeeruPrudhvi Awaiting requested review from VeeruPrudhvi VeeruPrudhvi will be requested when the pull request is marked ready for review VeeruPrudhvi is a code owner

@jeevantelukula jeevantelukula Awaiting requested review from jeevantelukula jeevantelukula will be requested when the pull request is marked ready for review jeevantelukula is a code owner

+1 more reviewer

@kwillis01 kwillis01 kwillis01 requested changes

Reviewers whose approvals may not affect merge requirements

At least 2 approving reviews are required to merge this pull request.

Assignees

@praneethbajjuri praneethbajjuri

@devarsht devarsht

@r-vignesh r-vignesh

@gehariprasath gehariprasath

@DhruvaG2000 DhruvaG2000

@uditkumarti uditkumarti

@vishalmti vishalmti

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@DhruvaG2000 @kwillis01 @cshilwant @praneethbajjuri @devarsht @r-vignesh @gehariprasath @uditkumarti @vishalmti