Skip to content

Various LPM Updates #472

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
Merged
merged 23 commits into from
Oct 17, 2025
Merged

Various LPM Updates #472

merged 23 commits into from
Oct 17, 2025

Conversation

kwillis01
Copy link
Contributor

@kwillis01 kwillis01 commented Oct 9, 2025 •
edited
Loading

AM62L LPM Changes:

  • Add back system suspend workaround page for AM62L since RTC Only + DDR entry requires the instructions
  • Remove Linux changes that are no longer needed from the system suspend workaround page
  • Fix grammar in the system suspend workaround page
  • Update entry to RTC Only + DDR where system suspend workarounds should be consulted first then USB wakeup should be disabled
  • Add memory usage table for various operating modes

AM62* LPM Changes:

  • Update I/O Only + DDR entry to denote that the Main UART wakeup needs to be disabled
  • Fix USB wakeup command style

Wakeup Changes:

  • Update Main UART wakeup enable instructions
  • Update Main GPIO wakeup explanations
  • Allow wakeup source explanation to only appear if it is available for the SoC
  • Add AM62L console logs or code where needed
  • Add table for AM62L wakeup sources
  • Generalize references to LPM in the wakeup file so that explanations can apply to AM62L
  • Add wakeup page to AM62L toc

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 9, 2025

New unreachable files found with check-files.sh:

source/images/am62l_lpm_wakeup_evm_pin.jpg

cshilwant
cshilwant previously requested changes Oct 9, 2025
View reviewed changes
@kwillis01 kwillis01 changed the title AM62L LPM Updates LPM Updates Oct 9, 2025
@kwillis01 kwillis01 changed the title LPM Updates Various LPM Updates Oct 9, 2025
@praneethbajjuri
Copy link
Collaborator

@vishalmti , @ti-sebin , @DhruvaG2000 please review

@kwillis01 kwillis01 force-pushed the am62l-lpm-updates branch 3 times, most recently from cefb09a to a0b73f6 Compare October 9, 2025 20:48
StaticRocket
StaticRocket requested changes Oct 9, 2025
View reviewed changes
@praneethbajjuri
Copy link
Collaborator

@vishalmti , @ti-sebin , @DhruvaG2000 please review

Ping on review

@kwillis01
feat(linux): Revert "docs(power_management): AM62Lx: Update LPM featu…
646ebf4 
…re support information"

LPM is supported in the SDK 11.1 release for AM62L. As such, add back
the release specific information for AM62L.

This reverts commit 68b1fe0.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
fix(linux): Fix broken ATF link
552d9ee 
Fix broken ATF link to in LPM workarounds so that it can be referenced to
build ATF.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
fix(pm_am62lx_low_power_modes): Note LPM workaround for RTC Only + DDR
44bd0e7 
Remove note that DeepSleep and RTC Only + DDR for AM62L requires workarounds.
RTC Only + DDR still needs workarounds to enter into, so add note to consult
the LPM suspend workarounds before attempting to use RTC Only + DDR.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
feat(pm_am62lx_low_power_modes): Disable USB for RTC + DDR entry
d0a1148 
To enter RTC Only + DDR, the USB wakeup sources need to be disabled.
Note this in the instructions for RTC Only + DDR.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
feat(pm_am62lx_low_power_modes): Add memory usage table for LPM
32dc7f6 
Add table that shows how memory is used in different modes of operation
for AM62L.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
feat(linux): Revert "fix(AM62LX): remove invalid slug from toc"
672389c 
The AM62LX linux workaround page has been added again for LPM. Revert
this commit so that the workarounds are in the toc.

This reverts commit bade6f7.

