Skip to content

feat(android): Add note about OP-TEE logs disabling suspend/resume #614

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

Merged
cshilwant merged 1 commit into from
Jan 9, 2026
Merged

feat(android): Add note about OP-TEE logs disabling suspend/resume #614

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 6 additions & 7 deletions source/android/Application_Notes_Android_Low_Power.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,12 @@

#. **Suspend to RAM (S2R):** All power domains are off except for wake-up domain that has wake-up co-processor (DM R5) running. DDR is in self refresh.

.. note::

OP-TEE OS debug logs are disabled by default. However, if you enable them for debugging

Check warning on line 13 in source/android/Application_Notes_Android_Low_Power.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.PassiveVoice] 'are disabled' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice.

Raw Output:
{"message": "[RedHat.PassiveVoice] 'are disabled' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice.", "location": {"path": "source/android/Application_Notes_Android_Low_Power.rst", "range": {"start": {"line": 13, "column": 25}}}, "severity": "INFO"}

Check warning on line 13 in source/android/Application_Notes_Android_Low_Power.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.CaseSensitiveTerms] Use 'operating system' rather than 'OS'.

Raw Output:
{"message": "[RedHat.CaseSensitiveTerms] Use 'operating system' rather than 'OS'.", "location": {"path": "source/android/Application_Notes_Android_Low_Power.rst", "range": {"start": {"line": 13, "column": 11}}}, "severity": "WARNING"}
purposes (CFG_TEE_CORE_LOG_LEVEL > 0), the suspend/resume functionality will not work

Check warning on line 14 in source/android/Application_Notes_Android_Low_Power.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.Slash] Use either 'or' or 'and' in 'suspend/resume'

Raw Output:
{"message": "[RedHat.Slash] Use either 'or' or 'and' in 'suspend/resume'", "location": {"path": "source/android/Application_Notes_Android_Low_Power.rst", "range": {"start": {"line": 14, "column": 47}}}, "severity": "WARNING"}

Check warning on line 14 in source/android/Application_Notes_Android_Low_Power.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.Definitions] Define acronyms and abbreviations (such as 'TEE') on first occurrence if they're likely to be unfamiliar.

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'TEE') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/android/Application_Notes_Android_Low_Power.rst", "range": {"start": {"line": 14, "column": 18}}}, "severity": "INFO"}
properly. Make sure to disable OP-TEE OS logs when testing low power modes.

Check warning on line 15 in source/android/Application_Notes_Android_Low_Power.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.CaseSensitiveTerms] Use 'operating system' rather than 'OS'.

Raw Output:
{"message": "[RedHat.CaseSensitiveTerms] Use 'operating system' rather than 'OS'.", "location": {"path": "source/android/Application_Notes_Android_Low_Power.rst", "range": {"start": {"line": 15, "column": 42}}}, "severity": "WARNING"}

Check warning on line 15 in source/android/Application_Notes_Android_Low_Power.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.Definitions] Define acronyms and abbreviations (such as 'TEE') on first occurrence if they're likely to be unfamiliar.

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'TEE') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/android/Application_Notes_Android_Low_Power.rst", "range": {"start": {"line": 15, "column": 38}}}, "severity": "INFO"}

*****************************
Enable Suspend/Resume support
*****************************
Expand All @@ -29,13 +35,6 @@
[ 171.970763] psci: CPU1 killed (polled 4 ms)
[ 171.993689] psci: CPU2 killed (polled 0 ms)
[ 172.014894] psci: CPU3 killed (polled 4 ms)
[I/TC: Secondary CPU 1 initializing
I/TC: Secondary CPU 1 switching to normal world boot
I/TC: Secondary CPU 2 initializing
I/TC: Secondary CPU 2 switching to normal world boot
I/T[ 200.035482] am65-cpsw-nuss 8000000.ethernet: set new flow-id-base 19
C: Secondary CPU 3 initializing
I/TC: Secondary CPU 3 switching to normal world boot
[ 200.067533] am65-cpsw-nuss 8000000.ethernet eth0: PHY [8000f00.mdio:00] driver [TI DP83867] (irq=POLL)
[ 200.076971] am65-cpsw-nuss 8000000.ethernet eth0: configuring for phy/rgmii-rxid link mode
[ 200.103015] am65-cpsw-nuss 8000000.ethernet eth1: PHY [8000f00.mdio:01] driver [TI DP83867] (irq=POLL)
Expand Down