-
Notifications
You must be signed in to change notification settings - Fork 33
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.
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
Lpm documentation update #76
Lpm documentation update #76
Conversation
38cd301
to
e1af43c
Compare
New warnings found with rstcheck:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The commit-check workflow throws an error. Please check https://github.com/TexasInstruments/processor-sdk-doc/actions/runs/12198381961?pr=76
for more details on what's needed to be fixed in commit message.
Checkout https://github.com/TexasInstruments/processor-sdk-doc/blob/master/CONTRIBUTING.md for more details on CONTRIBUTING guidelines
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@makohoek , @DhruvaG2000 to review
e1af43c
to
ef3a704
Compare
New warnings found with rstcheck:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, with scope for a few improvements.
Left a few comments inline.
Thanks
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Show resolved
Hide resolved
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Show resolved
Hide resolved
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Power_Management/pm_wakeup_sources.rst
Show resolved
Hide resolved
ef3a704
to
a78d57c
Compare
@praneethbajjuri Me to review? I'm not particular confident in my LPM skills, I mainly work on Android. I can spend some time reviewing if it helps, but I'm not 100% sure I'm the best person to review this ^^' |
Sorry @makohoek , mistakenly tagged you here. Need review from Markus, will ping him |
New warnings found with rstcheck:
|
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Show resolved
Hide resolved
a78d57c
to
327544d
Compare
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Power_Management/pm_low_power_modes.rst
Show resolved
Hide resolved
New warnings found with rstcheck:
|
327544d
to
198c449
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
198c449
to
0f40431
Compare
8f5a342
to
045af67
Compare
New warnings found with rstcheck:
|
source/linux/Foundational_Components/Power_Management/pm_wakeup_sources.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/Power_Management/pm_wakeup_sources.rst
Show resolved
Hide resolved
045af67
to
35752c8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the cleanup. LGTM!
New warnings found with rstcheck:
|
Add documentation for I/O Only DDR low power mode applicable to AM62AX and AM62PX. This is a guide for customers on how to suspend resume from this mode, advantages of selecting this mode and starter kit specific connections required. Signed-off-by: Akashdeep Kaur <[email protected]>
Update the low power mode susepnd resume logs to the latest so as to include changes in this SDK. Signed-off-by: Akashdeep Kaur <[email protected]>
Change the wakeup sources list to the table so as to specify which wakeup source is supported in given low power mode. Signed-off-by: Akashdeep Kaur <[email protected]>
…rder Reorder the low power modes from lowest power consuming mode to highest power consuming mode. Signed-off-by: Akashdeep Kaur <[email protected]>
Add new low power mode I/O only plus DDR low power mode to the list of features for current release. Signed-off-by: Akashdeep Kaur <[email protected]>
22426af
35752c8
to
22426af
Compare
73809f9
into
TexasInstruments:master
Update Low power mode documentation to include the following: