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

[system] still pump system loop when SPARK_WLAN_SLEEP=1; Try to drain ISR task queue quicker #2702

Merged
merged 2 commits into from Oct 2, 2023

Conversation

avtolstoy
Copy link
Member

@avtolstoy avtolstoy commented Sep 29, 2023

Problem

System loop does not run within delay() with threading disabled and SPARK_WLAN_SLEEP=1.

Steps to Test/Example App

See SC122147

References

  • [SC122147]

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)

@@ -587,7 +593,7 @@ void system_delay_pump(unsigned long ms, bool force_no_background_loop=false)
HAL_Delay_Milliseconds(1);
}

if (SPARK_WLAN_SLEEP || force_no_background_loop)
if (force_no_background_loop)
Copy link
Member

Choose a reason for hiding this comment

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

😄

system/src/system_task.cpp Outdated Show resolved Hide resolved
@avtolstoy avtolstoy merged commit a6fd0a0 into develop Oct 2, 2023
13 checks passed
@avtolstoy avtolstoy deleted the fix/delay-pump-system-loop-wlan-sleep branch October 2, 2023 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants