feat(linux): s2idle: Document the mode selection logic#641
Conversation
kwillis01
left a comment
There was a problem hiding this comment.
It might be good to see if there are other places in the previous parts of the doc that can be updated or that the new info can loop into and consolidate.
995b85e to
6209735
Compare
kwillis01
left a comment
There was a problem hiding this comment.
Mostly looks good, I think that the QoS Latency Constraints and Mode Selection section needs to be fixed up to flow better and be a little more condensed.
|
Additionally, could you update the RTC + DDR section in |
6209735 to
8532d2e
Compare
|
New warnings found with rstcheck: |
8532d2e to
27dc856
Compare
|
New warnings found with rstcheck: |
3de7535 to
f4ad757
Compare
kwillis01
left a comment
There was a problem hiding this comment.
It might be good to put explicit instructions on how to enter RTC + DDR via s2idle (besides the constraint setting) in pm_am62lx_low_power_modes. It would give users a quick way to grab the instructions and go.
|
@DhruvaG2000 can you re-spin after checking comments from @kwillis01 @ti-scaria |
f4ad757 to
2c5e7bd
Compare
2c5e7bd to
19bd7c0
Compare
Document the mode selection logic using the s2idle flow Signed-off-by: Dhruva Gole <d-gole@ti.com>
Add mention of the s2idle based mode selection being part of low power mode support as of this release Signed-off-by: Dhruva Gole <d-gole@ti.com>
Also removes myself from AM62X device owner. Add Scaria as another PM component owner Signed-off-by: Dhruva Gole <d-gole@ti.com>
19bd7c0 to
34ae59d
Compare
Document the mode selection logic using the s2idle flow