Skip to content
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

[rtl872x] fix sleep PSRAM #2647

Merged
merged 4 commits into from
Apr 11, 2023
Merged

[rtl872x] fix sleep PSRAM #2647

merged 4 commits into from
Apr 11, 2023

Conversation

avtolstoy
Copy link
Member

Description

  1. Application thread stack is statically allocated from SRAM
  2. Heap allocations prefer SRAM first now, and then PSRAM, previously it was the opposite
  3. ^^ Makes System thread also have stack on SRAM, not PSRAM
  4. Minor: system thread has a distinct thread name vs default active_object
  5. PSRAM suspend/resume procedure issues resolved
  6. Stub out direct flash read from BLE SDK code (we do not allow direct flash access without our HAL functions)

Steps to Test

  • wiring/watchdog
  • runner/mailbox with System.reset() replaced with System.sleep()

Example App

N/A

References

N/A


Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA (Info here)
  • Problem and Solution clearly stated
  • Run unit/integration/application tests on device
  • Added documentation
  • Added to CHANGELOG.md after merging (add links to docs and issues)

@avtolstoy avtolstoy added the bug label Apr 11, 2023
@avtolstoy avtolstoy added this to the 5.3.1 milestone Apr 11, 2023
hal/src/rtl872x/sleep_hal.cpp Show resolved Hide resolved
hal/src/rtl872x/sleep_hal.cpp Show resolved Hide resolved
hal/src/rtl872x/sleep_hal.cpp Show resolved Hide resolved
@scott-brust scott-brust merged commit e6c5469 into develop Apr 11, 2023
@scott-brust scott-brust deleted the fix/rtl872x-sleep-psram branch April 11, 2023 19:03
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants