Skip to content

feat(linux): Add standby mode docs for AM62L#656

Open
ti-scaria wants to merge 1 commit intoTexasInstruments:masterfrom
ti-scaria:standby
Open

feat(linux): Add standby mode docs for AM62L#656
ti-scaria wants to merge 1 commit intoTexasInstruments:masterfrom
ti-scaria:standby

Conversation

@ti-scaria
Copy link
Copy Markdown

Introduce standby mode in AM62L and how it is used.

Extending the CPUidle documentation for AM62L.

@github-actions
Copy link
Copy Markdown

New warnings found with rstcheck:

source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst:160: (WARNING/2) Title overline too short.
source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst:160: (WARNING/2) Title overline too short.
source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst:230: (WARNING/2) Title overline too short.
source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst:230: (WARNING/2) Title overline too short.
source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst:300: (WARNING/2) Title overline too short.
source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst:300: (WARNING/2) Title overline too short.
source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst:337: (WARNING/2) Title overline too short.
source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst:337: (WARNING/2) Title overline too short.

Copy link
Copy Markdown
Collaborator

@praneethbajjuri praneethbajjuri left a comment

Choose a reason for hiding this comment

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

bunch of issues with text format . Review and address vale comments as well

Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Copy link
Copy Markdown
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.

One other consideration is whether the pm_am62l_standby.rst page will be used by future SoCs, however the references to only AM62L should be good for now, just might have to be updated in the future.

Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_cpuidle.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Copy link
Copy Markdown
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.

Overall the content LGTM, just a minor fix needed for scheduled suspend modes since we don't use that terminology anywhere else so it would probably be confusing for the user. One other thing is that maybe it should be mentioned explicitly that any interrupt to a CPU returns the system to an active state. This would make it so users aren't confused on what interrupts would wake the system up.

Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
@ti-scaria ti-scaria force-pushed the standby branch 2 times, most recently from 8acca1d to 0d8ed0a Compare April 20, 2026 06:06
@ti-scaria ti-scaria requested a review from StaticRocket April 20, 2026 12:02
Copy link
Copy Markdown
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.

Content looks good!

Copy link
Copy Markdown
Member

@cshilwant cshilwant left a comment

Choose a reason for hiding this comment

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

Please plan to resolve merge conflicts

StaticRocket
StaticRocket previously approved these changes Apr 21, 2026
jeevantelukula
jeevantelukula previously approved these changes Apr 22, 2026
Comment thread source/linux/Foundational_Components/Power_Management/pm_am62l_standby.rst Outdated
Introduce standby mode in AM62L and how it is used.

Extending the CPUidle documentation for AM62L.

Signed-off-by: Scaria Kochidanadu <s-kochidanadu@ti.com>
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.