Signed-off-by: Kendall Willis <k-willis@ti.com>
StaticRocket
StaticRocket requested changes Oct 13, 2025
View reviewed changes
@kwillis01
feat(pm_wakeup_sources): Update Main UART wakeup enable
9a0c4a1 
In SDK 11.1, the way to enable Main UART wakeup has changed. To enable
Main UART wakeup, a "wakeup" pinctrl state is added to the device tree.
Remove all references the previous way to enable Main UART.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
feat(pm_wakeup_sources): Update Main GPIO wakeup explanation
a686435 
Since the way to enable Main UART wakeup has changed, add in any
explanations that applied to Main GPIO to the Main GPIO section.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
feat(pm_wakeup_sources): Add AM62L rtcwake output
bd4bfd9 
Add the output when 'rtcwake' is used on AM62L.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
feat(pm_wakeup_sources): Disable invalid wakeup sources for AM62L
186d001 
For all wakeup sources that do not apply to AM62L, add logic to remove
the explanation and note that the wakeup source is not applicable. The
wakeup source explanations will still appear for other SoCs.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
feat(pm_wakeup_sources): Move I/O Daisy Chain info for AM62L
7d7e2d4 
Some information in the I/O Daisy Chain section does not apply to AM62L
since AM62L does not have an MCU Only mode and Main GPIO is not an
enabled wakeup source. Move that information into sections for other
SoCs.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
feat(pm_wakeup_sources): Add table for AM62L wakeup sources
db6b509 
Add a table for AM62L that shows each wakeup source and what low power
mode they work for.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
feat(linux): Add LPM wakeup sources page to toc
1790f82 
AM62L has multiple wakeup sources noted in the LPM wakeup sources page.
Add this to the toc so that it is part of the AM62L documentation.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
refactor(pm_wakeup_sources): Generalize references to LPM
8fbaf8b 
AM62L does not have MCU Only mode. In the pm_wakeup_sources file, there
are many references to MCU Only mode. Make these references general so
that these sentences can apply to all SoCs that use LPM.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
fix(pm_low_power_modes): Update disabled wakeup sources for I/O + DDR
cfa1b1c 
The Main UART needs to be disabled in order to I/O Only + DDR as of SDK
11.1. Update the commands of which wakeup sources need to be disabled.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
style(pm_wakeup_sources): Change USB command prefix
b866211 
The commands for the USB wakeup source all have a prefix of "#". Remove
the "HOST:~$" prefix and add "#" to one of the code blocks since it was
not consistent with the others.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
refactor(pm_wakeup_sources): Fix inconsistent style for DTS node
d5c3420 
For the MCU GPIO wakeup node, there is an explanation of what each
property does. The explanation mixed different styles. Fix the styles
and explanation to be consistent.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
refactor(pm_wakeup_sources): Rephrase MCU GPIO wakeup
22414ac 
Reword the MCU GPIO wakeup section to use assertive language instead of
passive.

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
fix(Power_Management): Change LPM reference for AM62L
9d16b44 
Fix the reference for AM62L LPM section to be the same as the reference
for the AM62* LPM section. This will allow any section that uses all
AM62 devices to reference the LPM section that is relevant without
having to add ifconfig blocks.

Fixes: 5bd8e23 ("feat(Linux): Add AM62L specific LPM documentation")

Signed-off-by: Kendall Willis <k-willis@ti.com>
@kwillis01
fix(pm_wakeup_sources): Use pinned versions of MCU+ SDK
2051f39 
Pinned versions of urls should be used instead of latest versions.
Update the links for MCU+ SDK to use the pinned version.

Signed-off-by: Kendall Willis <k-willis@ti.com>
StaticRocket
StaticRocket approved these changes Oct 13, 2025
View reviewed changes
Copy link
Member

@StaticRocket StaticRocket left a comment

Choose a reason for hiding this comment

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

I'm fine with this I guess. Ideally things should be bisectable, but I'm sure people have opinions about that.

@cshilwant
Copy link
Member

+ @akashdeep-ti / @DhruvaG2000 / @ti-sebin / @jeevantelukula for reviews

DhruvaG2000
DhruvaG2000 suggested changes Oct 14, 2025
View reviewed changes
Copy link
Collaborator

@DhruvaG2000 DhruvaG2000 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

It would really help in reviewing effectively if this PR was split into multiple PRs from next time. For now it's ok.

@jeevantelukula jeevantelukula merged commit 3f9baef into TexasInstruments:master Oct 17, 2025
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@StaticRocket StaticRocket StaticRocket approved these changes

@jeevantelukula jeevantelukula jeevantelukula approved these changes

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

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

@aniket-l aniket-l Awaiting requested review from aniket-l aniket-l is a code owner

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

@bryanbrattlof bryanbrattlof Awaiting requested review from bryanbrattlof

@devarsht devarsht Awaiting requested review from devarsht

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

@vishalmti vishalmti Awaiting requested review from vishalmti

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

+1 more reviewer

@DhruvaG2000 DhruvaG2000 DhruvaG2000 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@praneethbajjuri praneethbajjuri

@devarsht devarsht

@r-vignesh r-vignesh

@bryanbrattlof bryanbrattlof

@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.